kubernetes-sigs / karpenter / 31914165413

83%

main: 83%

Build Type

Pull #3063

github

Committed by ketanjani21

Commit Message

address PR review: don't gate delete-eligible pods behind priority tiers

Reviewer feedback: the previous refactor gated force-delete behind
tier-ordered eviction, so a past-deadline pod in a later tier waited for
earlier tiers to drain. At the hard deadline we must not wait.

Split Drain into two passes:
- Delete-eligible pods (past the force-delete threshold) are enqueued
  across all tiers immediately.
- Graceful eviction candidates are tier-gated so noncritical/non-daemon
  pods drain before critical/daemon ones (k8s graceful shutdown ordering).

Convert needsForceDelete from a *Queue method to a package-level function
so both Drain and Queue.Reconcile share the same predicate.

Add a regression test that pairs a fits-in-deadline non-critical pod with
a past-deadline system-critical pod and verifies the critical pod's
deletion isn't gated by the earlier tier.

Pull Request Pull Request #3063: chore: Consolidate pod force-deletion into the eviction queue

Coverage Stats

85 of 98 new or added lines in 3 files covered. (86.73%)

6 existing lines in 1 file now uncovered.

17147 of 20672 relevant lines covered (82.95%)

95661.11 hits per line