kubernetes-sigs / karpenter / 22591269383

80%

main: 80%

Pull #2882

github

fix: address PR review comments for sequential topology disruption budget

- Use c.Labels()[budget.TopologyKey] instead of hardcoded c.zone so
  sequential topology filtering works for any topology key, not just zones
- Make pickActiveDomain deterministic by sorting domain keys before
  selecting the one with in-flight disruptions
- Replace per-NodePool scanNodesByZone (O(NodePools×nodes)) with a single
  buildTopologyIndex pass (O(nodes+NodePools)) over all cluster nodes
- Only skip topology budgets in GetAllowedDisruptionsByReason for the
  Drifted reason; other methods (consolidation, expiration) now apply
  topology budgets as regular NodePool-level budgets
- Update budget test to assert MaxInt32 for Drifted and the configured
  value for other reasons when only topology budgets exist

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

108 of 116 new or added lines in 4 files covered. (93.1%)

12180 of 15154 relevant lines covered (80.37%)

120517.66 hits per line