Merge pull request #217 from gibizer/remove-double-update

TransportURL: rely on PatchInstance instead of manual Update
openstack-k8s-operators · Apr 26, 2024 · ba41f87 · ba41f87
2 parents 3266115 + 77fda22
commit ba41f87
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/controllers/rabbitmq/transporturl_controller.go b/controllers/rabbitmq/transporturl_controller.go
@@ -150,10 +150,8 @@ func (r *TransportURLReconciler) Reconcile(ctx context.Context, req ctrl.Request
 	instance.Status.ObservedGeneration = instance.Generation
 
 	if isNewInstance {
-		// Register overall status immediately to have an early feedback e.g. in the cli
-		if err := r.Status().Update(ctx, instance); err != nil {
-			return ctrl.Result{}, err
-		}
+		// Return to register overall status immediately to have an early feedback e.g. in the cli
+		return ctrl.Result{}, nil
 	}
 
 	return r.reconcileNormal(ctx, instance, helper)