8000 [feat][python-client]: add rayjob support to kuberay python-client by kryanbeane · Pull Request #3830 · ray-project/kuberay · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[feat][python-client]: add rayjob support to kuberay python-client #3830

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kryanbeane
Copy link
@kryanbeane kryanbeane commented Jun 24, 2025

Why are these changes needed?

Kuberay python-client has support for creating RayClusters. This change adds similar support to the client for Ray Jobs

Related issue number

3829

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

while timeout > 0:
status = self.get_ray_cluster_status(
name, k8s_namespace, timeout, delay_between_attempts
)

Copy link
Author
@kryanbeane kryanbeane Jun 25, 2025

Choose a reason for hiding this comment

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

Mostly formatting in this file, apart from this change. The status field for RayCluster has changed since this was implemented, and so the old version of this function did not actually wait for the Cluster to be ready.

@kryanbeane kryanbeane changed the title feat: add rayjob support to kuberay python-client [feat]: add rayjob support to kuberay python-client Jun 25, 2025
@kryanbeane kryanbeane changed the title [feat]: add rayjob support to kuberay python-client [feat][python-client]: add rayjob support to kuberay python-client Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0