Merge pull request #18123 from HealthengineAU/fix-ir-bug

k8s-ci-robot · web-flow · commit e91d2585b615 · 2026-03-28T07:34:12.000+05:30
Fix instanceRequirements memory assignment bug
diff --git a/upup/pkg/fi/cloudup/awstasks/instancerequirements.go b/upup/pkg/fi/cloudup/awstasks/instancerequirements.go
@@ -48,7 +48,7 @@ func findInstanceRequirements(asg *autoscalingtypes.AutoScalingGroup) (*Instance
 				}
 				if override.InstanceRequirements.MemoryMiB != nil {
 					actual.MemoryMax = override.InstanceRequirements.MemoryMiB.Max
-					actual.MemoryMax = override.InstanceRequirements.MemoryMiB.Min
+					actual.MemoryMin = override.InstanceRequirements.MemoryMiB.Min
 				}
 				if len(override.InstanceRequirements.ExcludedInstanceTypes) > 0 {
 					actual.ExcludedInstanceTypes = override.InstanceRequirements.ExcludedInstanceTypes

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ func findInstanceRequirements(asg autoscalingtypes.AutoScalingGroup) (Instance`
`48`	`48`	`}`
`49`	`49`	`if override.InstanceRequirements.MemoryMiB != nil {`
`50`	`50`	`actual.MemoryMax = override.InstanceRequirements.MemoryMiB.Max`
`51`		`- actual.MemoryMax = override.InstanceRequirements.MemoryMiB.Min`
	`51`	`+ actual.MemoryMin = override.InstanceRequirements.MemoryMiB.Min`
`52`	`52`	`}`
`53`	`53`	`if len(override.InstanceRequirements.ExcludedInstanceTypes) > 0 {`
`54`	`54`	`actual.ExcludedInstanceTypes = override.InstanceRequirements.ExcludedInstanceTypes`