Don't reconcile a failed deployment

Don't reconcile a failed deployment after the required number of
retries if there are changes in the nodeset or watched resources.

Author: rabi

controllers/dataplane/openstackdataplanedeployment_controller.go

@@ -109,6 +109,19 @@ func (r *OpenStackDataPlaneDeploymentReconciler) Reconcile(ctx context.Context,
 		return ctrl.Result{}, nil
 	}
 
+	// If the deployment has failed with backoff limit exceeded, do not reconcile
+	// even if nodesets or other watched resources change. The deployment is in a
+	// terminal failure state and should not be retried.
+	if instance.Status.Conditions != nil {
+		deploymentCondition := instance.Status.Conditions.Get(condition.DeploymentReadyCondition)
+		if deploymentCondition != nil &&
+			deploymentCondition.Reason == condition.JobReasonBackoffLimitExceeded {
+			Log.Info("Deployment has failed with backoff limit exceeded, skipping reconciliation",
+				"deployment", instance.Name)
+			return ctrl.Result{}, nil
+		}
+	}
+
 	// initialize status if Conditions is nil, but do not reset if it already
 	// exists
 	isNewInstance := instance.Status.Conditions == nil