Skip to content

ci: bump Go test timeout 15m -> 20m (server + ingestor)#1750

Merged
Kpa-clawbot merged 1 commit into
Kpa-clawbot:masterfrom
efiten:fix/ci-go-test-timeout-20m
Jun 19, 2026
Merged

ci: bump Go test timeout 15m -> 20m (server + ingestor)#1750
Kpa-clawbot merged 1 commit into
Kpa-clawbot:masterfrom
efiten:fix/ci-go-test-timeout-20m

Conversation

@efiten

@efiten efiten commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

Problem

The server Go suite runs ~13–15m against a 15m go test -timeout, so on slower CI runners it intermittently hits panic: test timed out after 15m0s (cmd/server, e.g. db_test.go) — false-red CI that a plain rerun clears. Observed on PR #1728 (15m25s on the passing attempt — right at the ceiling).

Fix

Bump both go test invocations (cmd/server and cmd/ingestor) from -timeout 15m to -timeout 20m for headroom. No test or application code changes — CI workflow only.

Verification

Workflow-only change; this PR's own CI is the confirmation.

🤖 Generated with Claude Code

ci: bump Go test timeout 15m -> 20m (server + ingestor)

The server Go suite runs ~13-15m against a 15m ceiling and has
intermittently hit `panic: test timed out after 15m0s` (cmd/server,
e.g. db_test.go) on slower runners, producing false-red CI that a
plain rerun clears. Bump both `go test` invocations to 20m to give
headroom; no test or application changes.

Co-Authored-By: Erwin Fiten <erwin.fiten@gmail.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@
@Kpa-clawbot

Copy link
Copy Markdown
Owner

🤖 Hourly PR-watch review

Verdict: APPROVE — trivial CI timeout bump

Diff is two characters: -timeout 15m-timeout 20m on the server + ingestor go test invocations in .github/workflows/deploy.yml. CI is green on this branch with the new timeout, which is the only assertion that matters for a workflow-config change.

TDD exemption

Pure config change. AGENTS.md TDD-exempt rule applies: "existing tests MUST stay green AND unaltered" + "no test files modified" + "CI green without test edits." All three hold (CI green ✅, no test files in diff ✅).

Findings: 0 BLOCKER / 0 MAJOR / 1 NIT

  • NIT: PR body would benefit from a one-liner about why 15m was insufficient (slowest job duration or specific test that drifted near the cap), so the next operator knows whether 20m is enough headroom or just kicking the can.

3-axis merge gate

  • ✅ mergeable: MERGEABLE
  • ✅ mergeStateStatus: CLEAN
  • ✅ All CI: SUCCESS

Auto-merging.

Reviewed by: orchestrator self-review (CI config, no code surface).

@Kpa-clawbot Kpa-clawbot merged commit f780fe7 into Kpa-clawbot:master Jun 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants