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

Show pod list with IPs when getting clusters in client #408

Closed
wants to merge 2 commits into from

Conversation

harryge00
Copy link
Contributor

@harryge00 harryge00 commented Jul 25, 2022

Why are these changes needed?

Currently, in kuberay cli, cluster get does not provide any more useful information than cluster list.

image

In many cases, ray developer may not have the access to the kubernetes environment. After using kuberay client to create their ray clusters, the list of pods and ips are necessary for them to debug logs and ssh.
image

Related issue number

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Manual tests

@harryge00 harryge00 changed the title Getcluster cli Show pods' IPs when get cluster in client Jul 25, 2022
@harryge00 harryge00 changed the title Show pods' IPs when get cluster in client Show pod list with IPs when getting clusters in client Jul 25, 2022
@harryge00 harryge00 force-pushed the getcluster-cli branch 2 times, most recently from b9d20e0 to 512241a Compare July 25, 2022 10:15
@Jeffwan Jeffwan added this to the v0.4.0 release milestone Jul 25, 2022
@Jeffwan
Copy link
Collaborator

Jeffwan commented Jul 26, 2022

@harryge00 Is it ok to tag this change to v0.4.0 release?

@harryge00
Copy link
Contributor Author

@harryge00 Is it ok to tag this change to v0.4.0 release?

Of course. It's ok

@Jeffwan
Copy link
Collaborator

Jeffwan commented Aug 26, 2022

@harryge00 Could you rebase the master and I feel we can review and merge the code

@DmitriGekhtman
Copy link
Collaborator

@harryge00 could we resolve the merge conflicts on this one?

@DmitriGekhtman
Copy link
Collaborator

We should use a stale bot #681

In the meantime, @harryge00 @Jeffwan is there still interest in merging this?

Otherwise, I'll close it in a week.

@DmitriGekhtman DmitriGekhtman removed this from the v0.4.0 release milestone Nov 4, 2022
@DmitriGekhtman
Copy link
Collaborator

Closing as "stale". Obviously, feel free to reopen or open a new PR.

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.

3 participants