add pause condition back into lc and lm, fix tests

Author: scmeggitt

internal/controller/linodecluster_controller.go

@@ -113,6 +113,11 @@ func (r *LinodeClusterReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 		return ctrl.Result{}, fmt.Errorf("failed to create cluster scope: %w", err)
 	}
 
+	isPaused, conditionChanged, err := paused.EnsurePausedCondition(ctx, clusterScope.Client, clusterScope.Cluster, clusterScope.LinodeCluster)
+	if err != nil {
+		return err
+	}
+
 	if reconciler.IsPaused(cluster, linodeCluster) {
 		logger.Info("linodeCluster or linked cluster is marked as paused, won't reconcile.")
 		return ctrl.Result{}, nil

internal/controller/linodecluster_controller_test.go

@@ -182,7 +182,7 @@ var _ = Describe("cluster-lifecycle", Ordered, Label("cluster", "cluster-lifecyc
 					// Set VPC as ready
 					linodeVPC.Status.Ready = true
 					linodeVPC.Labels = map[string]string{
-						clusterv1.ClusterNameLabel: cScope.LinodeCluster.Name
+						clusterv1.ClusterNameLabel: cScope.LinodeCluster.Name,
 					}
 					k8sClient.Status().Update(ctx, &linodeVPC)
 
@@ -451,7 +451,7 @@ var _ = Describe("cluster-lifecycle-dns", Ordered, Label("cluster", "cluster-lif
 					clusterKey := client.ObjectKeyFromObject(&linodeCluster)
 					Expect(k8sClient.Get(ctx, clusterKey, &linodeCluster)).To(Succeed())
 					Expect(linodeCluster.Status.Ready).To(BeTrue())
-					Expect(linodeCluster.Status.Conditions).To(HaveLen(1))
+					Expect(linodeCluster.Status.Conditions).To(HaveLen(2))
 					readyCond := conditions.Get(&linodeCluster, string(clusterv1.ReadyCondition))
 					Expect(readyCond).NotTo(BeNil())
 
@@ -725,7 +725,7 @@ var _ = Describe("dns-override-endpoint", Ordered, Label("cluster", "dns-overrid
 					clusterKey := client.ObjectKeyFromObject(&linodeCluster)
 					Expect(k8sClient.Get(ctx, clusterKey, &linodeCluster)).To(Succeed())
 					Expect(linodeCluster.Status.Ready).To(BeTrue())
-					Expect(linodeCluster.Status.Conditions).To(HaveLen(1))
+					Expect(linodeCluster.Status.Conditions).To(HaveLen(2))
 					cond := conditions.Get(&linodeCluster, string(clusterv1.ReadyCondition))
 					Expect(cond).NotTo(BeNil())
 

internal/controller/linodemachine_controller.go

@@ -166,6 +166,11 @@ func (r *LinodeMachineReconciler) Reconcile(ctx context.Context, req ctrl.Reques
 		return ctrl.Result{}, fmt.Errorf("failed to create machine scope: %w", err)
 	}
 
+	isPaused, conditionChanged, err := paused.EnsurePausedCondition(ctx, machineScope.Client, machineScope.Cluster, machineScope.LinodeMachine)
+	if err != nil {
+		return err
+	}
+
 	// Stop if paused
 	if reconciler.IsPaused(machineScope.Cluster, machineScope.Machine) {
 		log.Info("LinodeMachine or linked cluster is marked as paused, won't reconcile.")