Skip to content

Conversation

joostjager
Copy link
Contributor

Primarily to debug the current test flakes in CI, but also for future debugging generally.

@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Sep 19, 2025

👋 Hi! This PR is now in draft status.
I'll wait to assign reviewers until you mark it as ready for review.
Just convert it out of draft status when you're ready for review!

@joostjager joostjager force-pushed the test-logging branch 2 times, most recently from 9eb545e to 042816a Compare September 19, 2025 08:44
@joostjager joostjager removed the request for review from tnull September 19, 2025 08:45
@joostjager joostjager force-pushed the test-logging branch 4 times, most recently from bb56249 to 63d53f1 Compare September 19, 2025 14:17
@joostjager joostjager marked this pull request as draft September 19, 2025 14:18
@joostjager
Copy link
Contributor Author

Potential problems so far:

  • With multiple listeners, only the first successful listener is bound
  • Random ports range is maybe too large and collisions with outgoing connections may happen?

@joostjager joostjager force-pushed the test-logging branch 3 times, most recently from 9456b73 to 386e6cb Compare September 23, 2025 08:00
To avoid conflicts ("port in use") with ports that were used for
outgoing connections and are now in the TIME_WAIT state.
@tnull tnull changed the base branch from develop to main October 7, 2025 08:52
@tnull
Copy link
Collaborator

tnull commented Oct 7, 2025

We decided to make the develop behavior (tracking more-recent LDK main) the default behavior. With #655 we merged develop into main and now deleted the develop branch. Hence the base branch is changed to main here, too.

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.

3 participants