ray: update KubeRay and Ray versions#3528
Conversation
|
Welcome to the Kubeflow Community Distribution Repository Thanks for opening your first PR. Your contribution means a lot to the Kubeflow community. Before making more PRs: Community Resources:
Thanks again for helping to improve Kubeflow. |
There was a problem hiding this comment.
Pull request overview
This pull request updates the experimental Ray manifests to align the checked-in KubeRay operator/chart version and the Ray example/test versions.
Changes:
- Bumped the KubeRay Helm chart/operator version to
1.6.2in the experimental Ray Makefile. - Updated the Ray example/test versions to Ray
2.56.0across the README, example RayCluster manifest, and test script. - Updated the RayCluster example images to
rayproject/ray:2.56.0-py311-cpu.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| experimental/ray/test.sh | Updates the expected Ray version for the smoke test. |
| experimental/ray/README.md | Updates documented Ray image and client install version to 2.56.0. |
| experimental/ray/raycluster_example.yaml | Updates RayCluster rayVersion and container images to 2.56.0. |
| experimental/ray/Makefile | Bumps KubeRay chart version to 1.6.2. |
| @@ -5,7 +5,7 @@ set -euxo | |||
| NAMESPACE=$1 | |||
| TIMEOUT=120 # timeout in seconds | |||
| SLEEP_INTERVAL=30 # interval between checks in seconds | |||
Signed-off-by: danish9039 <danishsiddiqui040@gmail.com>
557a8b6 to
530a416
Compare
|
/lgtm are there changes needed for https://github.com/danish9039/manifests/blob/pr3487/ray-kuberay-version-alignment/experimental/ray/kuberay-operator/base/aggregated-roles.yaml ? |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juliusvonkohout The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@juliusvonkohout No change was required for this version bump because the existing ray.io wildcard ( * ) already covers RayCronJob, but replacing it with an explicit resource list would be a good least-privilege follow-up. Do you want that change ?? |
Pull Request Template for Kubeflow Manifests
Summary of Changes
This PR updates the experimental Ray manifests to align the checked-in KubeRay operator and Ray example/test versions:
1.6.2rayproject/ray:2.56.0-py311-cpurayVersion:2.56.0The KubeRay operator resources were regenerated with the existing Makefile path.
Reviewer note: the generated KubeRay
1.6.2chart has large expected churn. It adds theraycronjobs.ray.ioCRD and RayCronJob roles, expands operator RBAC for resources such assecrets,pods/resize,endpointslices, and Gateway API resources, and raw generated namespaced objects includenamespace: default; the Kubeflow overlay still renders namespaced resources intokubeflow.This PR intentionally does not include the restricted-PSS Ray example changes from #3487.
Dependencies
No direct dependency. Related to the broader PR split from #3487.
Related Issues
Related to #3487.
Contributor Checklist
Validation:
make -C experimental/ray kuberay-operator/basekustomize build experimental/ray/kuberay-operator/basekustomize build experimental/ray/kuberay-operator/overlays/kubeflowbash -n experimental/ray/test.shgit diff --check