ci: extend PR e2e job cap to 75 minutes - #1236
Conversation
Co-Authored-By: Paperclip <noreply@paperclip.ing>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
@ally please review exact head 20d3e9e only. This independent kkroo CI-maintenance PR raises only the dedicated PR e2e job cap from 60 to 75 minutes after #1224 was cancelled at the prior cap while its browser suite was still active. It does not change Playwright retries, assertions, specs, runners, or pnpm-lock.yaml. The allyblockcast User review has also been requested. Auto-merge remains off. |
There was a problem hiding this comment.
Ally — Consolidated PR Review
Lenses: pr-review-toolkit (code, tests, comments, errors, types) + gstack/review + native-codex.
Reviewed head: 20d3e9e
Critical Issues (0)
Important Issues (0)
Suggestions (0)
Strengths
- The timeout increase is scoped to the serial e2e job, which runs on its dedicated
arc-e2epool. - The existing rationale and job isolation make the operational change clear and low risk.
Recommended Action
- Merge when the applicable CI checks complete.
allyblockcast
left a comment
There was a problem hiding this comment.
Ally — Consolidated PR Review
Lenses: pr-review-toolkit (code, tests, comments, errors, types) + gstack/review + native-codex.
Reviewed head: 20d3e9e
Critical Issues (0)
Important Issues (0)
Suggestions (0)
Strengths
- The timeout increase is scoped to the serial e2e job, which runs on its dedicated
arc-e2epool. - The existing rationale and job isolation make the operational change clear and low risk.
Recommended Action
- Merge when the applicable CI checks complete.
Thinking Path
Linked Issues or Issue Description
93255878198ran from2026-08-09T15:03:39Zto2026-08-09T16:04:22Z, and itsRun e2e testsstep was cancelled at16:03:53Z. Affected deployment: GitHub Actionspull_request/merge-group workflow onarc-e2e.What Changed
.github/workflows/pr.yml: increased the PRe2ejob cap from 60 to 75 minutes.pnpm-lock.yaml.Verification
node --test scripts/check-github-runner-labels.test.js— 10 passing tests.e2ejob in.github/workflows/pr.ymlcontainstimeout-minutes: 75.git diff --checkpassed..github/workflows/pr.ymland does not modifypnpm-lock.yaml.Risks
Model Used
Checklist
Fixes: #/Closes #/Refs #OR (b) described the issue in-PR following the relevant issue template