add operator controller that can eliminate co-scheduled pods that should be separate#1330
Conversation
|
/hold for proof |
openshift-ci
bot
added
the
do-not-merge/hold
openshift-ci
bot
added
the
approved
deads2k
force-pushed
the
kill-mal-scheduled-pod
branch
from
917680a to
129e11e
Compare
deads2k
force-pushed
the
kill-mal-scheduled-pod
branch
2 times, most recently
from
f6536ca to
eae28b4
Compare
deads2k
force-pushed
the
kill-mal-scheduled-pod
branch
from
eae28b4 to
53b4e03
Compare
|
/hold cancel |
|
/retest |
|
@deads2k: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions 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. |
deads2k
removed
the
do-not-merge/hold
| oldestPod.Spec.NodeName, | ||
| oldestPod.Name, | ||
| ) | ||
| err := c.kubeClient.CoreV1().Pods(oldestPod.Namespace).EvictV1(ctx, |
There was a problem hiding this comment.
Glad that we're using eviction API.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, ravisantoshgudimetla, sanchezl The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
4 participants
This will mitigate the platform impact of the scheduler problems.