nodereload controller fix

instaclustr · Nov 8, 2023 · 8f79c7e · 8f79c7e
1 parent 7a6135a
commit 8f79c7e
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/controllers/clusterresources/nodereload_controller.go b/controllers/clusterresources/nodereload_controller.go
@@ -74,9 +74,8 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 		return reconcile.Result{}, err
 	}
 
-	patch := nrs.NewPatch()
-
 	if len(nrs.Status.PendingNodes)+len(nrs.Status.CompletedNodes)+len(nrs.Status.FailedNodes) == 0 {
+		patch := nrs.NewPatch()
 		nrs.Status.PendingNodes = nrs.Spec.Nodes
 		err := r.Status().Patch(ctx, nrs, patch)
 		if err != nil {
@@ -120,6 +119,8 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 			return reconcile.Result{}, err
 		}
 
+		patch := nrs.NewPatch()
+
 		nrs.Status.NodeInProgress = nodeInProgress
 		if len(nrs.Status.PendingNodes) > 0 {
 			nrs.Status.PendingNodes = nrs.Status.PendingNodes[1:]
@@ -154,6 +155,8 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 		return reconcile.Result{}, err
 	}
 
+	patch := nrs.NewPatch()
+
 	nrs.Status.CurrentOperationStatus = &v1beta1.Operation{
 		OperationID:  nodeReloadStatus.OperationID,
 		TimeCreated:  nodeReloadStatus.TimeCreated,
@@ -177,7 +180,7 @@ func (r *NodeReloadReconciler) Reconcile(ctx context.Context, req ctrl.Request)
 
 	if nodeReloadStatus.Status != nodeReloadOperationStatusCompleted {
 		l.Info("Node Reload operation is not completed yet, please wait a few minutes",
-			"nodeID", nrs.Status.NodeInProgress,
+			"node ID", nrs.Status.NodeInProgress,
 			"status", nrs.Status,
 		)