Include all unregistered nodes in node group acceptable range

BigDarkClown · BigDarkClown · commit 624d74c000a9 · 2023-06-27T14:23:02.000Z
diff --git a/cluster-autoscaler/clusterstate/clusterstate.go b/cluster-autoscaler/clusterstate/clusterstate.go
@@ -512,7 +512,7 @@ func (csr *ClusterStateRegistry) updateAcceptableRanges(targetSize map[string]in
 		size := targetSize[nodeGroup.Id()]
 		readiness := csr.perNodeGroupReadiness[nodeGroup.Id()]
 		result[nodeGroup.Id()] = AcceptableRange{
-			MinNodes:      size - len(readiness.LongUnregistered),
+			MinNodes:      size - len(readiness.Unregistered) - len(readiness.LongUnregistered),
 			MaxNodes:      size,
 			CurrentTarget: size,
 		}

Original file line number	Diff line number	Diff line change
`@@ -512,7 +512,7 @@ func (csr *ClusterStateRegistry) updateAcceptableRanges(targetSize map[string]in`
`512`	`512`	`size := targetSize[nodeGroup.Id()]`
`513`	`513`	`readiness := csr.perNodeGroupReadiness[nodeGroup.Id()]`
`514`	`514`	`result[nodeGroup.Id()] = AcceptableRange{`
`515`		`- MinNodes: size - len(readiness.LongUnregistered),`
	`515`	`+ MinNodes: size - len(readiness.Unregistered) - len(readiness.LongUnregistered),`
`516`	`516`	`MaxNodes: size,`
`517`	`517`	`CurrentTarget: size,`
`518`	`518`	`}`