Skip to content

[Infra] Update ubuntu GitHub runners#7337

Open
martincostello wants to merge 2 commits into
open-telemetry:mainfrom
martincostello:update-ubuntu-runners
Open

[Infra] Update ubuntu GitHub runners#7337
martincostello wants to merge 2 commits into
open-telemetry:mainfrom
martincostello:update-ubuntu-runners

Conversation

@martincostello
Copy link
Copy Markdown
Member

@martincostello martincostello commented May 21, 2026

Changes

  • Upgrade from ubuntu-22.04 to ubuntu-24.04.
  • Use ubuntu-24.04 instead of ubuntu-latest.

I can see in the dashboard that renovate is detecting the runner labels, but I don't know why it isn't trying to update them.

Merge requirement checklist

  • CONTRIBUTING guidelines followed (license requirements, nullable enabled, static analysis, etc.)
  • Unit tests added/updated
  • Appropriate CHANGELOG.md files updated for non-trivial changes
  • Changes in public API reviewed (if applicable)

- Upgrade from `ubuntu-22.04` to `ubuntu-24.04`.
- Use `ubuntu-24.04` instead of `ubuntu-latest`.
@github-actions github-actions Bot added the infra Infra work - CI/CD, code coverage, linters label May 21, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 21, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.94%. Comparing base (7145855) to head (51c85cd).
✅ All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #7337      +/-   ##
==========================================
- Coverage   89.94%   89.94%   -0.01%     
==========================================
  Files         276      276              
  Lines       14007    14007              
==========================================
- Hits        12599    12598       -1     
- Misses       1408     1409       +1     
Flag Coverage Δ
unittests-Project-Experimental 89.74% <ø> (-0.17%) ⬇️
unittests-Project-Stable 89.82% <ø> (-0.06%) ⬇️
unittests-Solution 89.75% <ø> (-0.12%) ⬇️
unittests-UnstableCoreLibraries-Experimental 47.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 7 files with indirect coverage changes

@martincostello martincostello marked this pull request as ready for review May 21, 2026 14:44
@martincostello martincostello requested a review from a team as a code owner May 21, 2026 14:45
Copilot AI review requested due to automatic review settings May 21, 2026 14:45
@martincostello martincostello enabled auto-merge May 21, 2026 14:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates GitHub Actions workflow runner labels to explicitly use ubuntu-24.04 (including updating remaining ubuntu-22.04 and ubuntu-latest references), aligning CI execution environments on a consistent Ubuntu version.

Changes:

  • Upgrade Ubuntu runner labels from ubuntu-22.04 to ubuntu-24.04 in matrix-based workflows.
  • Replace ubuntu-latest with ubuntu-24.04 in several standalone workflow jobs.
  • Update the reusable component build workflow default OS list and matrix exclusions to use ubuntu-24.04 / ubuntu-24.04-arm.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.github/workflows/verifyaotcompat.yml Updates the Ubuntu OS matrix entry to ubuntu-24.04.
.github/workflows/survey-on-merged-pr.yml Pins the job runner from ubuntu-latest to ubuntu-24.04.
.github/workflows/ossf-scorecard.yml Pins the Scorecard job runner from ubuntu-latest to ubuntu-24.04.
.github/workflows/fossa.yml Pins the FOSSA scan job runner from ubuntu-latest to ubuntu-24.04.
.github/workflows/concurrency-tests.yml Updates the Ubuntu OS matrix entry to ubuntu-24.04.
.github/workflows/Component.BuildTest.yml Updates default OS list and net462 exclusions to ubuntu-24.04 / ubuntu-24.04-arm.
.github/workflows/codeql-analysis-steps.yml Pins the results job runner from ubuntu-latest to ubuntu-24.04.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@thompson-tomo
Copy link
Copy Markdown
Contributor

Fyi @martincostello reason renovate is not updating it is the version is not being extracted correctly by the regex. The version should not contain Ubuntu but use the number and potentially -arm. See ruby & ruby contrib for an example of where I have gotten it working.

@martincostello
Copy link
Copy Markdown
Member Author

Thanks - I could find few examples when I searched GitHub for syntax that seemed to have evidence in the Git history of it working at some point when I added this to my own personal repositories, but as they were all already using up-to-date images I didn't see any evidence either way of it working or not. The changes were in anticipation of ubuntu-26.04 runners becoming available at some point.

@martincostello
Copy link
Copy Markdown
Member Author

Your approach appears to be different in that it's matching to a conventional based property name and then the customManager telling it explicitly it's github-runners rather than using the comment-based syntax (based on my interpretation of this and this).

Ideally I'd like it work without needing to match to a specific matrix/property name and use the comment-hint as I think it's more declarative when you need to go outside of the defaults (like how Docker _VERSION support works).

@martincostello martincostello disabled auto-merge May 21, 2026 21:43
@martincostello martincostello enabled auto-merge May 21, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Infra work - CI/CD, code coverage, linters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants