Skip to content

ci: move the remaining CI workflows to the dedicated runner pool - #380

Closed
robinnsc wants to merge 1 commit into
mainfrom
ci/dedicated-pool-remaining-workflows
Closed

robinnsc wants to merge 1 commit into
mainfrom
ci/dedicated-pool-remaining-workflows

Conversation

@robinnsc

@robinnsc robinnsc commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator

What

The same three changes #379 made to integration.yml, applied to the other five CI workflows (integration-mongodb, test, clippy, fmt, licenses):

  1. runs-on: extenddb_ubuntu-2404_4-core on every job (9 jobs across the five files; zero ubuntu-latest remaining in CI workflows).
  2. Per-job timeout-minutes: 30 for the MongoDB suite (longest observed job 18 min across the last three main runs), 15 for the sub-5-minute workflows (test 3 min, clippy/fmt ~1 min); licenses keeps its existing 20.
  3. The concurrency block in its corrected form — per-PR supersede on pull_request; everything else keyed by run_id. This is the fix from ci: move all CI workflows to the dedicated ExtendDB runner pool #379's audit: merge-group runs reuse a ref across re-queues and GitHub replaces a pending run in a group regardless of cancel-in-progress, so ref-keyed groups could cancel merge-group runs.

Independent of #379 (disjoint files); merge in either order.

Why

After this, every CI workflow in the repo runs on the dedicated pool, so nothing in the merge-group round competes with other orgs for runners — the full round is five fast workflows plus the two integration suites, all starting within seconds. Combined with #379 this is what makes the merge queue's 30-minute check timeout comfortably sufficient again.

Testing done

  • yaml.safe_load on all five; 9 runs-on swapped, timeouts present on every job, one concurrency block per file.
  • This PR's own CI is the live test — all five workflows trigger on pull_request, so every job below runs on the new pool. Confirm they start within a minute before approving.

Checklist

  • Tests / fmt / clippy — not applicable, CI only
  • Documentation — inline comments on the pool and the concurrency rationale
  • Breaking changes noted below

ADR / RFC: n/a — CI infrastructure only.

Breaking changes

None.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache License 2.0 and I agree to the Developer Certificate of Origin (DCO). See CONTRIBUTING.md for details.

Same three changes #379 made to integration.yml, applied to
integration-mongodb, test, clippy, fmt, and licenses: runs-on
extenddb_ubuntu-2404_4-core (pinned Ubuntu 24.04), a per-job
timeout (30 min for the MongoDB suite whose longest job is 18 min;
15 min for the sub-5-minute workflows; licenses keeps its existing
20), and the corrected concurrency block — per-PR supersede on
pull_request, run_id-unique groups for everything else so merge-group
runs can never cancel one another.
@robinnsc

robinnsc commented Oct 1, 2026

Copy link
Copy Markdown
Collaborator Author

Folded into #379 so the whole runner-pool migration is one review. The live runs on this branch already proved the pool for these five workflows (every job picked up in 24–26s).

@robinnsc robinnsc closed this Oct 1, 2026
@robinnsc
robinnsc deleted the ci/dedicated-pool-remaining-workflows branch October 1, 2026 23:14
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.

1 participant