Skip to content

lansport: replace the rounds test helper with sleepAndWait - #4

Closed
bradfitz wants to merge 2 commits into
mainfrom
bradfitz/rendezvous-settle
Closed

bradfitz wants to merge 2 commits into
mainfrom
bradfitz/rendezvous-settle

Conversation

@bradfitz

@bradfitz bradfitz commented Oct 1, 2026

Copy link
Copy Markdown
Member

A loop over probe rounds hid what each wait was for. Each call site now
sleeps for an explicit duration and says why that much.

Updates #cleanup

Without hysteresis, a peer that flaps or a pool whose members become
reachable one at a time on startup moves keys around on every change.

RouterConfig adds two delays. JoinDelay holds newly reachable peers out
of the table until that long after the latest one appeared, so members
discovered seconds apart are adopted together and ownership changes
once. LeaveDelay keeps an unreachable member in the table that long
before reassigning its keys; meanwhile Pick reports those keys as the
local process's to handle, so a restart or short outage neither moves
keys away nor back. Zero delays adopt and reassign immediately, as
before.

Updates tailscale/corp#48514

Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
Change-Id: I2d7a9e4b1c6f8035e9a2b7d4c1f0e3a6b8d5c290
A loop over probe rounds hid what each wait was for. Each call site now
sleeps for an explicit duration and says why that much.

Updates #cleanup

Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
Change-Id: I5c1e8f2a9d3b7046c0e4a8f1b2d6c9e3a7f0b418
@bradfitz
bradfitz requested a review from a team October 1, 2026 03:17
@bradfitz bradfitz closed this Oct 1, 2026
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.

1 participant