Apply suggestions from code review

Co-authored-by: Aitor Pérez Cedres <1515757+Zerpet@users.noreply.github.com>

Author: jmealo

controllers/reconcile_persistence.go

@@ -16,8 +16,7 @@ func (r *RabbitmqClusterReconciler) reconcilePVC(ctx context.Context, rmq *rabbi
 	logger := ctrl.LoggerFrom(ctx)
 	desiredCapacity, err := persistenceStorageCapacity(desiredSts.Spec.VolumeClaimTemplates)
 	if err != nil {
-		msg := fmt.Sprintf("Failed to determine PVC capacity: %s", err.Error())
-		logger.Error(err, msg)
+		logger.Error(err, "failed to determine PVC capacity")
 		r.Recorder.Event(rmq, corev1.EventTypeWarning, "FailedReconcilePersistence", msg)
 		return err
 	}
@@ -36,10 +35,8 @@ func persistenceStorageCapacity(templates []corev1.PersistentVolumeClaim) (k8sre
 			storage := t.Spec.Resources.Requests[corev1.ResourceStorage]
 			if storage.IsZero() {
 				return storage, fmt.Errorf(
-					"PVC template 'persistence' has spec.resources.requests.storage=0 (or missing). " +
-						"If using override.statefulSet.spec.volumeClaimTemplates, you must provide " +
-						"the COMPLETE template including spec.resources.requests.storage. " +
-						"Overrides replace the entire volumeClaimTemplate, not merge with it")
+					"PVC template 'persistence' is missing the storage request. " +
+						"If you are overriding the persistence template, please provide the full template definition including storage request and metadata")
 			}
 			return storage, nil
 		}