Skip to content

Commit cd6c883

Browse files
committed
ci(release): give quality checks thirty minutes on hosted runners
The release quality job runs the whole repository's check, typecheck, and test suites serially on one runner, the same work ci.yml spreads over four jobs. Blacksmith's 8 vCPU finished the Test step in 3m16s, so a 10-minute ceiling was fine. On the 4-vCPU GitHub-hosted runner that replaced it (#99) the Test step was still running at 7m44s when the ceiling cut the job, which GitHub reports as cancelled, and every publish job downstream skipped: the first nightly after the migration produced four successful platform builds and no release. Thirty minutes is roughly 2x the expected duration.
1 parent 37208c9 commit cd6c883

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/release.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,13 @@ jobs:
189189
needs: [preflight]
190190
if: ${{ !failure() && !cancelled() && needs.preflight.result == 'success' }}
191191
runs-on: ubuntu-24.04
192-
timeout-minutes: 10
192+
# `vp run test` runs the whole repository's suites serially here — the
193+
# same work ci.yml spreads over four jobs. On a 4-vCPU GitHub-hosted
194+
# runner the job lands around fifteen minutes (the Test step alone was
195+
# still running at 7m44s when a 10-minute ceiling cut it on 2026-08-26,
196+
# cancelling the first nightly after the runner migration), so give it
197+
# 2x headroom. Blacksmith's 8 vCPU did the Test step in 3m16s.
198+
timeout-minutes: 30
193199
steps:
194200
- name: Checkout
195201
uses: actions/checkout@v6

0 commit comments

Comments
 (0)