-
Notifications
You must be signed in to change notification settings - Fork 268
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
Preemption admission check controller #1113
Conversation
Skipping CI for Draft Pull Request. |
✅ Deploy Preview for kubernetes-sigs-kueue canceled.
|
260cce8
to
1465d40
Compare
5685266
to
c2d3d12
Compare
561228c
to
8f2898b
Compare
/test all |
8f2898b
to
fdc606b
Compare
fdc606b
to
f7b8cfb
Compare
f7b8cfb
to
0d26e43
Compare
09146eb
to
bd5bdbe
Compare
bd5bdbe
to
68c14f4
Compare
ce7e628
to
415867d
Compare
415867d
to
9a116d3
Compare
PR needs rebase. 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. |
@trasc: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. I understand the commands that are listed here. |
Let's close this for now. Given all the work we are doing to implement fair sharing preemption, it's not a good time to put this in place. /close |
@alculquicondor: Closed this PR. 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. |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Add the possibility to delay the eviction triggering in case of preemption based on the preempting workload admission checks configuration.
Which issue(s) this PR fixes:
Fixes #1677
Special notes for your reviewer:
Some additional test will be added.
Does this PR introduce a user-facing change?