Skip to content

Fix CI on the ubuntu-26.04 and Windows runner images - #337

Merged
StephenWakely merged 1 commit into
DataDog:masterfrom
corsonknowles:fix-ci-runner-images
Nov 11, 2026
Merged

StephenWakely merged 1 commit into
DataDog:masterfrom
corsonknowles:fix-ci-runner-images

Conversation

@corsonknowles

@corsonknowles corsonknowles commented Nov 9, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Two runner-image changes broke 7 jobs in the Ruby Test matrix on master. Every failure happens in ruby/setup-ruby, before any spec runs. This PR fixes both.

ubuntu: Ruby 2.1 and 2.2

ubuntu-latest is moving from 24.04 to 26.04. ruby-builder publishes 26.04 builds only for CRuby 2.3 and later (ruby/setup-ruby#954), so these jobs fail with:

Error: Unavailable version 2.1.9 for ruby on ubuntu-26.04

This PR excludes those two from ubuntu-latest and runs them on ubuntu-24.04 instead. Both builds exist there (the release assets return 200). 2.3 and later stay on ubuntu-latest.

windows: Ruby 2.4 through 3.0

These Rubies use the mingw64 msys2 toolchain. setup-ruby v1.335.0 downloads it from setup-msys2-gcc/releases/latest/download/..., but the latest release no longer ships mingw64 assets, so the download 404s:

Downloading msys2 build tools
https://github.com/ruby/setup-msys2-gcc/releases/latest/download/msys2-mingw64-mingw-w64@r688-gcc@14-openssl@1.1.7z
Error: Unexpected HTTP response: 404

In v1.338.0, windows-toolchain-versions.json pins those Rubies to a tagged release (v20261024.010459) that still has the asset. This PR bumps the pin to v1.338.0 (commit b3aacea, still SHA-pinned). The bump also brings in v1.337.0, which tests 2.4–3.1 on ubuntu-26.04.

Change

.github/workflows/test.yml only. No library or spec changes.

ubuntu-latest now resolves to ubuntu-26.04, which has no CRuby builds
older than 2.3, so run 2.1 and 2.2 on ubuntu-24.04 instead.

Bump ruby/setup-ruby to v1.338.0. v1.335.0 downloads the mingw64 msys2
toolchain for Ruby 2.4-3.0 on Windows from the latest setup-msys2-gcc
release, which no longer ships it (404); newer versions pin a tagged
release that does.
@corsonknowles
corsonknowles requested a review from a team as a code owner November 9, 2026 23:58
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 9, 2026 •

Copy link
Copy Markdown
Codex Review Summary: ✅ Completed

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-10T00:00:28.287599Z 5bf0d8a PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@corsonknowles

Copy link
Copy Markdown
Contributor Author

Thanks Codex!

@StephenWakely StephenWakely left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice, thanks.

@StephenWakely
StephenWakely merged commit 45f342a into DataDog:master Nov 11, 2026
37 checks passed
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