Adjust Host Lease phases so that it is not Ready until after its template is complete

Author: DanNiESh

internal/controller/hostlease_controller.go

@@ -116,15 +116,13 @@ func (r *HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease *v1alp
 		return ctrl.Result{}, nil
 	}
 
-	if hostLease.Status.Phase == "" {
-		hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
-	}
-
 	if !controllerutil.ContainsFinalizer(hostLease, hostLeaseFinalizer) {
 		controllerutil.AddFinalizer(hostLease, hostLeaseFinalizer)
 		if err := r.Update(ctx, hostLease); err != nil {
+			hostLease.Status.Phase = v1alpha1.HostLeasePhaseFailed
 			return ctrl.Result{}, err
 		}
+		hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
 		return ctrl.Result{Requeue: true}, nil
 	}
 
@@ -136,6 +134,12 @@ func (r *HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease *v1alp
 			return result, provErr
 		}
 		if !result.IsZero() {
+			if hostLease.Status.Phase != v1alpha1.HostLeasePhaseFailed {
+				hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
+			}
+			return result, nil
+		}
+		if hostLease.Status.Phase == v1alpha1.HostLeasePhaseFailed {
 			return result, nil
 		}
 	}
@@ -167,6 +171,7 @@ func (r *HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease *v1alp
 	if hostLease.Spec.PoweredOn != nil {
 		currentlyOn := node.PowerState == ironic.PowerOn.String()
 		if *hostLease.Spec.PoweredOn != currentlyOn {
+			hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
 			return ctrl.Result{RequeueAfter: r.RecheckInterval}, nil
 		}
 	}