operator v1: do not "lose" error in some situations

this code may hide an error returned by ar.Ensure. we don't want that;
the defer func around observedGeneration and OperatorQuiescent wants to
see all errors.

Author: birdayz

src/go/k8s/internal/controller/redpanda/cluster_controller.go

@@ -228,7 +228,7 @@ func (r *ClusterReconciler) Reconcile(
 	}
 
 	result, errs := ar.Ensure()
-	if !result.IsZero() {
+	if !result.IsZero() && errs == nil {
 		return result, nil
 	}
 	if errs != nil {