infra(ci): raise test step timeout 15m → 20m (mc#1099)
Lint pre-flip continue-on-error / Verify continue-on-error flips have run-log proof (pull_request) Waiting to run
lint-required-no-paths / lint-required-no-paths (pull_request) Waiting to run
Lint workflow YAML (Gitea-1.22.6-hostile shapes) / Lint workflow YAML for Gitea-1.22.6-hostile shapes (pull_request) Waiting to run
Runtime PR-Built Compatibility / detect-changes (pull_request) Waiting to run
Runtime PR-Built Compatibility / PR-built wheel + import smoke (pull_request) Blocked by required conditions
Secret scan / Scan diff for credential-shaped strings (pull_request) Waiting to run
Block internal-flavored paths / Block forbidden paths (pull_request) Successful in 26s
gate-check-v3 / gate-check (pull_request) Waiting to run
qa-review / approved (pull_request) Waiting to run
CI / Shellcheck (E2E scripts) (pull_request) Successful in 30s
Handlers Postgres Integration / detect-changes (pull_request) Successful in 24s
Lint curl status-code capture / Scan workflows for curl status-capture pollution (pull_request) Successful in 20s
CI / Detect changes (pull_request) Successful in 1m34s
E2E API Smoke Test / detect-changes (pull_request) Successful in 1m31s
E2E Staging Canvas (Playwright) / detect-changes (pull_request) Successful in 1m50s
sop-tier-check / tier-check (pull_request) Successful in 27s
sop-checklist / all-items-acked (pull_request) Successful in 40s
security-review / approved (pull_request) Failing after 48s
Handlers Postgres Integration / Handlers Postgres Integration (pull_request) Successful in 11s
E2E API Smoke Test / E2E API Smoke Test (pull_request) Successful in 13s
E2E Staging Canvas (Playwright) / Canvas tabs E2E (pull_request) Successful in 12s
lint-mask-pr-atomicity / lint-mask-pr-atomicity (pull_request) Successful in 3m16s
lint-continue-on-error-tracking / lint-continue-on-error-tracking (pull_request) Successful in 3m53s
lint-required-context-exists-in-bp / lint-required-context-exists-in-bp (pull_request) Successful in 2m58s
CI / Python Lint & Test (pull_request) Successful in 7m34s
CI / Platform (Go) (pull_request) Failing after 15m24s
CI / all-required (pull_request) Failing after 15m31s
CI / Canvas (Next.js) (pull_request) Successful in 16m2s
CI / Canvas Deploy Reminder (pull_request) Successful in 2s

Cold runner observed: golangci-lint ~10m + test suite ~8-12m = up to 22m.
15m step timeout still too tight (PR #1151 Platform (Go) failed at 18m).
20m gives sufficient headroom while staying within the 50m job ceiling.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
2026-05-15 08:49:45 +00:00
parent 820c7828e9
commit 7c08352dca
+5 -4
View File
@@ -198,12 +198,13 @@ jobs:
- if: always()
name: Run tests with race detection and coverage
# Explicit timeout: cold runner cache causes OOM kills at ~4m39s on the
# full ./... suite with race detection + coverage. A 15m per-step timeout
# full ./... suite with race detection + coverage. A 20m per-step timeout
# lets the suite complete on cold cache (~5-12m) while failing cleanly
# instead of OOM-killing. The job-level timeout (50m) is a backstop.
# mc#1099: raised from 10m to 15m — cold runner test suite consistently
# hits 10m ceiling after golangci-lint ~20m on very-cold runner (PR #1151).
run: go test -race -timeout 15m -coverprofile=coverage.out ./...
# mc#1099: raised from 10m 15m → 20m. Cold runner: golangci-lint ~10m
# + test suite ~8-12m = up to 22m total. 15m was still too tight (PR #1151
# Platform (Go) failed at 18m on a very-cold runner). 20m gives headroom.
run: go test -race -timeout 20m -coverprofile=coverage.out ./...
- if: always()
name: Per-file coverage report