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

DanNiESh · DanNiESh · commit f22af09493c0 · 2026-06-03T14:44:53.000-04:00
diff --git a/internal/controller/hostlease_controller.go b/internal/controller/hostlease_controller.go
@@ -116,9 +116,7 @@ func (r *HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease *v1alp
 		return ctrl.Result{}, nil
 	}
 
-	if hostLease.Status.Phase == "" {
-		hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
-	}
+	hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing
 
 	if !controllerutil.ContainsFinalizer(hostLease, hostLeaseFinalizer) {
 		controllerutil.AddFinalizer(hostLease, hostLeaseFinalizer)
@@ -138,6 +136,9 @@ func (r *HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease *v1alp
 		if !result.IsZero() {
 			return result, nil
 		}
+		if hostLease.Status.Phase == v1alpha1.HostLeasePhaseFailed {
+			return result, nil
+		}
 	}
 
 	node, err := r.IronicClient.GetNode(ctx, hostLease.Spec.ExternalHostID)

Original file line number	Diff line number	Diff line change
`@@ -116,9 +116,7 @@ func (r HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease v1alp`
`116`	`116`	`return ctrl.Result{}, nil`
`117`	`117`	`}`
`118`	`118`
`119`		`- if hostLease.Status.Phase == "" {`
`120`		`- hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing`
`121`		`- }`
	`119`	`+ hostLease.Status.Phase = v1alpha1.HostLeasePhaseProgressing`
`122`	`120`
`123`	`121`	`if !controllerutil.ContainsFinalizer(hostLease, hostLeaseFinalizer) {`
`124`	`122`	`controllerutil.AddFinalizer(hostLease, hostLeaseFinalizer)`
`@@ -138,6 +136,9 @@ func (r HostLeaseReconciler) handleUpdate(ctx context.Context, hostLease v1alp`
`138`	`136`	`if !result.IsZero() {`
`139`	`137`	`return result, nil`
`140`	`138`	`}`
	`139`	`+ if hostLease.Status.Phase == v1alpha1.HostLeasePhaseFailed {`
	`140`	`+ return result, nil`
	`141`	`+ }`
`141`	`142`	`}`
`142`	`143`
`143`	`144`	`node, err := r.IronicClient.GetNode(ctx, hostLease.Spec.ExternalHostID)`