Implement review

Author: simcod

internal/controller/metalstackcluster_controller.go

@@ -123,7 +123,7 @@ func (r *MetalStackClusterReconciler) Reconcile(ctx context.Context, req ctrl.Re
 		conditions.Set(infraCluster, metav1.Condition{
 			Status: metav1.ConditionFalse,
 			Type:   clusterv1.PausedCondition,
-			Reason: clusterv1.PausedReason,
+			Reason: clusterv1.NotPausedReason,
 		})
 	}
 

internal/controller/metalstackcluster_controller_test.go

@@ -233,7 +233,7 @@ var _ = Describe("MetalStackCluster Controller", func() {
 			Expect(resource.Status.Conditions).To(ContainElement(MatchFields(IgnoreExtras, Fields{
 				"Type":   Equal(clusterv1beta2.PausedCondition),
 				"Status": Equal(metav1.ConditionFalse),
-				"Reason": Equal(clusterv1beta2.PausedReason),
+				"Reason": Equal(clusterv1beta2.NotPausedReason),
 			})))
 		})
 
@@ -350,7 +350,7 @@ var _ = Describe("MetalStackCluster Controller", func() {
 			Expect(resource.Status.Conditions).To(ContainElement(MatchFields(IgnoreExtras, Fields{
 				"Type":   Equal(clusterv1beta2.PausedCondition),
 				"Status": Equal(metav1.ConditionFalse),
-				"Reason": Equal(clusterv1beta2.PausedReason),
+				"Reason": Equal(clusterv1beta2.NotPausedReason),
 			})))
 		})
 

internal/controller/metalstackmachine_controller.go

@@ -146,12 +146,13 @@ func (r *MetalStackMachineReconciler) Reconcile(ctx context.Context, req ctrl.Re
 		conditions.Set(infraMachine, metav1.Condition{
 			Status: metav1.ConditionTrue,
 			Type:   clusterv1.PausedCondition,
+			Reason: clusterv1.PausedReason,
 		})
 	} else {
 		conditions.Set(infraMachine, metav1.Condition{
 			Status: metav1.ConditionFalse,
 			Type:   clusterv1.PausedCondition,
-			Reason: clusterv1.PausedReason,
+			Reason: clusterv1.NotPausedReason,
 		})
 	}