Skip to content

Commit 32808dd

Browse files
committed
fix(ci): bump tests/ timeout 120s → 300s to match release workflow
The CI Go matrix step ran `go test -timeout 120s ./tests/ ./pkg/beacon/`, which has been right at the limit for months — the tests/ integration suite reliably takes 120-140s on the GitHub runners (heavy UDP socket churn). After the recent additions it tipped past the ceiling and both Go (ubuntu-latest) + Go (macos-latest) started failing with "panic: test timed out after 2m0s". Release workflow already uses -timeout 300s. Pre-push hook uses -timeout 300s. Coverage step (same file) had the same 120s issue. Aligned all three on 300s so the surfaces don't disagree. Not a code fix — the test suite hasn't regressed, the limit was too tight. Local runs and the release workflow have passed throughout.
1 parent f772a1a commit 32808dd

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,17 @@ jobs:
3636
go build ./cmd/updater
3737
3838
- name: Test
39-
run: go test -parallel 4 -count=1 -timeout 120s ./tests/ ./pkg/beacon/
39+
# 300s matches the release workflow + pre-push hook. The tests/
40+
# integration suite reliably takes 120-140s on these runners
41+
# (lots of UDP socket churn), which was right at the prior 120s
42+
# ceiling and tipped over after some recent additions. Bumped
43+
# to the same budget as release.yml so the two surfaces agree.
44+
run: go test -parallel 4 -count=1 -timeout 300s ./tests/ ./pkg/beacon/
4045

4146
- name: Coverage
4247
if: matrix.os == 'ubuntu-latest'
4348
run: |
44-
cd tests && go test -parallel 4 -count=1 -coverprofile=coverage.out -covermode=atomic -timeout 120s
49+
cd tests && go test -parallel 4 -count=1 -coverprofile=coverage.out -covermode=atomic -timeout 300s
4550
go tool cover -func=coverage.out | tail -1
4651
4752
website:

0 commit comments

Comments
 (0)