Skip to content

tests: Fix ip_wins_over_custom flakiness#4047

Merged
rklaehn merged 6 commits intomainfrom
test_ip_wins_over_custom_fail
Mar 24, 2026
Merged

tests: Fix ip_wins_over_custom flakiness#4047
rklaehn merged 6 commits intomainfrom
test_ip_wins_over_custom_fail

Conversation

@rklaehn
Copy link
Copy Markdown
Contributor

@rklaehn rklaehn commented Mar 24, 2026

Description

Increase the bias so we can't accidentally get a wrong selection because IP transport is super slow.

Breaking Changes

None

Notes & open questions

Should I make this a minute?

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.
  • Tests if relevant.
  • All breaking changes documented.
    • List all breaking changes in the above "Breaking Changes" section.
    • Open an issue or PR on any number0 repos that are affected by this breaking change. Give guidance on how the updates should be handled or do the actual updates themselves. The major ones are:

rklaehn added 2 commits March 24, 2026 12:08
The purpose of these tests is to show that the biasing mechanism works.
Maybe 100ms is too low to make this work reliably.
flub added a commit that referenced this pull request Mar 24, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2026

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/4047/docs/iroh/

Last updated: 2026-03-24T12:25:16Z

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 24, 2026

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: bfa7c6d

@n0bot n0bot bot added this to iroh Mar 24, 2026
@github-project-automation github-project-automation bot moved this to 🚑 Needs Triage in iroh Mar 24, 2026
rklaehn added 4 commits March 24, 2026 13:07
We really don't care. This is just about the selection. Now it would only
fail if no ip transport is available at all.
@rklaehn rklaehn enabled auto-merge March 24, 2026 12:23
@rklaehn rklaehn added this pull request to the merge queue Mar 24, 2026
Merged via the queue into main with commit 174b58b Mar 24, 2026
30 checks passed
@github-project-automation github-project-automation bot moved this from 🚑 Needs Triage to ✅ Done in iroh Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants