Remove condition setting on finalizer removal

Per Goncalves da Silva · Per Goncalves da Silva · commit 0b26ee536c55 · 2025-10-10T16:39:50.000+02:00
Signed-off-by: Per Goncalves da Silva &lt;pegoncal@redhat.com&gt;
diff --git a/internal/operator-controller/controllers/clusterextensionrevision_controller.go b/internal/operator-controller/controllers/clusterextensionrevision_controller.go
@@ -343,13 +343,6 @@ func (c *ClusterExtensionRevisionReconciler) teardown(ctx context.Context, rev *
 	}
 
 	if err := c.removeFinalizer(ctx, rev, clusterExtensionRevisionTeardownFinalizer); err != nil {
-		meta.SetStatusCondition(&rev.Status.Conditions, metav1.Condition{
-			Type:               ocv1.ClusterExtensionRevisionTypeAvailable,
-			Status:             metav1.ConditionFalse,
-			Reason:             ocv1.ClusterExtensionRevisionReasonReconcileFailure,
-			Message:            err.Error(),
-			ObservedGeneration: rev.Generation,
-		})
 		return ctrl.Result{}, fmt.Errorf("error removing teardown finalizer: %v", err)
 	}
 	return ctrl.Result{}, nil

Original file line number	Diff line number	Diff line change
`@@ -343,13 +343,6 @@ func (c ClusterExtensionRevisionReconciler) teardown(ctx context.Context, rev `
`343`	`343`	`}`
`344`	`344`
`345`	`345`	`if err := c.removeFinalizer(ctx, rev, clusterExtensionRevisionTeardownFinalizer); err != nil {`
`346`		`- meta.SetStatusCondition(&rev.Status.Conditions, metav1.Condition{`
`347`		`- Type: ocv1.ClusterExtensionRevisionTypeAvailable,`
`348`		`- Status: metav1.ConditionFalse,`
`349`		`- Reason: ocv1.ClusterExtensionRevisionReasonReconcileFailure,`
`350`		`- Message: err.Error(),`
`351`		`- ObservedGeneration: rev.Generation,`
`352`		`- })`
`353`	`346`	`return ctrl.Result{}, fmt.Errorf("error removing teardown finalizer: %v", err)`
`354`	`347`	`}`
`355`	`348`	`return ctrl.Result{}, nil`