Skip to content

[Doc] Bind the version of kuberay to v1.3.0 in related docs#53661

Merged
edoakes merged 1 commit intoray-project:masterfrom
Myasuka:kuberay-doc-v1.3.0
Jun 11, 2025
Merged

[Doc] Bind the version of kuberay to v1.3.0 in related docs#53661
edoakes merged 1 commit intoray-project:masterfrom
Myasuka:kuberay-doc-v1.3.0

Conversation

@Myasuka
Copy link
Copy Markdown
Contributor

@Myasuka Myasuka commented Jun 9, 2025

Why are these changes needed?

Current ray's doc has a description of how to enable autoscaler V2. However, the ray-cluster.autoscaler-v2.yaml example to the https://github.com/ray-project/kuberay/blob/master/ray-operator/config/samples/ray-cluster.autoscaler-v2.yaml which does not have related configuration of RAY_enable_autoscaler_v2. This is because that commit-a9c48c introduced a new feature, which has not been released yet.

To reduce such problem, we should make the ray's doc bind to a specific released kuberay version.

Related issue number

Closes #53655

Checks

  • I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • I've run scripts/format.sh to lint the changes in this PR.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • I've added any new APIs to the API Reference. For example, if I added a
      method in Tune, I've added it in doc/source/tune/api/ under the
      corresponding .rst file.
  • I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • Unit tests
    • Release tests
    • This PR is not tested :(

Copilot AI review requested due to automatic review settings June 9, 2025 16:20
@Myasuka Myasuka requested review from a team, kevin85421 and pcmoritz as code owners June 9, 2025 16:20
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates Kubernetes autoscaling and Prometheus/Grafana documentation to reference the released KubeRay v1.3.0 tag rather than the master branch, preventing confusion when examples lag behind unreleased features.

  • Bind sample YAML and scripts to v1.3.0 in docs
  • Ensure Prometheus/Grafana config links point to the released version

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
doc/source/cluster/kubernetes/user-guides/configuring-autoscaling.ipynb Updated ray-cluster.autoscaler-v2.yaml link to v1.3.0
doc/source/cluster/kubernetes/k8s-ecosystem/prometheus-grafana.md Updated install.sh, podMonitor.yaml, and prometheusRules.yaml links to v1.3.0
Comments suppressed due to low confidence (2)

doc/source/cluster/kubernetes/k8s-ecosystem/prometheus-grafana.md:38

  • The link for Grafana JSON files still points to the master branch. To ensure consistency with the rest of this guide, bind it to the v1.3.0 tag (e.g., https://github.com/ray-project/kuberay/tree/v1.3.0/config/grafana).
* Import Ray Dashboard’s [Grafana JSON files](https://github.com/ray-project/kuberay/tree/master/config/grafana) into Grafana using the `--auto-load-dashboard true` flag.

doc/source/cluster/kubernetes/user-guides/configuring-autoscaling.ipynb:1089

  • Since the link now targets a specific v1.3.0 tag, consider updating the text from "latest KubeRay version" to "KubeRay v1.3.0" for clarity.
[ray-cluster.autoscaler-v2.yaml](https://github.com/ray-project/kuberay/blob/v1.3.0/ray-operator/config/samples/ray-cluster.autoscaler-v2.yaml) is an example YAML file of a RayCluster with Autoscaler V2 enabled that works with the latest KubeRay version.

Copy link
Copy Markdown
Member

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

Revert the changes for prometheus-grafana.md and others LGTM.

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.

There have been some recent changes to install.sh, and we plan to release KubeRay v1.4.0 this week. That’s why we’re pointing to the master branch for now. We will update it to v1.4.0 recently.

Can you revert the changes for this file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Already reverted the changes.

Signed-off-by: Yun Tang <myasuka@live.com>
@Myasuka Myasuka force-pushed the kuberay-doc-v1.3.0 branch from 92d5c04 to b5760f3 Compare June 10, 2025 12:51
@kevin85421 kevin85421 added the go add ONLY when ready to merge, run all tests label Jun 11, 2025
@kevin85421
Copy link
Copy Markdown
Member

@Myasuka I have already triggered the full CI run. Please ping me when all CI tests pass. Thank you!

@Myasuka
Copy link
Copy Markdown
Contributor Author

Myasuka commented Jun 11, 2025

@kevin85421 all tests already passed.

@kevin85421
Copy link
Copy Markdown
Member

cc @jjyao @edoakes would you mind merging this PR? Thanks!

@edoakes edoakes merged commit 339c019 into ray-project:master Jun 11, 2025
6 checks passed
elliot-barn pushed a commit that referenced this pull request Jun 18, 2025
Current ray's doc has a description of [how to enable autoscaler
V2](https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/configuring-autoscaling.html#autoscaler-v2-with-kuberay).
However, the ray-cluster.autoscaler-v2.yaml example to the
https://github.com/ray-project/kuberay/blob/master/ray-operator/config/samples/ray-cluster.autoscaler-v2.yaml
which does not have related configuration of RAY_enable_autoscaler_v2.
This is because that
[commit-a9c48c](ray-project/kuberay@a9c48c1)
introduced a new feature, which has not been released yet.

To reduce such problem, we should make the ray's doc bind to a specific
released kuberay version.

Closes #53655

Signed-off-by: Yun Tang <myasuka@live.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
elliot-barn pushed a commit that referenced this pull request Jul 2, 2025
Current ray's doc has a description of [how to enable autoscaler
V2](https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/configuring-autoscaling.html#autoscaler-v2-with-kuberay).
However, the ray-cluster.autoscaler-v2.yaml example to the
https://github.com/ray-project/kuberay/blob/master/ray-operator/config/samples/ray-cluster.autoscaler-v2.yaml
which does not have related configuration of RAY_enable_autoscaler_v2.
This is because that
[commit-a9c48c](ray-project/kuberay@a9c48c1)
introduced a new feature, which has not been released yet.

To reduce such problem, we should make the ray's doc bind to a specific
released kuberay version.

Closes #53655

Signed-off-by: Yun Tang <myasuka@live.com>
Signed-off-by: elliot-barn <elliot.barnwell@anyscale.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Kuberay] The reference of Kuberay code link should bind to a release version

4 participants