From 51565fb88a16738c1053e5e1e47ce5c3979a3b74 Mon Sep 17 00:00:00 2001 From: Wes Date: Wed, 7 Feb 2024 10:55:28 -0700 Subject: [PATCH] fix: reduce controller log warnings (#890) Instead of removing these log messages, I lowered them to `Debug` in case they are useful in the future. I can remove if that's preferred as well. --- backend/controller/controller.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/backend/controller/controller.go b/backend/controller/controller.go index 23f31ce52b..8e450c388c 100644 --- a/backend/controller/controller.go +++ b/backend/controller/controller.go @@ -818,7 +818,7 @@ func (s *Service) reapStaleRunners(ctx context.Context) (time.Duration, error) { if err != nil { return 0, fmt.Errorf("%s: %w", "Failed to delete stale runners", err) } else if count > 0 { - logger.Warnf("Reaped %d stale runners", count) + logger.Debugf("Reaped %d stale runners", count) } return s.config.RunnerTimeout, nil } @@ -857,7 +857,7 @@ func (s *Service) reconcileDeployments(ctx context.Context) (time.Duration, erro deploymentLogger.Debugf("Need %d more runners for %s", require, reconcile.Deployment) wg.Go(func(ctx context.Context) error { if err := s.deploy(ctx, deployment); err != nil { - deploymentLogger.Warnf("Failed to increase deployment replicas: %s", err) + deploymentLogger.Debugf("Failed to increase deployment replicas: %s", err) } else { deploymentLogger.Debugf("Reconciled %s", reconcile.Deployment) } @@ -973,7 +973,7 @@ func (s *Service) reapStaleControllers(ctx context.Context) (time.Duration, erro if err != nil { return 0, fmt.Errorf("%s: %w", "failed to delete stale controllers", err) } else if count > 0 { - logger.Warnf("Reaped %d stale controllers", count) + logger.Debugf("Reaped %d stale controllers", count) } return time.Second * 10, nil }