Skip to content

Update canned gossmaps#8878

Merged
rustyrussell merged 4 commits intoElementsProject:masterfrom
rustyrussell:guilt/update-canned-gossmaps
Feb 4, 2026
Merged

Update canned gossmaps#8878
rustyrussell merged 4 commits intoElementsProject:masterfrom
rustyrussell:guilt/update-canned-gossmaps

Conversation

@rustyrussell
Copy link
Contributor

@rustyrussell rustyrussell commented Feb 4, 2026

Our previous "real" gossip maps were from October 2024. Update them with the latest from my node.

This alters tests a little, but I also sped them up a great deal, since they were taking far too long. And the change also revealed a bug in our "channeld_fakenet" which had a divide by zero bug.

Changelog-None: testsuite changes only

1. If max was 0, we crashed with SIGFPE due to % 0.
2. If min was non-zero, logic was incorrect (but all callers had min == 0).

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Reduce randome delay from 0.1 - 1 seconds, to 0.01 to 0.1 seconds.

Running tests/test_xpay.py::test_xpay_fake_channeld[False]

Before:
	348.41s
After:
	76.76s

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Taked /tmp/gossip_store and created canned gossmap for testing.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell added this to the v26.04 milestone Feb 4, 2026
@rustyrussell rustyrussell enabled auto-merge (rebase) February 4, 2026 04:11
@rustyrussell rustyrussell force-pushed the guilt/update-canned-gossmaps branch from a6adb6f to aa7b2d4 Compare February 4, 2026 05:11
We delete the test_xpay_maxfee test which required the specific
topology.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell force-pushed the guilt/update-canned-gossmaps branch from aa7b2d4 to ade95a0 Compare February 4, 2026 11:38
@rustyrussell rustyrussell merged commit 96adac4 into ElementsProject:master Feb 4, 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