Skip to content

Commit

Permalink
Don't fail for high RTTs
Browse files Browse the repository at this point in the history
  • Loading branch information
thomaseizinger committed Sep 25, 2023
1 parent 5824dfb commit 6ea9d7d
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions hole-punch-interop/testplans.ts
Original file line number Diff line number Diff line change
Expand Up @@ -112,12 +112,8 @@ import path from "path";
const rttDifference = Math.abs(report.rtt_to_holepunched_peer_millis - rttDirectConnection);

if (rttDifference > 5) {
failures.push(testSpec.name)
statuses.push([testSpec.name, "failure (RTT too high)"])

console.log(`Expected RTT of direction connection to be ~${rttDirectConnection}ms but was ${report.rtt_to_holepunched_peer_millis}ms`)

continue;
// Emit a warning but don't do anything for now.
console.warn(`Expected RTT of direction connection to be ~${rttDirectConnection}ms but was ${report.rtt_to_holepunched_peer_millis}ms`)
}

statuses.push([testSpec.name, "success"])
Expand Down

0 comments on commit 6ea9d7d

Please sign in to comment.