Skip to content

Conversation

@JoeTurki
Copy link
Member

@JoeTurki JoeTurki commented Oct 25, 2025

Description

a test to makes sure our SCTP loss-detection copes with a sudden RTT drop (high -> low) that causes packet reordering, without spuriously declaring losses.
It asserts that

  1. No missing data: the client must receive all 40 echoes with correct payloads.
  2. Currently disabled until rack is merged ( Add RACK to SCTP #390 ) no excessive fast retransmits both client and server report fast retransmit count <= 2, i.e., no widespread spurious loss due to the rtt shift.

Reference issue

@philipch07 asked me to help with this test. in discord dms :)

@codecov
Copy link

codecov bot commented Oct 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.85%. Comparing base (c875185) to head (e6e1653).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #408      +/-   ##
==========================================
+ Coverage   81.77%   81.85%   +0.07%     
==========================================
  Files          51       51              
  Lines        4522     3439    -1083     
==========================================
- Hits         3698     2815     -883     
+ Misses        680      482     -198     
+ Partials      144      142       -2     
Flag Coverage Δ
go 81.85% <ø> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JoeTurki JoeTurki force-pushed the joe/rtt-switch-test branch from a6295b5 to 2b3180a Compare October 25, 2025 09:32
@JoeTurki JoeTurki force-pushed the joe/rtt-switch-test branch from 2b3180a to e6e1653 Compare October 25, 2025 09:33
@JoeTurki JoeTurki merged commit 012d2b4 into master Oct 25, 2025
17 checks passed
@JoeTurki JoeTurki deleted the joe/rtt-switch-test branch October 25, 2025 09:36
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.

2 participants