Skip to content
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

[fix][kubectl-plugin] incorrect console message #3019

Closed
wants to merge 1 commit into from

Conversation

davidxia
Copy link
Contributor

in kubectl ray job submit command.

There's no error details we can show so remove them. Currently console messages show.

$ kubectl ray job submit -f /tmp/test/ray-job.sample.yaml --working-dir /tmp/test -- python /tmp/test/script.py

Submitted RayJob dxia-rayjob.
Waiting for RayCluster
Checking Cluster Status for cluster dxia-rayjob-raycluster-sslzr...
Cluster is not ready: %!w(<nil>)
Cluster is not ready: %!w(<nil>)
Cluster is not ready: %!w(<nil>)
...

Checks

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

in `kubectl ray job submit` command.

There's no error details we can show so remove them.
Currently console messages show.

```console
$ kubectl ray job submit -f /tmp/test/ray-job.sample.yaml --working-dir /tmp/test -- python /tmp/test/script.py

Submitted RayJob dxia-rayjob.
Waiting for RayCluster
Checking Cluster Status for cluster dxia-rayjob-raycluster-sslzr...
Cluster is not ready: %!w(<nil>)
Cluster is not ready: %!w(<nil>)
Cluster is not ready: %!w(<nil>)
...
```

Signed-off-by: David Xia <[email protected]>
@davidxia davidxia marked this pull request as ready for review February 12, 2025 00:41
@davidxia
Copy link
Contributor Author

dupe of #3027

@davidxia davidxia closed this Feb 12, 2025
@davidxia davidxia deleted the fix20 branch February 12, 2025 19:05
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.

1 participant