[proposal] call eviction API before kill container

[j4ckstraw]

What is your proposal:

call eviction API before kill container

Why is this needed:

memoryEvict will kill container then call eviction api, this may disobey pod disruption budgets (PDB)

koordinator/pkg/koordlet/qosmanager/plugins/memoryevict/memory_evict.go

Line 163 in 661dd71

func (m *memoryEvictor) killAndEvictBEPods(node *corev1.Node, podMetrics map[string]float64, memoryNeedRelease int64) {

Is there a suggested solution, if so, please add it:

if eviction response OK, then we kill pod/container, otherwise skip. refer https://kubernetes.io/docs/concepts/scheduling-eviction/api-eviction/#how-api-initiated-eviction-works

There is a bonus with call eviction API first, we can disable eviction by update rolebinding.

[j4ckstraw]

reference https://github.com/kubernetes/kubernetes/blob/d61cbac69aae97db1839bd2e0e86d68f26b353a7/test/integration/evictions/evictions_test.go#L126C5-L127C13