moko-poi / karpenter / 22945548761

80%

main: 82%

Pull #1

github

fix: skip ephemeral volume topology requirements for bound pods during consolidation

Ephemeral volume PVCs are owned by the pod and deleted when the pod is
deleted. When a pod is rescheduled during consolidation, a new PVC will
be created, so the old PVC's zone constraint should not constrain the
new scheduling decision.

Previously, Karpenter would extract zone constraints from ephemeral
volume PVCs even for bound pods being evaluated for consolidation. This
caused the nodeDomains to be narrowed to a single zone, making topology
spread constraints unsatisfiable and blocking consolidation with
"Unconsolidatable" events.

This fix adds an early return in VolumeTopology.getRequirements() to
skip ephemeral volumes when the pod is already bound (pod.Spec.NodeName
is set), while preserving the existing behavior for regular PVCs and
unbound pods.

Fixes: kubernetes-sigs/karpenter#2803

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

7 of 7 new or added lines in 1 file covered. (100.0%)

12087 of 15054 relevant lines covered (80.29%)

121312.5 hits per line