diff --git a/pkg/cluster-operator/cluster_controller.go b/pkg/cluster-operator/cluster_controller.go index c461de378..7dbe5e066 100644 --- a/pkg/cluster-operator/cluster_controller.go +++ b/pkg/cluster-operator/cluster_controller.go @@ -127,7 +127,6 @@ func (r *ClusterController) Reconcile(ctx context.Context, req ctrl.Request) (_ cluster.Status.Phase = string(capiv1.ClusterPhaseProvisioning) conditions.MarkFalse(cluster, clusterv1alpha1.ReadyCondition, clusterv1alpha1.ProvisioningReason, capiv1.ConditionSeverityError, "Provisioning") controllerutil.AddFinalizer(cluster, ClusterFinalizer) - return ctrl.Result{}, nil } // Handle deletion reconciliation loop. diff --git a/pkg/fleet-manager/application_controller.go b/pkg/fleet-manager/application_controller.go index 6c4f6154b..80e0f9cfa 100644 --- a/pkg/fleet-manager/application_controller.go +++ b/pkg/fleet-manager/application_controller.go @@ -134,7 +134,6 @@ func (a *ApplicationManager) Reconcile(ctx context.Context, req ctrl.Request) (_ // Add finalizer if not exist to void the race condition. if !controllerutil.ContainsFinalizer(app, ApplicationFinalizer) { controllerutil.AddFinalizer(app, ApplicationFinalizer) - return ctrl.Result{}, nil } // there only one fleet, so pre-fetch it here. diff --git a/pkg/fleet-manager/fleet.go b/pkg/fleet-manager/fleet.go index a39717767..a54493b7c 100644 --- a/pkg/fleet-manager/fleet.go +++ b/pkg/fleet-manager/fleet.go @@ -128,7 +128,6 @@ func (f *FleetManager) Reconcile(ctx context.Context, req ctrl.Request) (_ ctrl. if !controllerutil.ContainsFinalizer(fleet, FleetFinalizer) { fleet.Status.Phase = fleetapi.RunningPhase controllerutil.AddFinalizer(fleet, FleetFinalizer) - return ctrl.Result{}, nil } // Handle deletion reconciliation loop.