Skip to content

ci: run check-arbitrary-configs on github-hosted runners (release-12.1)#8629

Merged
ihalatci merged 1 commit into
release-12.1from
ci-arbitrary-configs-github-hosted
Jun 30, 2026
Merged

ci: run check-arbitrary-configs on github-hosted runners (release-12.1)#8629
ihalatci merged 1 commit into
release-12.1from
ci-arbitrary-configs-github-hosted

Conversation

@ihalatci

Copy link
Copy Markdown
Contributor

Problem

The 1ES self-hosted pool (1es-gha-citusdata-pool) is no longer dispatching jobs, so the required check-arbitrary-configs jobs sit QUEUED indefinitely. This blocks every open release-12.1 PR (#8519, #8328, #8306, #8626, #8627) from merging despite all other required checks passing.

Fix

Flip the test-arbitrary-configs job's runs-on from the dead self-hosted 1ES pool to ubuntu-latest — a one-line backport of the config already shipping on main.

Why it is safe

  • main already runs this exact job/suite/container on ubuntu-latest.
  • Required-check names (PG14 - check-arbitrary-configs-0, ...) derive from the job name:, not runs-on, so branch protection is unaffected.
  • AC was the only job still pinned to the self-hosted pool; everything else already runs github-hosted.

Rollout

Merge this first, then "Update branch" on the 5 open PRs so they inherit the fix and their AC checks run on github-hosted runners.

Unblocks the release-12.1 serial-merge queue.

The 1ES self-hosted pool (1es-gha-citusdata-pool) is no longer dispatching,
leaving the required check-arbitrary-configs jobs stuck QUEUED indefinitely and
blocking every release-12.1 PR. main already runs this job on ubuntu-latest;
this backports that config so the required AC checks execute on GitHub-hosted
runners. Required-check names are unchanged (derived from the job name), so
branch protection is unaffected.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@ihalatci ihalatci added this to the 12.1.13 milestone Jun 30, 2026
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.65%. Comparing base (377c1f2) to head (a1e6b1b).
⚠️ Report is 1 commits behind head on release-12.1.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-12.1    #8629      +/-   ##
================================================
- Coverage         89.65%   89.65%   -0.01%     
================================================
  Files               276      276              
  Lines             59790    59804      +14     
  Branches           7471     7473       +2     
================================================
+ Hits              53605    53616      +11     
- Misses             4041     4042       +1     
- Partials           2144     2146       +2     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ihalatci ihalatci merged commit 0ed95d1 into release-12.1 Jun 30, 2026
211 of 212 checks passed
@ihalatci ihalatci deleted the ci-arbitrary-configs-github-hosted branch June 30, 2026 12:02
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.

2 participants