Update kind configs and docs to v1.29.0#4595
Merged
andrewsykim merged 1 commit intoray-project:masterfrom Mar 16, 2026
Merged
Conversation
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
mboersma
force-pushed
the
update-kind-configs
branch
from
c50f301 to
1800576
Compare
| nodes: | ||
| - role: control-plane | ||
| image: kindest/node:v1.25.0@sha256:428aaa17ec82ccde0131cb2d1ca6547d13cf5fdabcc0bbecf749baa935387cbf | ||
| image: kindest/node:v1.29.0 |
There was a problem hiding this comment.
Two CI kind configs are now identical
Low Severity
After updating ci/kind-config-buildkite.yml to use kindest/node:v1.29.0, it is now effectively identical to ci/kind-config-buildkite-1-29.yml (which already used v1.29.0). The Buildkite E2E pipeline in .buildkite/test-e2e.yml references both files in different test steps — the "RayService Incremental Upgrade E2E" test specifically uses the -1-29 variant while all others use the default. This distinction is now meaningless since both produce the same cluster configuration, creating redundant files and a misleading test setup.
Contributor
Author
There was a problem hiding this comment.
It seems like the -buildkite-1-29.yml config was intentionally there to test v1.29. I don't think it's worth refactoring them in this PR; instead we should look next at bumping the default everywhere forward from 1.29 (which is old), and leaving the -buildkite-1-29.yml config in place, at which point they will again diverge intentionally.
andrewsykim
approved these changes
Mar 16, 2026
Member
andrewsykim
left a comment
andrewsykim
left a comment
There was a problem hiding this comment.
LGTM and +1 to update to newer Kubernetes version in follow-up PR as v1.29 is already out of support.
Contributor
Author
|
I opened #4606 to track updating to a newer Kubernetes version. |
rueian
added a commit
that referenced
this pull request
Mar 19, 2026
* Disable RayMultiHostIndexing feature for TestReconcile_Multihost_Replicas (#4583) Signed-off-by: Future-Outlier <[email protected]> * Disable the field alignment lint check (#4560) * chore: disable govet fieldalignment lint Signed-off-by: jinbum9958 <[email protected]> * chore: remove redundant //nolint:govet comments Signed-off-by: jinbum9958 <[email protected]> * chore: restore RayCronJobSpec/RayCronJob comments Signed-off-by: jinbum9958 <[email protected]> * chore: restore RayCronJob comment Signed-off-by: jinbum9958 <[email protected]> * chore: regenerate artifacts for RayCronJob comment changes Signed-off-by: jinbum9958 <[email protected]> --------- Signed-off-by: jinbum9958 <[email protected]> * Mark RC releases as pre-release in GoReleaser (#4590) Signed-off-by: Future-Outlier <[email protected]> * Add example YAML for manually enabling Ray k8s auth (#4582) Signed-off-by: Andrew Sy Kim <[email protected]> * [RayService] Rollback Support for Incremental Upgrades (#4109) * Implement rollback support Signed-off-by: Ryan O'Leary <[email protected]> * Fix unit test file Signed-off-by: Ryan O'Leary <[email protected]> * Ensure upgrade in progress status is cleared Signed-off-by: Ryan O'Leary <[email protected]> * Clarify rollback scenarios and clear pending apps during rollback Signed-off-by: Ryan O'Leary <[email protected]> * Fix typo Signed-off-by: Ryan O'Leary <[email protected]> * Add guard for rollback to prepare new cluster Signed-off-by: Ryan O'Leary <[email protected]> * fix e2e test checking pending cluster name Signed-off-by: Ryan O'Leary <[email protected]> * Fix rollback calculation Signed-off-by: Ryan O'Leary <[email protected]> * Fix httproute name in test Signed-off-by: Ryan O'Leary <[email protected]> * Increase timeout for checking RayCluster deletion Signed-off-by: Ryan O'Leary <[email protected]> * Fix rollback check and change httproute equal check so it doesn't always evaluate false Signed-off-by: Ryan O'Leary <[email protected]> * Adjust logic to use timestamp of growing cluster Signed-off-by: Ryan O'Leary <[email protected]> * Fix naming of vars Signed-off-by: Ryan O'Leary <[email protected]> --------- Signed-off-by: Ryan O'Leary <[email protected]> Signed-off-by: Ryan O'Leary <[email protected]> * [RayService] Promote Incremental Upgrade Feature to Beta (#4599) Signed-off-by: Ryan O'Leary <[email protected]> * Revert "[RayService] Promote Incremental Upgrade Feature to Beta (#4599)" (#4602) This reverts commit 0c8aab9. * [Github Action] Skip krew-index update for pre-release tags (#4587) * [CI] Skip krew-index update for pre-release tags Co-Authored-By: Claude Opus 4.6 <[email protected]> Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> --------- Signed-off-by: Future-Outlier <[email protected]> Co-authored-by: Claude Opus 4.6 <[email protected]> * [Helm] update ray-cluster chart to apiVersion: v2 (#4593) * Update kind configs and docs to v1.29.0 (#4595) * Remove ingress template support for k8s < 1.19 (#4591) * [Helm] Update ray-cluster default resource values (#4588) * update kuberay 1.6.0 helm chart and readme Signed-off-by: Future-Outlier <[email protected]> * [Helm] Fix apiserver README chart version and align ray-cluster resource defaults Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * fix ctl test Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> --------- Signed-off-by: Future-Outlier <[email protected]> * fix lint Signed-off-by: Future-Outlier <[email protected]> * chore: Use RayCluster name as SA name for RBAC auth (#4611) Signed-off-by: JiangJiaWei1103 <[email protected]> * Add Google Artifact Registry image build/push guide (#4618) * [History Server] Fix API response format to match Ray Dashboard frontend schema (#4615) * [History Server] Fix API response format to match Ray Dashboard frontend schema Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * fix bug 1 Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * add comments to explain frontend fields naming issue Signed-off-by: Future-Outlier <[email protected]> * fix tests Signed-off-by: Future-Outlier <[email protected]> * Add comments Signed-off-by: Future-Outlier <[email protected]> * update Signed-off-by: Future-Outlier <[email protected]> * Add frontend reference for this PR Signed-off-by: Future-Outlier <[email protected]> * fix codex and cursor bug Signed-off-by: Future-Outlier <[email protected]> * upadte Signed-off-by: Future-Outlier <[email protected]> * add actor.LabelSelector Signed-off-by: Future-Outlier <[email protected]> * better query for additional endpoints, request URI (path + query) Signed-off-by: Future-Outlier <[email protected]> * Add comments Signed-off-by: Future-Outlier <[email protected]> --------- Signed-off-by: Future-Outlier <[email protected]> --------- Signed-off-by: Future-Outlier <[email protected]> Signed-off-by: jinbum9958 <[email protected]> Signed-off-by: Andrew Sy Kim <[email protected]> Signed-off-by: Ryan O'Leary <[email protected]> Signed-off-by: Ryan O'Leary <[email protected]> Signed-off-by: JiangJiaWei1103 <[email protected]> Co-authored-by: Han-Ju Chen (Future-Outlier) <[email protected]> Co-authored-by: Jinbum Kim <[email protected]> Co-authored-by: Andrew Sy Kim <[email protected]> Co-authored-by: Ryan O'Leary <[email protected]> Co-authored-by: Claude Opus 4.6 <[email protected]> Co-authored-by: Matt Boersma <[email protected]> Co-authored-by: Jia-Wei Jiang <[email protected]> Co-authored-by: Chia-Yi Liang <[email protected]>
6 tasks
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
2 participants
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.
Why are these changes needed?
Several kind cluster configs reference various Kubernetes versions that are well past end-of-life. This syncs all the configs and docs to v1.29.0, the latest version already referenced.
v1.29.0 is also old, so perhaps we should consider moving that forward as well, but I wanted to ensure this didn't break things first.
Related issue number
Checks