stacklok / toolhive / 30347108009

Build # 30347108009

Build Type

push

github

Committed by web-flow

Commit Message

Retry dual-era backend startup on a lost port-bind race (#6080)

allocateVMCPPort probes for a free port and closes its own listener
immediately, but the actual bind happens later, asynchronously, inside
the container Docker starts for the detached `thv run` process. Under
back-to-back workload churn another container can grab the same
ephemeral port first, failing with "address already in use" and
leaving the spec's BeforeEach to time out after 2 minutes.

Retry once with a freshly allocated port when this specific failure is
observed. Confirmed live under a 20-repeat stress run: the race still
fires (~1 in 12-13 attempts), but the retry now absorbs it instead of
failing the spec.

Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>

Coverage Stats

81246 of 116527 relevant lines covered (69.72%)

98.2 hits per line

Source File

Press 'n' to go to next uncovered line, 'b' for previous

Source Not Available