TestRouteWatchLiveFeed streams the live RIPE RIS feed for ~5s and is inherently network-flaky (it also carries a data race on live-feed data), so make check is nondeterministic on main and in CI.
Definition of done
The live-network test no longer runs by default under make check / CI: guard it behind an opt-in (a network build tag, or a testing.Short() skip with -short wired into script/test), so the default run is deterministic and offline.
The live test remains runnable on demand (documented how — e.g. go test -tags network ./... or without -short).
No change to production (non-test) behavior; the guard is test-only.
make check is green and deterministic across repeated runs.
The finishing commit's title must end with (closes #N) referencing this issue.
`TestRouteWatchLiveFeed` streams the live RIPE RIS feed for ~5s and is inherently network-flaky (it also carries a data race on live-feed data), so `make check` is nondeterministic on `main` and in CI.
## Definition of done
1. The live-network test no longer runs by default under `make check` / CI: guard it behind an opt-in (a `network` build tag, or a `testing.Short()` skip with `-short` wired into `script/test`), so the default run is deterministic and offline.
2. The live test remains runnable on demand (documented how — e.g. `go test -tags network ./...` or without `-short`).
3. No change to production (non-test) behavior; the guard is test-only.
4. `make check` is green and deterministic across repeated runs.
The finishing commit's title must end with ` (closes #N)` referencing this issue.
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
TestRouteWatchLiveFeedstreams the live RIPE RIS feed for ~5s and is inherently network-flaky (it also carries a data race on live-feed data), somake checkis nondeterministic onmainand in CI.Definition of done
make check/ CI: guard it behind an opt-in (anetworkbuild tag, or atesting.Short()skip with-shortwired intoscript/test), so the default run is deterministic and offline.go test -tags network ./...or without-short).make checkis green and deterministic across repeated runs.The finishing commit's title must end with
(closes #N)referencing this issue.