Skip to content

Commit d77025e

Browse files
TeoSlayerteovlclaude
authored
nightly: serialize integration jobs to end runner starvation (#382)
nightly: serialize integration jobs (-parallel 1) to end runner starvation The integration suites spin up multiple daemons + registry + WSS per test; running 4 concurrently on a 4-vCPU public runner starved them into timeouts — the dominant cause of the chronic nightly redness (verified: representative failing tests pass locally when run -parallel 1). Serialize both jobs so each integration test gets the whole runner; bump go-test and job timeouts to fit the longer wall-clock. Co-authored-by: Teodor Calin <teodor@vulturelabs.io> Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
1 parent 3b392a8 commit d77025e

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

.github/workflows/nightly.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
integration:
2525
name: Integration suite (default tags)
2626
runs-on: ubuntu-latest
27-
timeout-minutes: 45
27+
timeout-minutes: 55
2828
steps:
2929
- uses: actions/checkout@v7
3030

@@ -37,7 +37,7 @@ jobs:
3737
# The full ./tests/ default-tag suite (~300 tests). Moved out of
3838
# PR CI in PR #121 — sustained 15m timeouts on public runners.
3939
run: |
40-
go test -parallel 4 -count=1 -timeout 30m \
40+
go test -parallel 1 -count=1 -timeout 45m \
4141
-skip 'TestMultipleLargeWrites|TestConcurrentDialEncryptDecrypt|TestHTTPPrivateWithTrust|TestInviteInboxCapEnforced|TestConnectionLimits|TestRC6PeerRestartRecoveryEndToEnd|TestEndToEndRelay|TestNameserverOverwriteA|TestNameserverPersistence|TestNameserverRegisterN|TestNameserver$' \
4242
./tests/
4343
@@ -64,7 +64,7 @@ jobs:
6464
# and run sufficiently via the architecture-gates stress
6565
# harness instead.
6666
run: |
67-
go test -tags nightly -parallel 4 -count=1 -timeout 30m \
67+
go test -tags nightly -parallel 1 -count=1 -timeout 50m \
6868
-skip 'TestMultipleLargeWrites|TestConcurrentDialEncryptDecrypt|TestHTTPPrivateWithTrust|TestInviteInboxCapEnforced|TestConnectionLimits|TestRC6PeerRestartRecoveryEndToEnd|TestEndToEndRelay|TestNameserverOverwriteA|TestNameserverPersistence|TestNameserverRegisterN|TestNameserver$' \
6969
./tests/
7070

0 commit comments

Comments
 (0)