-
Notifications
You must be signed in to change notification settings - Fork 228
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
A mechanism to stop a ClusterQueue #1299
Conversation
Skipping CI for Draft Pull Request. |
✅ Deploy Preview for kubernetes-sigs-kueue ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@trasc: GitHub didn't allow me to request PR reviews from the following users: yaroslava-serdiuk. Note that only kubernetes-sigs members and repo collaborators can review this PR, and authors cannot review their own PRs. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/retitle A mechanism to stop a ClusterQueue |
/release-note-edit |
dccdbf6
to
282edf4
Compare
Please update based on the KEP |
/assign @yaroslava-serdiuk |
I'll review when tests are fixed |
5425b8d
to
b9f7d02
Compare
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.
Thanks for the change! Left few comments
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
I'll leave LGTM to @yaroslava-serdiuk
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.
Totaly LGTM
I left comments for some nits.
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: d26a520d427dc784205402ed64bf5ec15124c947
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: alculquicondor, trasc, yaroslava-serdiuk 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 |
/hold |
/unhold too late :) |
Could the title be misleading? Maybe "A mechanism to pause or stop a ClusterQueue" would be better? |
/release-note-edit
|
/kind feature |
/kind api-change |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Added a setting in a ClusterQueue
stopPolicy
, for an administrator to pause new admissions and/or stop all running jobs.Why is this needed:
Which issue(s) this PR fixes:
Fixes #1284
Special notes for your reviewer:
Does this PR introduce a user-facing change?