Merge pull request #399 from Xieql/remove-patch-option

cleanup: remove unused patch option
kurator-dev · Oct 17, 2023 · a82ec3e · a82ec3e
2 parents c0e0844 + 3822871
commit a82ec3e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 6 deletions.
diff --git a/pkg/cluster-operator/attachedcluster_controller.go b/pkg/cluster-operator/attachedcluster_controller.go
@@ -74,8 +74,7 @@ func (a *AttachedClusterController) Reconcile(ctx context.Context, req ctrl.Requ
 	}
 
 	defer func() {
-		patchOpts := []patch.Option{}
-		if err := patchHelper.Patch(ctx, attachedCluster, patchOpts...); err != nil {
+		if err := patchHelper.Patch(ctx, attachedCluster); err != nil {
 			reterr = utilerrors.NewAggregate([]error{reterr, errors.Wrapf(err, "failed to patch fleet %s", req.NamespacedName)})
 		}
 	}()

diff --git a/pkg/fleet-manager/application_controller.go b/pkg/fleet-manager/application_controller.go
@@ -125,8 +125,7 @@ func (a *ApplicationManager) Reconcile(ctx context.Context, req ctrl.Request) (_
 	}
 
 	defer func() {
-		patchOpts := []patch.Option{}
-		if err := patchHelper.Patch(ctx, app, patchOpts...); err != nil {
+		if err := patchHelper.Patch(ctx, app); err != nil {
 			reterr = utilerrors.NewAggregate([]error{reterr, errors.Wrapf(err, "failed to patch application %s", req.NamespacedName)})
 		}
 	}()

diff --git a/pkg/fleet-manager/fleet.go b/pkg/fleet-manager/fleet.go
@@ -118,8 +118,7 @@ func (f *FleetManager) Reconcile(ctx context.Context, req ctrl.Request) (_ ctrl.
 	}
 
 	defer func() {
-		patchOpts := []patch.Option{}
-		if err := patchHelper.Patch(ctx, fleet, patchOpts...); err != nil {
+		if err := patchHelper.Patch(ctx, fleet); err != nil {
 			reterr = utilerrors.NewAggregate([]error{reterr, errors.Wrapf(err, "failed to patch fleet %s", req.NamespacedName)})
 		}
 	}()