-
Notifications
You must be signed in to change notification settings - Fork 234
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
Add create clusterqueue command #2201
Add create clusterqueue command #2201
Conversation
Skipping CI for Draft Pull Request. |
✅ Deploy Preview for kubernetes-sigs-kueue canceled.
|
/test all |
5cf6b77
to
befd007
Compare
/test all |
/assign |
/test all |
/test pull-kueue-test-integration-main |
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 nit, otherwise 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.
/lgtm
LGTM label has been added. Git tree hash: 34a8eec2e5c5f3d01ef9e6a709b008e4c87f1f59
|
I'll leave the LGTM to @mwielgus |
/retest |
1 similar comment
/retest |
One review was done by @mwielgus in the past and all the points ware addressed. We can have a follow-up if something is still missing. /approve |
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.
just nit
Otherwise lgtm
/hold for comments leave lgtm on @trasc |
Thanks! |
LGTM label has been added. Git tree hash: 20b02b1124d646a2840d027acca31de89522e794
|
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.
/approve
/hold cancel
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, IrvingMg, tenzen-y, trasc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
* Add create clusterqueue command * Use ClientGetter from utils * Add unit tests for resource quotas parsing * Use flavorQuota wrapper for testing * Fix test case for unordered resources list * Validate resource groups * Validate quota params * Use Map function to cast slice * Make command usage message consistent with examples * Allow some special chars in resource name * Update create command examples * Replace error creation * Change validate method visibility
/release-note-edit
|
What type of PR is this?
/kind feature
What this PR does / why we need it:
Adds
kueuectl create clusterqueue
command.Which issue(s) this PR fixes:
Fixes #2113
Special notes for your reviewer:
Does this PR introduce a user-facing change?