autoscaler icon indicating copy to clipboard operation
autoscaler copied to clipboard

Change eviction machinery

Open RomanenkoDenys opened this issue 11 months ago • 3 comments

What type of PR is this?

/kind feature

What this PR does / why we need it:

Currently, when vertical-pod-autoscaler counts pods that can be evicted, VPA cannot keep track of the status of the pods. This can lead to some problems when pods are not ready yet, but replicas are still being evicted.

To prevent this, we can use PDB, but PDB is not available for DaemonSets.

This PR proposes a different scheme where we only consider ready-pods to eviction.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?


Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:


RomanenkoDenys avatar Mar 04 '24 09:03 RomanenkoDenys

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: RomanenkoDenys Once this PR has been reviewed and has the lgtm label, please assign krzysied for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Mar 04 '24 09:03 k8s-ci-robot