From cd6c88312be57691ab0dc3cc421f313da3fd56cd Mon Sep 17 00:00:00 2001 From: Trevor Walker Date: Wed, 26 Aug 2026 13:40:08 -0600 Subject: [PATCH] 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. --- .github/workflows/release.yml | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 6dc047f31..234cc7de2 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -189,7 +189,13 @@ jobs: needs: [preflight] if: ${{ !failure() && !cancelled() && needs.preflight.result == 'success' }} runs-on: ubuntu-24.04 - timeout-minutes: 10 + # `vp run test` runs the whole repository's suites serially here — the + # same work ci.yml spreads over four jobs. On a 4-vCPU GitHub-hosted + # runner the job lands around fifteen minutes (the Test step alone was + # still running at 7m44s when a 10-minute ceiling cut it on 2026-08-26, + # cancelling the first nightly after the runner migration), so give it + # 2x headroom. Blacksmith's 8 vCPU did the Test step in 3m16s. + timeout-minutes: 30 steps: - name: Checkout uses: actions/checkout@v6