Skip to content

CI: raise the job timeout to 120 minutes — the suite outgrew 60 - #572

Merged
lmoresi merged 1 commit into
developmentfrom
ci/raise-job-timeout
Aug 15, 2026
Merged

CI: raise the job timeout to 120 minutes — the suite outgrew 60#572
lmoresi merged 1 commit into
developmentfrom
ci/raise-job-timeout

Conversation

@lmoresi

@lmoresi lmoresi commented Aug 15, 2026

Copy link
Copy Markdown
Member

Two PRs were cancelled at exactly 1h00m and reported as failures when nothing had failed. Measured from the logs on 2026-08-15:

So the cap is the binding constraint, not the content: a run with no additions was already within five minutes of it, and three individual serial batches take 14–15 minutes each.

The failure mode is worse than slow CI, because a cancellation looks like a red X. #571's first timeout sent us hunting a test failure that did not exist, and the same would happen to anyone else adding tests.

This raises the cap to 120 so work can land. It is a stop-gap, and the comment in the workflow says so: the durable fix is splitting the batches across parallel jobs so wall-clock stops tracking total test time — filed separately.

Underworld development team with AI support from Claude Code

Two PRs were cancelled at 1h00m and reported as failures when nothing had
failed: the serial phase alone now takes ~55 minutes, so the parallel
phase runs last and gets cut mid-run. A run with no additions at all
(PR #568) took 55m07s, which means any new tests exceeded the cap.

This is a stop-gap so work can land. The durable fix is splitting the
batches across parallel jobs so wall-clock stops tracking total test time
- filed separately.

Underworld development team with AI support from Claude Code
Copilot AI lite review requested due to automatic review settings August 15, 2026 07:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

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