Skip to content

[Docs][KubeRay] Update KubeRay + Kueue guides to use newer versions of Kueue#48564

Merged
pcmoritz merged 1 commit intoray-project:masterfrom
andrewsykim:kueue-guide-version-update
Nov 5, 2024
Merged

[Docs][KubeRay] Update KubeRay + Kueue guides to use newer versions of Kueue#48564
pcmoritz merged 1 commit intoray-project:masterfrom
andrewsykim:kueue-guide-version-update

Conversation

@andrewsykim
Copy link
Member

Why are these changes needed?

Update KubeRay + Kueue guides to use newer versions of Kueue

Related issue number

Checks

  • 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.
  • I've included any doc changes needed for https://docs.ray.io/en/master/.
    • 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.
  • 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 :(

Signed-off-by: Andrew Sy Kim <andrewsy@google.com>
@andrewsykim andrewsykim force-pushed the kueue-guide-version-update branch from 37f8cea to ff3d7b8 Compare November 5, 2024 16:09
gcloud container clusters create kuberay-gpu-cluster \
--num-nodes=1 --min-nodes 0 --max-nodes 1 --enable-autoscaling \
--zone=us-west1-b --machine-type e2-standard-4 --cluster-version 1.29
--zone=us-east4-c --machine-type e2-standard-4
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I switched the zone just based on personal experience, this zone has less stockouts for L4 GPUs

@kevin85421
Copy link
Member

I have already informed our doc team to review the PR.

@pcmoritz pcmoritz enabled auto-merge (squash) November 5, 2024 18:07
@github-actions github-actions bot added the go add ONLY when ready to merge, run all tests label Nov 5, 2024
@pcmoritz pcmoritz merged commit 11aca59 into ray-project:master Nov 5, 2024
```

This command creates a node pool which initially has zero nodes. Use the `gcloud beta` command because some of the flags have beta status.
This command creates a node pool which initially has zero nodes.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing comma:
This command creates a node pool, which initially has zero nodes.

@angelinalg angelinalg mentioned this pull request Nov 5, 2024
angelinalg added a commit that referenced this pull request Nov 5, 2024
missing comma related to #48564
Had to try again because @can-anyscale suspects a rare ReadTheDocs failure and there's no easy way to kick off a rebuild from the browser.

Signed-off-by: angelinalg <122562471+angelinalg@users.noreply.github.com>
angelinalg added a commit that referenced this pull request Nov 6, 2024
missing comma related to #48564
Had to try again because @can-anyscale suspects a rare ReadTheDocs
failure and there's no easy way to kick off a rebuild from the browser.

Signed-off-by: angelinalg <122562471+angelinalg@users.noreply.github.com>
JP-sDEV pushed a commit to JP-sDEV/ray that referenced this pull request Nov 14, 2024
…f Kueue (ray-project#48564)

## Why are these changes needed?

Update KubeRay + Kueue guides to use newer versions of Kueue

## Related issue number

<!-- For example: "Closes ray-project#1234" -->

## Checks

- [ ] 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.
- [ ] I've included any doc changes needed for
https://docs.ray.io/en/master/.
- [ ] 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 :(

Signed-off-by: Andrew Sy Kim <andrewsy@google.com>
JP-sDEV pushed a commit to JP-sDEV/ray that referenced this pull request Nov 14, 2024
missing comma related to ray-project#48564
Had to try again because @can-anyscale suspects a rare ReadTheDocs
failure and there's no easy way to kick off a rebuild from the browser.

Signed-off-by: angelinalg <122562471+angelinalg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go add ONLY when ready to merge, run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants