Skip to content

tests: wait for peer disconnect in test_blindedpath_noaddr - #9526

Merged
daywalker90 merged 1 commit into
ElementsProject:masterfrom
daywalker90:fix-flake-test_migration_no_bkpr
Sep 18, 2026
Merged

daywalker90 merged 1 commit into
ElementsProject:masterfrom
daywalker90:fix-flake-test_migration_no_bkpr

Conversation

@daywalker90

Copy link
Copy Markdown
Collaborator

l2.rpc.disconnect() returns before l3 notices the socket close, so l3 could still see l2 as connected and add a route hint, flaking the check that a disconnected peer is not used as a blinded path entry.

Changelog-None

Fixes: #9524

l2.rpc.disconnect() returns before l3 notices the socket close, so l3
could still see l2 as connected and add a route hint, flaking the check
that a disconnected peer is not used as a blinded path entry.

Changelog-None

@Andezion Andezion left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@daywalker90
daywalker90 enabled auto-merge (rebase) September 18, 2026 10:01
@daywalker90
daywalker90 merged commit 8cfd1fc into ElementsProject:master Sep 18, 2026
86 of 89 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.

CI flake test_blindedpath_noaddr

2 participants