chore(ray-image): bump version matrix to latest 3 Ray releases - #259
Merged
Merged
Conversation
Replace the build matrix in images/ray/versions.json with the 3 latest upstream Ray releases (2.56.1, 2.57.0, 2.58.0), moving the default (:latest) combo from 2.56.0 to 2.58.0. Sync the Makefile's RAY_VERSION fallback and the README's documented defaults/examples to match. No CI workflow currently reads versions.json directly; the offline e2e harness (tests/e2e/stack/harness/*_conformance.sh) resolves the image via `jq max_by`, which continues to pick 2.58.0 correctly. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…sion-matrix # Conflicts: # images/ray/Makefile # images/ray/README.md # images/ray/versions.json
Alyssa Vu (alyssa1303)
marked this pull request as ready for review
September 14, 2026 19:00
Kevin Cho (chokevin)
approved these changes
Sep 14, 2026
Kevin Cho (chokevin)
left a comment
Contributor
There was a problem hiding this comment.
some pending comments offline but the idea looks good to me
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Bumps the
images/raybuild matrix (images/ray/versions.json) to append the3 latest Ray releases published in ray-project/ray
and move the default (
:latest-tagged) combo forward:images/ray/Makefile'sRAY_VERSIONfallback and all version references inimages/ray/README.mdare synced to the new default (2.58.0).#233 merged while this PR was open, with a different fix than my original
draft (it moved to Ray 2.56.1 upstream, not a local
protobuf<5pin), andit also introduced a new documented policy for this matrix: append-only —
preserve historical entries, don't drop them (
versions.jsonis "releaseinput, not evidence of published availability", and matrix changes must
"coordinate ... with the ADO release owner").
I merged
maininto this branch and resolved the conflicts to follow thatpolicy instead of my original plan (which replaced the matrix with only the
3 latest entries). Net effect on top of current
main:versions.json: append2.57.0and2.58.0, movedefault: truefrom2.56.1to2.58.0. All 4 pre-existing entries (2.54.0/2.55.1/2.56.0/2.56.1)are untouched.
Makefile:RAY_VERSIONfallback2.56.1→2.58.0.README.md: version-specific examples/table values updated to2.58.0.I also corrected one line the automatic line-based merge combined
incorrectly — the "Image Tag Format" section's prose said "currently
published 2.56.0" while the code block under it had been merged to show
2.58.0; restored the code block to2.56.0since I have no evidence a2.58.0image has actually been published (publishing is owned by the ADOrepo per the new CI/CD section, not this PR).
Verification
versions.jsondirectly.tests/e2e/stack/harness/{fineweb,large_gpu}_ib_conformance.shresolvetheir Ray image via
jq -er 'max_by(.ray | split(".") | map(tonumber))'against
versions.json— reran that exact query against the merged fileand confirmed it resolves to
2.58.0as expected.python3 -c "import json; json.load(open('images/ray/versions.json'))"—valid JSON, exactly one
default: trueentry (verified withjq '[.[] | select(.default==true)] | length'→1).make -n docker-build/make -n test— Makefile rendersRAY_VERSION=2.58.0correctly, no syntax issues.python3 scripts/check-license-headers.pypasses.<<<<<<</=======/>>>>>>>markers — none.Scope note
Still scoped to the
images/raybuild matrix only — not the other reporeferences to the currently-published
ray:py3.12-ray2.56.0-cuda13.0tag(examples/, charts/taugrid-core, tests/e2e fixtures, cluster-overlays
dashboards). Per the new README, bumping what's actually published/consumed
is an ADO-owned release step, not a contributor PR.
Test plan
images/rayfor the new matrix entries(2.57.0, 2.58.0), per the repo's ADO-owned build/publish process.
make testagainst each newly built image.