Skip to content

Conversation

@Arqu
Copy link
Collaborator

@Arqu Arqu commented Oct 1, 2025

Description

Split off from the original PR as this is dubious whether it's useful but interesting to test a bit.

Breaking Changes

Notes & open questions

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:

@Arqu Arqu self-assigned this Oct 1, 2025
@github-actions
Copy link

github-actions bot commented Oct 1, 2025

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

Last updated: 2025-10-08T06:39:56Z

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

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: 7d926d7

@n0bot n0bot bot added this to iroh Oct 1, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Oct 1, 2025
@dignifiedquire
Copy link
Contributor

the jitter part makes sense to me in general, @flub I am wondering is there sth in multipath/quic to deal with this issue, seems like problem that could happen there as well

@flub
Copy link
Contributor

flub commented Oct 2, 2025

the jitter part makes sense to me in general, @flub I am wondering is there sth in multipath/quic to deal with this issue, seems like problem that could happen there as well

So the idea is to always add some jitter when scheduling a holepunching attempt in the future?

@Arqu Arqu force-pushed the arqu/congestion_metrics branch from a89e698 to bf1548c Compare October 7, 2025 16:43
Base automatically changed from arqu/congestion_metrics to main October 7, 2025 20:13
@Arqu Arqu force-pushed the arqu/congestion_metrics_trickery branch from e7ba93a to 62c51e3 Compare October 8, 2025 06:32
@Arqu Arqu force-pushed the arqu/congestion_metrics_trickery branch from 62c51e3 to 65dbbad Compare October 8, 2025 06:37
@Arqu
Copy link
Collaborator Author

Arqu commented Oct 23, 2025

Closing this PR but keeping the branch as future reference. Unsure if we should file the jitter stuff as an issue to do at a later point?

@Arqu Arqu closed this Oct 23, 2025
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Oct 23, 2025
@flub
Copy link
Contributor

flub commented Oct 23, 2025

Closing this PR but keeping the branch as future reference. Unsure if we should file the jitter stuff as an issue to do at a later point?

Can you please file the jitter thing as an issue? Because I already forgot the details. Maybe even as a sub-issue of the multipath issue (#3276 ), though definitely on one of the pre-1.0 milestones.

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.

3 participants