Skip to content

Conversation

Future-Outlier
Copy link
Member

@Future-Outlier Future-Outlier commented Aug 5, 2025

Why are these changes needed?

Some users may not know how to configure the ReconcileConcurrency in kuberay.

Docs link: https://anyscale-ray--55236.com.readthedocs.build/en/55236/cluster/kubernetes/troubleshooting/troubleshooting.html#how-to-configure-reconcile-concurrency-when-there-are-large-mount-of-crs

Related issue number

ray-project/kuberay#3909

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 :(

@Future-Outlier Future-Outlier requested review from a team as code owners August 5, 2025 03:06
Copy link
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@ray-gardener ray-gardener bot added community-contribution Contributed by the community docs An issue or change related to documentation core Issues that should be addressed in Ray Core labels Aug 5, 2025
Copy link
Member

@owenowenisme owenowenisme left a comment

Choose a reason for hiding this comment

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

LG!

Copy link
Member

@owenowenisme owenowenisme left a comment

Choose a reason for hiding this comment

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

But would it be better if we add link of this section to KubeRay autoscaler?
So users can know what to do when experiencing such scenario.
https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/configuring-autoscaling.html

@Future-Outlier
Copy link
Member Author

But would it be better if we add link of this section to KubeRay autoscaler? So users can know what to do when experiencing such scenario. https://docs.ray.io/en/latest/cluster/kubernetes/user-guides/configuring-autoscaling.html

great advice, let me wait for ruiean's look, thank you!


### How to configure reconcile concurrency?

The KubeRay operator supports configuring the `ReconcileConcurrency` setting, which controls the number of concurrent workers processing Ray custom resources (CRs).
Copy link
Contributor

Choose a reason for hiding this comment

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

Mentioning concurrent workers is great, but could you also add why a user wants to change the setting more explicitly?

Copy link
Member Author

Choose a reason for hiding this comment

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

updated, thank you!

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, the issue is still open. We'd better wait for it to be concluded if we want to reference the issue here.

Copy link
Member Author

Choose a reason for hiding this comment

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

no problem, thank you!

@angelinalg angelinalg added the go add ONLY when ready to merge, run all tests label Aug 19, 2025

### How to configure reconcile concurrency when there are large mount of CRs?

Take [kuberay#3909](https://github.com/ray-project/kuberay/issues/3909) as an example.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Take [kuberay#3909](https://github.com/ray-project/kuberay/issues/3909) as an example.
In this example, [kuberay#3909](https://github.com/ray-project/kuberay/issues/3909),

Copy link

github-actions bot commented Sep 3, 2025

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

@github-actions github-actions bot added the stale The issue is stale. It will be closed within 7 days unless there are further conversation label Sep 3, 2025
Copy link

This pull request has been automatically closed because there has been no more activity in the 14 days
since being marked stale.

Please feel free to reopen or open a new pull request if you'd still like this to be addressed.

Again, you can always ask for help on our discussion forum or Ray's public slack channel.

Thanks again for your contribution!

@github-actions github-actions bot closed this Sep 17, 2025
Signed-off-by: Future-Outlier <[email protected]>
@Future-Outlier
Copy link
Member Author

cc @jjyao for merge, thank you!

@github-actions github-actions bot added unstale A PR that has been marked unstale. It will not get marked stale again if this label is on it. and removed stale The issue is stale. It will be closed within 7 days unless there are further conversation labels Oct 13, 2025
@edoakes edoakes merged commit 21b68c5 into ray-project:master Oct 13, 2025
6 checks passed
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 go add ONLY when ready to merge, run all tests unstale A PR that has been marked unstale. It will not get marked stale again if this label is on it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants