fix machine reconcilation bug

Signed-off-by: Prajyot-Parab <[email protected]>

api/v1beta2/conditions_consts.go

@@ -36,8 +36,11 @@ const (
 	// InstanceErroredReason instance is in a errored state.
 	InstanceErroredReason = "InstanceErrored"
 
-	// InstanceNotReadyReason used when the instance is in a pending state.
+	// InstanceNotReadyReason used when the instance is in a not ready state.
 	InstanceNotReadyReason = "InstanceNotReady"
+
+	// InstanceStateUnknownReason used when the instance is in a unknown state.
+	InstanceStateUnknownReason = "InstanceStateUnknown"
 )
 
 const (

cloud/scope/powervs_machine.go

@@ -201,6 +201,14 @@ func (m *PowerVSMachineScope) CreateMachine() (*models.PVMInstanceReference, err
 	} else if instanceReply != nil {
 		// TODO need a reasonable wrapped error.
 		return instanceReply, nil
+	} else if instanceReply == nil {
+		// Check if create request has been already triggered.
+		// If InstanceReadyCondition is Unknown then return and wait for it to get updated.
+		for _, con := range m.IBMPowerVSMachine.Status.Conditions {
+			if con.Type == infrav1beta2.InstanceReadyCondition && con.Status == corev1.ConditionUnknown {
+				return nil, nil
+			}
+		}
 	}
 	cloudInitData, err := m.GetBootstrapData()
 	if err != nil {

controllers/ibmpowervsmachine_controller.go

@@ -262,6 +262,9 @@ func (r *IBMPowerVSMachineReconciler) reconcileNormal(machineScope *scope.PowerV
 			machineScope.Info("PowerVS instance state is undefined", "state", *instance.Status, "instance-id", machineScope.GetInstanceID())
 			conditions.MarkUnknown(machineScope.IBMPowerVSMachine, infrav1beta2.InstanceReadyCondition, "", "")
 		}
+	} else {
+		machineScope.SetNotReady()
+		conditions.MarkUnknown(machineScope.IBMPowerVSMachine, infrav1beta2.InstanceReadyCondition, infrav1beta2.InstanceStateUnknownReason, "")
 	}
 
 	// Requeue after 2 minute if machine is not ready to update status of the machine properly.