Update event recorder name back to LinodeVPCReconciler and remove redundant logging in handleRetainedSubnets method for improved clarity and maintainability.

komer3 · komer3 · commit d68d6c204222 · 2025-07-10T11:55:47.000-05:00
diff --git a/cmd/main.go b/cmd/main.go
@@ -295,7 +295,7 @@ func setupControllers(mgr manager.Manager, flags flagVars, linodeClientConfig, d
 	// LinodeVPC Controller
 	if err := (&controller.LinodeVPCReconciler{
 		Client:             mgr.GetClient(),
-		Recorder:           mgr.GetEventRecorderFor("linodevpc-controller"),
+		Recorder:           mgr.GetEventRecorderFor("LinodeVPCReconciler"),
 		LinodeClientConfig: linodeClientConfig,
 		WatchFilterValue:   flags.clusterWatchFilter,
 	}).SetupWithManager(mgr, crcontroller.Options{MaxConcurrentReconciles: flags.linodeVPCConcurrency}); err != nil {
diff --git a/internal/controller/linodevpc_controller.go b/internal/controller/linodevpc_controller.go
@@ -374,7 +374,6 @@ func (r *LinodeVPCReconciler) handleRetainedSubnets(ctx context.Context, logger
 		return nil
 	}
 
-	logger.Info("subnet deletion enabled, checking subnets")
 	vpc, err := vpcScope.LinodeClient.GetVPC(ctx, *vpcScope.LinodeVPC.Spec.VPCID)
 	if err != nil {
 		if util.IgnoreLinodeAPIError(err, http.StatusNotFound) == nil {

Original file line number	Diff line number	Diff line change
`@@ -374,7 +374,6 @@ func (r *LinodeVPCReconciler) handleRetainedSubnets(ctx context.Context, logger`
`374`	`374`	`return nil`
`375`	`375`	`}`
`376`	`376`
`377`		`- logger.Info("subnet deletion enabled, checking subnets")`
`378`	`377`	`vpc, err := vpcScope.LinodeClient.GetVPC(ctx, *vpcScope.LinodeVPC.Spec.VPCID)`
`379`	`378`	`if err != nil {`
`380`	`379`	`if util.IgnoreLinodeAPIError(err, http.StatusNotFound) == nil {`