Fix instanceRequirements memory assignment bug

jim-barber-he · jim-barber-he · commit c515950bfdd0 · 2026-03-28T08:38:28.000+08:00
The instanceRequirements memory maximum value ends up being assigned the
minimum value specified by the user, while eh minimum value doesn't get
assigned at all.
This is now fixed.
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`