fix: don't set AMI ID if empty

If empty CAPA will fail with: failed to create AWSMachine instance: failed to run instance: MissingParameter: The request must contain the parameter ImageId
nutanix-cloud-native · Feb 9, 2024 · f486d5b · f486d5b
1 parent 0cc2133
commit f486d5b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/pkg/handlers/aws/mutation/ami/inject.go b/pkg/handlers/aws/mutation/ami/inject.go
@@ -88,7 +88,9 @@ func (h *awsAMISpecPatchHandler) Mutate(
 				"patchedObjectName", client.ObjectKeyFromObject(obj),
 			).Info("setting AMI in AWSMachineTemplate spec")
 
-			obj.Spec.Template.Spec.AMI = capav1.AMIReference{ID: &amiSpecVar.ID}
+			if amiSpecVar.ID != "" {
+				obj.Spec.Template.Spec.AMI = capav1.AMIReference{ID: &amiSpecVar.ID}
+			}
 			if amiSpecVar.Lookup != nil {
 				obj.Spec.Template.Spec.ImageLookupFormat = amiSpecVar.Lookup.Format
 				obj.Spec.Template.Spec.ImageLookupOrg = amiSpecVar.Lookup.Org