Skip to content

Conversation

@seanlaii
Copy link
Contributor

Description

Previously, the ray.io/scheduler-name label was used to determine the scheduler for KubeRay CRDs. However, with the adoption of the new scheduler framework, the scheduler name is now configured via the controller configuration, --batch-scheduler, and the label is no longer needed.
Therefore, we can remove the information related to ray.io/scheduler-name to avoid ambiguity.

Related issues

Related to ray-project/kuberay#4305

Additional information

@seanlaii seanlaii requested review from a team as code owners December 23, 2025 20:46
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the KubeRay & Volcano integration documentation. The changes correctly remove all references to the deprecated ray.io/scheduler-name: volcano label, which is no longer needed for scheduler configuration. The documentation is updated with a clear note explaining this change and now reflects the current practice of configuring the scheduler at the operator level. The PR also beneficially adds documentation for new Volcano labels related to network topology and clarifies the gang scheduling behavior with autoscaling. The updates are accurate and improve the clarity of the document for users.

@seanlaii
Copy link
Contributor Author

Hi @win5923 @Future-Outlier , please take a look when you have a chance. Thank you!
I messed up the previous PR so I create a new one.

@seanlaii seanlaii changed the title [doc] Update KubeRay & Volcano integration document [Docs] Update KubeRay & Volcano integration document Dec 23, 2025
Signed-off-by: seanlaii <qazwsx0939059006@gmail.com>
@ray-gardener ray-gardener bot added docs An issue or change related to documentation core Issues that should be addressed in Ray Core community-contribution Contributed by the community labels Dec 24, 2025
```

You can also be provide the following labels in the RayCluster metadata:
You can also provide the following labels in the RayCluster and RayJob metadata:
Copy link
Member

Choose a reason for hiding this comment

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

and RayService?

Copy link
Contributor Author

@seanlaii seanlaii Dec 24, 2025

Choose a reason for hiding this comment

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

We currently don't support RayService IIUC. I can help if we would like to support it.

Copy link
Member

@Future-Outlier Future-Outlier left a comment

Choose a reason for hiding this comment

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

cc @win5923 to take a look, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Contributed by the community core Issues that should be addressed in Ray Core docs An issue or change related to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants