From d4ae543d7cd9259e38db015b565ef9beb5967e2c Mon Sep 17 00:00:00 2001 From: Max Ghenis Date: Sun, 18 Oct 2026 14:51:22 -0400 Subject: [PATCH 1/2] Give the UK qualify job 330 minutes Since #651 the wrapper.household contract rows run the household calculator instead of erroring at once, and with contract 2.0.0 the local gates pass two hours. The 120-minute limit cancelled every attempt and the catch-up re-staged it, so no candidate has finished its gates since 17 October. Fixes #659 Co-Authored-By: Claude Opus 5.5 --- .github/workflows/certify-uk-release.yaml | 7 ++++++- changelog.d/659.fixed.md | 1 + 2 files changed, 7 insertions(+), 1 deletion(-) create mode 100644 changelog.d/659.fixed.md diff --git a/.github/workflows/certify-uk-release.yaml b/.github/workflows/certify-uk-release.yaml index 23f1c04e4..892161740 100644 --- a/.github/workflows/certify-uk-release.yaml +++ b/.github/workflows/certify-uk-release.yaml @@ -294,7 +294,12 @@ jobs: needs: [resolve, prepare] if: needs.prepare.outputs.duplicate == 'false' runs-on: ubuntu-latest - timeout-minutes: 120 + # The local gates took about 80 minutes while every wrapper.household + # contract row errored at once. Since #651 those rows run the household + # calculator, and with contract 2.0.0's extra years the gates pass two + # hours, so a 120-minute limit cancelled every attempt and re-staged it. + # 330 stays under the 360-minute ceiling for GitHub-hosted jobs. + timeout-minutes: 330 concurrency: group: uk-candidate-${{ needs.resolve.outputs.pair_id }} cancel-in-progress: false diff --git a/changelog.d/659.fixed.md b/changelog.d/659.fixed.md new file mode 100644 index 000000000..4c8627d36 --- /dev/null +++ b/changelog.d/659.fixed.md @@ -0,0 +1 @@ +The UK qualification job may now run for up to 330 minutes, so candidates whose gates take over two hours, now that the household contract route runs, finish instead of being cancelled and re-staged. From 6588f78b02cd6e3ca5c4111c689c072b135614ef Mon Sep 17 00:00:00 2001 From: Max Ghenis Date: Sun, 18 Oct 2026 15:04:17 -0400 Subject: [PATCH 2/2] Correct the cause in the qualify timeout comment and note the staging lease #627 is not part of the cause: attempts with #651 but not #627 also timed out. The four-hour staging lease is shorter than the new limit; #662 tracks raising it. Co-Authored-By: Claude Opus 5.5 --- .github/workflows/certify-uk-release.yaml | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/.github/workflows/certify-uk-release.yaml b/.github/workflows/certify-uk-release.yaml index 892161740..a1dd71962 100644 --- a/.github/workflows/certify-uk-release.yaml +++ b/.github/workflows/certify-uk-release.yaml @@ -294,11 +294,13 @@ jobs: needs: [resolve, prepare] if: needs.prepare.outputs.duplicate == 'false' runs-on: ubuntu-latest - # The local gates took about 80 minutes while every wrapper.household + # The local gates took 80-89 minutes while every wrapper.household # contract row errored at once. Since #651 those rows run the household - # calculator, and with contract 2.0.0's extra years the gates pass two - # hours, so a 120-minute limit cancelled every attempt and re-staged it. - # 330 stays under the 360-minute ceiling for GitHub-hosted jobs. + # calculator, which takes the gates past two hours, so a 120-minute limit + # cancelled every attempt and re-staged it. 330 stays under the 360-minute + # ceiling for GitHub-hosted jobs. Note that stage_attempt's four-hour lease + # still lets a catch-up re-stage a live attempt whose gates pass about four + # hours (#662), so in practice runs should stay under that. timeout-minutes: 330 concurrency: group: uk-candidate-${{ needs.resolve.outputs.pair_id }}