Regression caused from previous commit

Author: sriram-30

internal/controllers/upgrademgr.go

@@ -151,11 +151,6 @@ func (n *upgradeMgr) HandleUpgrade(ctx context.Context, deviceConfig *amdv1alpha
 		// 1. Set init status for unprocessed nodes
 		n.helper.handleInitStatus(ctx, &nodeList.Items[i])
 
-		if !n.helper.isNodeReadyForUpgrade(ctx, &nodeList.Items[i]) {
-			res = ctrl.Result{Requeue: true, RequeueAfter: time.Second * 20}
-			continue
-		}
-
 		// 2. Handle failed nodes
 		if n.helper.isNodeStateUpgradeFailed(ctx, &nodeList.Items[i], deviceConfig) {
 			n.helper.clearUpgradeStartTime(nodeList.Items[i].Name)
@@ -193,6 +188,11 @@ func (n *upgradeMgr) HandleUpgrade(ctx context.Context, deviceConfig *amdv1alpha
 			continue
 		}
 
+		if !n.helper.isNodeReadyForUpgrade(ctx, &nodeList.Items[i]) {
+			res = ctrl.Result{Requeue: true, RequeueAfter: time.Second * 20}
+			continue
+		}
+
 		//This node is a candidate for selection
 		candidateNodes = append(candidateNodes, nodeList.Items[i])
 	}