Merge pull request #5514 from yaroslava-serdiuk/min-size-fix

Fix RemovableAt()
kubernetes · Feb 16, 2023 · 2f1c895 · 2f1c895
2 parents 19487b0 + 6b9d55b
commit 2f1c895
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/cluster-autoscaler/core/scaledown/unneeded/nodes.go b/cluster-autoscaler/core/scaledown/unneeded/nodes.go
@@ -185,7 +185,6 @@ func (n *Nodes) unremovableReason(context *context.AutoscalingContext, v *node,
 	if reason := verifyMinSize(node.Name, nodeGroup, nodeGroupSize, as); reason != simulator.NoReason {
 		return reason
 	}
-	nodeGroupSize[nodeGroup.Id()]--
 
 	resourceDelta, err := n.limitsFinder.DeltaForNode(context, node, nodeGroup, resourcesWithLimits)
 	if err != nil {
@@ -209,6 +208,7 @@ func (n *Nodes) unremovableReason(context *context.AutoscalingContext, v *node,
 		return simulator.MinimalResourceLimitExceeded
 	}
 
+	nodeGroupSize[nodeGroup.Id()]--
 	return simulator.NoReason
 }