Skip to content

Test flakes revealed by gossmap compaction#8888

Merged
rustyrussell merged 2 commits intoElementsProject:masterfrom
rustyrussell:flakes28
Feb 9, 2026
Merged

Test flakes revealed by gossmap compaction#8888
rustyrussell merged 2 commits intoElementsProject:masterfrom
rustyrussell:flakes28

Conversation

@rustyrussell
Copy link
Contributor

Since CI seems upset on these, trying in a standalone PR.

Changelog-None: tests only

```
>       assert len(layers['layers']) == 1
E       AssertionError: assert 2 == 1
E        +  where 2 = len([{'layer': 'xpay', 'persistent': True, 'disabled_nodes': [], 'created_channels': [], 'channel_updates': [], 'constraints': [{'short_channel_id_dir': '45210x2134x44171/0', 'timestamp': 1770341134, 'minimum_msat': 289153519}, {'short_channel_id_dir': '1895x7x1895/1', 'timestamp': 1770341134, 'minimum_msat': 289007015}, {'short_channel_id_dir': '1906x1039x1906/1', 'timestamp': 1770341134, 'minimum_msat': 289008304}, {'short_channel_id_dir': '10070x60x10063/1', 'timestamp': 1770341134, 'minimum_msat': 289005726}, {'short_channel_id_dir': '18772x60x18743/0', 'timestamp': 1770341134, 'minimum_msat': 289005726}, {'short_channel_id_dir': '18623x208x18594/0', 'timestamp': 1770341134, 'minimum_msat': 289004859}, {'short_channel_id_dir': '33935x826x33727/1', 'timestamp': 1770341134, 'maximum_msat': 491501488}], 'biases': [], 'node_biases': []}, {'layer': 'xpay-94', 'persistent': False, 'disabled_nodes': [], 'created_channels': [], 'channel_updates': [], 'constraints': [], 'biases': [], 'node_biases': []}])
```

Signed-off-by: Rusty Russell <[email protected]>
…eout.

Avoids guessing what the timeout should be, use a file trigger.  This
is more optimal, and should reduce a flake in test_sql under valgrind.

Signed-off-by: Rusty Russell <[email protected]>
@rustyrussell rustyrussell added this to the v26.04 milestone Feb 8, 2026
@rustyrussell rustyrussell enabled auto-merge (rebase) February 9, 2026 04:06
@rustyrussell rustyrussell merged commit 939aec3 into ElementsProject:master Feb 9, 2026
83 of 85 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant