From 6ab56abd0a1b06e11a6d594122ce1f8e7474eec8 Mon Sep 17 00:00:00 2001 From: michaeljguarino Date: Mon, 19 Feb 2024 02:55:44 -0500 Subject: [PATCH] Reduce some unnecessary logging (#125) Some debug messages are being emitted that don't necessarily need to be in the production build of the agent. --- pkg/controller/pipelinegates/reconciler.go | 2 +- pkg/controller/service/reconciler.go | 2 +- pkg/controller/service/reconciler_status.go | 10 +++++----- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/pkg/controller/pipelinegates/reconciler.go b/pkg/controller/pipelinegates/reconciler.go index ac4032e2..8a4d7d1b 100644 --- a/pkg/controller/pipelinegates/reconciler.go +++ b/pkg/controller/pipelinegates/reconciler.go @@ -112,7 +112,7 @@ func (s *GateReconciler) Poll(ctx context.Context) (done bool, err error) { after = resp.PagedClusterGates.PageInfo.EndCursor for _, gate := range resp.PagedClusterGates.Edges { - logger.Info("sending update for", "gate", gate.Node.ID) + logger.V(2).Info("sending update for", "gate", gate.Node.ID) s.GateQueue.Add(gate.Node.ID) } } diff --git a/pkg/controller/service/reconciler.go b/pkg/controller/service/reconciler.go index ae04d3f6..7bcefc79 100644 --- a/pkg/controller/service/reconciler.go +++ b/pkg/controller/service/reconciler.go @@ -225,7 +225,7 @@ func (s *ServiceReconciler) Reconcile(ctx context.Context, id string) (result re } }() - logger.Info("local", "flag", Local) + logger.V(2).Info("local", "flag", Local) if Local && svc.Name == OperatorService { return } diff --git a/pkg/controller/service/reconciler_status.go b/pkg/controller/service/reconciler_status.go index a978bc75..dff51bae 100644 --- a/pkg/controller/service/reconciler_status.go +++ b/pkg/controller/service/reconciler_status.go @@ -264,15 +264,15 @@ func FormatActionGroupEvent(ctx context.Context, age event.ActionGroupEvent) err switch age.Action { case event.ApplyAction: - logger.Info("apply phase", "status", strings.ToLower(age.Status.String())) + logger.V(2).Info("apply phase", "status", strings.ToLower(age.Status.String())) case event.PruneAction: - logger.Info("prune phase ", "status", strings.ToLower(age.Status.String())) + logger.V(2).Info("prune phase ", "status", strings.ToLower(age.Status.String())) case event.DeleteAction: - logger.Info("delete phase", "status", strings.ToLower(age.Status.String())) + logger.V(2).Info("delete phase", "status", strings.ToLower(age.Status.String())) case event.WaitAction: - logger.Info("reconcile phase", "status", strings.ToLower(age.Status.String())) + logger.V(2).Info("reconcile phase", "status", strings.ToLower(age.Status.String())) case event.InventoryAction: - logger.Info("inventory update", "status", strings.ToLower(age.Status.String())) + logger.V(2).Info("inventory update", "status", strings.ToLower(age.Status.String())) default: return fmt.Errorf("invalid action group action: %+v", age) }