-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
samples: Kueue cohort tutorial samples #498
Conversation
…ue-cohort Merging to personal repo
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
Here is the summary of changes. You are about to add 6 region tags.
This comment is generated by snippet-bot.
|
FYI @alculquicondor |
Co-authored-by: Aldo Culquicondor <[email protected]>
Co-authored-by: Aldo Culquicondor <[email protected]>
Co-authored-by: Aldo Culquicondor <[email protected]>
Co-authored-by: Aldo Culquicondor <[email protected]>
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, pending removal of old yamls.
@rbarberop can you update this branch with the latest changes from the base branch? |
We shall bypass CLA check for @charlieyu1996 since he was a Googler when the commits were made! |
|
||
|
||
|
||
# ORIGINAL BELOW - KEPT FOR DOUBLE-CHECKING SYNTAX - TO BE DELETED BEFORE MERGE |
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.
Remove old yamls
Unable to merge due to local repo issues. Will open a new PR. |
This PR contains the yaml examples for the new Kueue tutorial using cohorts for resource sharing.
It contains v1beta1 code, so this PR should be blocked by kubernetes-sigs/kueue#532
Original v1alpha2 code is still on each file for reference. That part should be deleted on each file before merging.