diff options
| author | Mike O'Driscoll <mikeo@tailscale.com> | 2026-02-25 13:52:01 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2026-02-25 13:52:01 -0500 |
| commit | 329d2e2643b804c3666d93b3f7195c22b6ae2523 (patch) | |
| tree | 0e337b92b5277599849ac3b9af096867943cf5b9 /control/controlhttp/controlhttpserver/controlhttpserver.go | |
| parent | eb819c580eb7a2b47047fd5f0a63bf29dbb423fe (diff) | |
| download | tailscale-329d2e2643b804c3666d93b3f7195c22b6ae2523.tar.xz tailscale-329d2e2643b804c3666d93b3f7195c22b6ae2523.zip | |
prober: fix race condition in TestExcludeInRunAll (#18807)
The test was making HTTP requests before waiting for probes to
complete their initial run in "once" mode. This created a race where
sometimes the probe's previous state was empty (0 results) and
sometimes it had one result, causing inconsistent RecentResults and
PreviousSuccessRatio values.
Fixed by waiting for all probes to complete via their stopped channels
before making HTTP requests, matching the pattern used in other tests
like TestProberRunHandler and TestRunAllHandler.
Fixes #18806
Signed-off-by: Mike O'Driscoll <mikeo@tailscale.com>
Diffstat (limited to 'control/controlhttp/controlhttpserver/controlhttpserver.go')
0 files changed, 0 insertions, 0 deletions