Initial attemtp. Testing now.

Joshua Reed · Joshua Reed · commit 278617932972 · 2022-06-23T10:12:10.000-07:00
diff --git a/controllers/cloudstackmachine_controller.go b/controllers/cloudstackmachine_controller.go
@@ -148,6 +148,7 @@ func (r *CloudStackMachineReconciliationRunner) SetFailureDomainOnCSMachine() (r
 				return ctrl.Result{}, errors.Errorf("could not find zone by zoneName: %s", r.ReconciliationSubject.Spec.ZoneName)
 			}
 		} else { // No Zone Specified, pick a Random Zone.
+			// TODO: Add an additional controller test for this once controller test upgrades are merged.
 			if len(r.Zones.Items) < 1 { // Double check that zones are present.
 				return r.RequeueWithMessage("no zones found, requeueing")
 			}

Original file line number	Diff line number	Diff line change
`@@ -148,6 +148,7 @@ func (r *CloudStackMachineReconciliationRunner) SetFailureDomainOnCSMachine() (r`
`148`	`148`	`return ctrl.Result{}, errors.Errorf("could not find zone by zoneName: %s", r.ReconciliationSubject.Spec.ZoneName)`
`149`	`149`	`}`
`150`	`150`	`} else { // No Zone Specified, pick a Random Zone.`
	`151`	`+ // TODO: Add an additional controller test for this once controller test upgrades are merged.`
`151`	`152`	`if len(r.Zones.Items) < 1 { // Double check that zones are present.`
`152`	`153`	`return r.RequeueWithMessage("no zones found, requeueing")`
`153`	`154`	`}`