Skip to content

ci: pin ubuntu-latest runners to ubuntu-24.04 - #3456

Merged
CTTY merged 1 commit into
apache:mainfrom
kevinjqliu:kevinjqliu-ci-runner-labels
Oct 15, 2026
Merged

CTTY merged 1 commit into
apache:mainfrom
kevinjqliu:kevinjqliu-ci-runner-labels

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

What changes are included in this PR?

ubuntu-latest jobs across the apache org are queuing for 50–80 minutes, while ubuntu-24.04 jobs on the same image start within a couple of minutes. This PR's own CI finished in about 11 minutes.

There's a separate optimization to move jobs to use ubuntu-slim as much as possible, in #3457.

Are these changes tested?

Covered by this PR's CI. Release, publish, website, and nightly workflows don't run on PRs. actionlint and zizmor report no new findings.

AI Disclosure

Drafted with GitHub Copilot. Reviewers should check that nothing depends on the ubuntu-latest string. Linux matrix job names and wheel artifact names change; none are required checks.

ubuntu-latest jobs across the apache org are queuing for 50-80 minutes,
while ubuntu-24.04 jobs on the same image start within minutes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings October 14, 2026 18:26
@kevinjqliu

Copy link
Copy Markdown
Contributor Author

This is kind of a wack-a-mole BUT it'll help us in the short terms 🤣 hopefully no one else migrate to use 24.04

@kevinjqliu
kevinjqliu requested review from CTTY and blackmwk October 14, 2026 18:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Copilot review overview

0 open findings

What changed in this PR

Pins GitHub Actions Linux runners from ubuntu-latest to ubuntu-24.04 to reduce queue times and avoid the upcoming ubuntu-latest move to Ubuntu 26.04, while keeping the effective environment on 24.04.

Changes:

  • Replaced runs-on: ubuntu-latest with runs-on: ubuntu-24.04 across CI/security/publish/release workflows.
  • Updated OS values in job matrices / includes from ubuntu-latest to ubuntu-24.04.
  • Left macOS/Windows -latest runner labels unchanged.
File Description
.github/​workflows/​zizmor.yml Pins zizmor workflow runner to ubuntu-24.04 to avoid ubuntu-latest queue contention.
.github/​workflows/​website.yml Pins website build/deploy runner to ubuntu-24.04.
.github/​workflows/​release_python_nightly.yml Pins nightly packaging/publish jobs and Linux wheel matrix entries to ubuntu-24.04.
.github/​workflows/​release_python.yml Pins release validation/build/publish jobs and Linux wheel matrix entries to ubuntu-24.04.
.github/​workflows/​publish.yml Pins publish workflow to ubuntu-24.04.
.github/​workflows/​public-api.yml Pins public API check workflow to ubuntu-24.04.
.github/​workflows/​ci.yml Pins main CI jobs and OS matrices to ubuntu-24.04.
.github/​workflows/​bindings_python_ci.yml Pins bindings Python CI matrix Linux runner to ubuntu-24.04.
.github/​workflows/​audit.yml Pins security audit workflow to ubuntu-24.04.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@kevinjqliu

Copy link
Copy Markdown
Contributor Author

needs fix security audit fix from #3455

@kevinjqliu

Copy link
Copy Markdown
Contributor Author

but good thing is, CI is much much faster 😄

All of #3456's CI finished in under 11 minutes. #3455 and #3457 still use ubuntu-latest, and their CI has been waiting 21–44 minutes without a single Linux job starting.

@CTTY CTTY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me, I'd be happy to trade for a faster CI by using an slightly older version of ubuntu

@CTTY
CTTY added this pull request to the merge queue Oct 14, 2026
Merged via the queue into apache:main with commit bdee10e Oct 15, 2026
28 of 29 checks passed
@kevinjqliu
kevinjqliu deleted the kevinjqliu-ci-runner-labels branch October 15, 2026 02:54
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.

3 participants