kubernetes-sigs / karpenter / 20048108203

80%

main: 80%

Build Type

Pull #2674

github

Committed by ellistarn

Commit Message

fix: NodeSelectorRequirements() now returns both bounds when Gte and Lte exist

Previously, when a Requirement had both upper and lower bounds (e.g., from
intersecting Gte and Lte on the same key), NodeSelectorRequirements() would
only return the Gte bound due to the switch statement short-circuiting.

This caused NodeClaims to be missing the Lte constraint from their parent
NodePool, making the NodeClaim's requirements broader than intended.

Added test case to verify both bounds are returned when present.

Pull Request Pull Request #2674: feat: Add Gte and Lte operators for requirements

Run Details

85 of 87 new or added lines in 7 files covered. (97.7%)

6 existing lines in 2 files now uncovered.

11889 of 14797 relevant lines covered (80.35%)

71006.51 hits per line