Skip to content

chore(ray-image): bump version matrix to latest 3 Ray releases - #259

Merged
Alyssa Vu (alyssa1303) merged 2 commits into
mainfrom
alyssavu-bump-ray-version-matrix
Sep 14, 2026
Merged

Alyssa Vu (alyssa1303) merged 2 commits into
mainfrom
alyssavu-bump-ray-version-matrix

Conversation

@alyssa1303

@alyssa1303 Alyssa Vu (alyssa1303) commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Bumps the images/ray build matrix (images/ray/versions.json) to append the
3 latest Ray releases published in ray-project/ray
and move the default (:latest-tagged) combo forward:

Ray version Before After
2.54.0 present present
2.55.1 present present
2.56.0 present present
2.56.1 present (default) present
2.57.0 added
2.58.0 added, default

images/ray/Makefile's RAY_VERSION fallback and all version references in
images/ray/README.md are synced to the new default (2.58.0).

⚠️ Update: rebased on #233, approach revised

#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<5 pin), and
it also introduced a new documented policy for this matrix: append-only —
preserve historical entries, don't drop them
(versions.json is "release
input, not evidence of published availability", and matrix changes must
"coordinate ... with the ADO release owner").

I merged main into this branch and resolved the conflicts to follow that
policy instead of my original plan (which replaced the matrix with only the
3 latest entries). Net effect on top of current main:

  • versions.json: append 2.57.0 and 2.58.0, move default: true from
    2.56.1 to 2.58.0. All 4 pre-existing entries (2.54.0/2.55.1/2.56.0/2.56.1)
    are untouched.
  • Makefile: RAY_VERSION fallback 2.56.12.58.0.
  • README.md: version-specific examples/table values updated to 2.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 to 2.56.0 since I have no evidence a
    2.58.0 image has actually been published (publishing is owned by the ADO
    repo per the new CI/CD section, not this PR).

Verification

  • No CI workflow in this repo parses versions.json directly.
  • tests/e2e/stack/harness/{fineweb,large_gpu}_ib_conformance.sh resolve
    their Ray image via jq -er 'max_by(.ray | split(".") | map(tonumber))'
    against versions.json — reran that exact query against the merged file
    and confirmed it resolves to 2.58.0 as expected.
  • python3 -c "import json; json.load(open('images/ray/versions.json'))"
    valid JSON, exactly one default: true entry (verified with
    jq '[.[] | select(.default==true)] | length'1).
  • make -n docker-build / make -n test — Makefile renders
    RAY_VERSION=2.58.0 correctly, no syntax issues.
  • python3 scripts/check-license-headers.py passes.
  • Repo-wide grep for leftover <<<<<<</=======/>>>>>>> markers — none.

Scope note

Still scoped to the images/ray build matrix only — not the other repo
references to the currently-published ray:py3.12-ray2.56.0-cuda13.0 tag
(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

  • ADO release build of images/ray for the new matrix entries
    (2.57.0, 2.58.0), per the repo's ADO-owned build/publish process.
  • make test against each newly built image.

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
@alyssa1303
Alyssa Vu (alyssa1303) marked this pull request as ready for review September 14, 2026 19:00

@feiskyer Pengfei Ni (feiskyer) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@chokevin Kevin Cho (chokevin) 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.

some pending comments offline but the idea looks good to me

@alyssa1303
Alyssa Vu (alyssa1303) merged commit e1bcc77 into main Sep 14, 2026
30 checks passed
@alyssa1303
Alyssa Vu (alyssa1303) deleted the alyssavu-bump-ray-version-matrix branch September 14, 2026 22:46
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