🌱Add condition indicating deletion timestamp is set

🌱Fix lint errors

🌱Fix gofmt lint error

🐛Fix condition

🌱Remove Deletion condition

🐛Fix condition errors

🌱Remove condition and split if statements

🐛Fix lint error and remove needsUpdate

🌱Remove if statement

api/v1beta1/conditions_const.go

@@ -200,6 +200,11 @@ const (
 	HostAssociateFailedReason = "HostAssociateFailed"
 )
 
+const (
+	// DeletionInProgressReason indicates that a host is being deleted.
+	DeletionInProgressReason = "DeletionInProgress"
+)
+
 // deprecated conditions.
 
 const (

controllers/hetznerbaremetalhost_controller.go

@@ -180,19 +180,25 @@ func (r *HetznerBareMetalHostReconciler) reconcileSelectedStates(ctx context.Con
 	switch bmHost.Spec.Status.ProvisioningState {
 	// Handle StateNone: check whether needs to be provisioned or deleted.
 	case infrav1.StateNone:
-		var needsUpdate bool
-		if !bmHost.DeletionTimestamp.IsZero() && bmHost.Spec.ConsumerRef == nil {
-			bmHost.Spec.Status.ProvisioningState = infrav1.StateDeleting
-			needsUpdate = true
+		if !bmHost.DeletionTimestamp.IsZero() {
+			conditions.MarkFalse(
+				bmHost,
+				infrav1.HostReadyCondition,
+				infrav1.DeletionInProgressReason,
+				clusterv1.ConditionSeverityInfo,
+				"Host is not ready because it is being deleted",
+			)
+			conditions.SetSummary(bmHost)
+
+			if bmHost.Spec.ConsumerRef == nil {
+				bmHost.Spec.Status.ProvisioningState = infrav1.StateDeleting
+			}
 		} else if bmHost.NeedsProvisioning() {
 			bmHost.Spec.Status.ProvisioningState = infrav1.StatePreparing
-			needsUpdate = true
 		}
-		if needsUpdate {
-			err := r.Update(ctx, bmHost)
-			if err != nil {
-				return reconcile.Result{}, fmt.Errorf("failed to add finalizer: %w", err)
-			}
+
+		if err := r.Update(ctx, bmHost); err != nil {
+			return reconcile.Result{}, fmt.Errorf("failed to update host: %w", err)
 		}
 
 		return ctrl.Result{RequeueAfter: 10 * time.Second}, nil