-
Notifications
You must be signed in to change notification settings - Fork 392
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
[Feature][kubectl-plugin] Add all
and worker
node type to kubectl ray log
#2442
base: master
Are you sure you want to change the base?
Conversation
all
and worker
node type to kubectl ray logall
and worker
node type to kubectl ray log
d49d149
to
8c526e3
Compare
6380f38
to
03f0edc
Compare
The "Downloading file" logs seem pretty noisy. Can we only log it once per node?
|
03f0edc
to
3c314b5
Compare
3c314b5
to
f027d0d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One minor nit, otherwise LGTM
} | ||
|
||
fmt.Fprintf(options.ioStreams.Out, "Downloading file for Ray Node %s\n", rayNode.Name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Downloading logs for Ray Node
Why are these changes needed?
This PR adds the node type "worker" and "all" to
kubectl ray log
Worker logs:
All RayCluster logs:
Related issue number
Part of #2419
Checks