ray icon indicating copy to clipboard operation
ray copied to clipboard

Add section on launching clusters on Kubernetes to Getting Started

Open omrishiv opened this issue 1 year ago • 5 comments
trafficstars

Why are these changes needed?

Hopefully this helps give a quick overview of using KubeRay for deploying clusters to Kubernetes

Related issue number

Closes #42129

Checks

  • [x] I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
  • [ ] I've run scripts/format.sh to lint the changes in this PR.
  • [x] I've included any doc changes needed for https://docs.ray.io/en/master/.
    • [x] I've added any new APIs to the API Reference. For example, if I added a method in Tune, I've added it in doc/source/tune/api/ under the corresponding .rst file.
  • [x] I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
  • Testing Strategy
    • [ ] Unit tests
    • [ ] Release tests
    • [ ] This PR is not tested :(

omrishiv avatar Jun 20 '24 04:06 omrishiv

@kevin85421 Hi! Should the link just go directly there? It seemed like the rest of the drop downs had a bit of information, so I wanted to make sure this had the minimal amount needed for deploying to Kubernetes

omrishiv avatar Jun 25 '24 21:06 omrishiv

Should the link just go directly there?

Yes, adding a link directly is fine. KubeRay tries to maintain a single source of truth for documents. We are trying to avoid maintaining consistency between the same content across different documents.

kevin85421 avatar Jun 26 '24 05:06 kevin85421

@kevin85421 Makes sense, hopefully this is more aligned with what you were expecting

omrishiv avatar Jun 26 '24 17:06 omrishiv

I will ask our doc team to review it.

kevin85421 avatar Jun 27 '24 20:06 kevin85421

Sure!: Added section for getting started: image

Where it leads to: image

omrishiv avatar Jun 28 '24 14:06 omrishiv

@can-anyscale just checking to see if the above looks good. Thanks!

omrishiv avatar Jul 02 '24 19:07 omrishiv

@angelinalg Thank you! also sorry, I may have screwed something up by updating my branch with main 😞

omrishiv avatar Jul 02 '24 21:07 omrishiv

cc @angelinalg or @can-anyscale would you mind merging this PR? Thanks!

kevin85421 avatar Jul 03 '24 16:07 kevin85421

@kevin85421 @angelinalg @can-anyscale Just wanted to gently nudge on this if possible.

omrishiv avatar Jul 09 '24 18:07 omrishiv

merged, thank you

can-anyscale avatar Jul 09 '24 18:07 can-anyscale