Skip to content

[Fix] rayjob update raycluster status #4192

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.

Sign up for GitHub

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

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

[Fix] rayjob update raycluster status #4192

wants to merge 2 commits into from
+46 −1

Conversation

@machichima
Copy link
Collaborator

@machichima machichima commented Nov 15, 2025 •
edited
Loading

Why are these changes needed?

When the RayCluster created by RayJob has status update, it's not updated on the RayJob's RayClusterStatus field (e.g. KubeRay updates replicas field in RayCluster but not in RayJob when scaling up).

Modify the updateRayJobStatus function to also update rayjob status when RayClusterStatus field changes.

Test:

Result screenshot:

image

Related issue number

Closes #4190

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • e2e tests
    • Manual tests
    • This PR is not tested :(
@machichima machichima changed the title 4190 rayjob update raycluster status [Fix] rayjob update raycluster status Nov 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@machichima