Skip to content

Raise default CI batch timeout from 3m to 15m#750

Merged
wesm merged 1 commit into
mainfrom
feat/ci-default-batch-timeout
May 27, 2026
Merged

Raise default CI batch timeout from 3m to 15m#750
wesm merged 1 commit into
mainfrom
feat/ci-default-batch-timeout

Conversation

@wesm
Copy link
Copy Markdown
Member

@wesm wesm commented May 27, 2026

Summary

  • Bumps the default [ci].batch_timeout from 3 minutes to 15 minutes in CIConfig.ResolvedBatchTimeout.
  • The 3m default expired before slower agents (codex, gemini) could finish, causing PR comments with partial counters (e.g. 1+0/3) and discarded output from still-running child processes.
  • Updates the doc comment on the BatchTimeout field and the unit tests that assert the default.

The 3-minute default routinely expired before slow agents (codex,
gemini) could finish, leaving PR comments with partial review
counters. 15m gives realistic reviews room to complete while still
bounding worst-case latency.
@roborev-ci
Copy link
Copy Markdown

roborev-ci Bot commented May 27, 2026

roborev: Combined Review (be2521d)

All reviewed agents found the CI batch timeout change clean, with no Medium, High, or Critical issues to report.


Synthesized from 3 reviews (agents: codex, gemini | types: default, security)

@wesm wesm merged commit 7c77bbe into main May 27, 2026
8 checks passed
@wesm wesm deleted the feat/ci-default-batch-timeout branch May 27, 2026 12:03
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