From 8aa8a77881f29e14a4f0c4ae3875de06f56918f1 Mon Sep 17 00:00:00 2001 From: Apoorv Mahajan Date: Tue, 24 Aug 2021 11:12:30 -0700 Subject: [PATCH] refactor(front50): suppress noisy logs in DependentPipelineStarter --- .../orca/front50/DependentPipelineStarter.groovy | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/orca-front50/src/main/groovy/com/netflix/spinnaker/orca/front50/DependentPipelineStarter.groovy b/orca-front50/src/main/groovy/com/netflix/spinnaker/orca/front50/DependentPipelineStarter.groovy index e68ae60efc..9e711344fc 100644 --- a/orca-front50/src/main/groovy/com/netflix/spinnaker/orca/front50/DependentPipelineStarter.groovy +++ b/orca-front50/src/main/groovy/com/netflix/spinnaker/orca/front50/DependentPipelineStarter.groovy @@ -76,8 +76,9 @@ class DependentPipelineStarter implements ApplicationContextAware { throw new ConfigurationException("Pipeline '${pipelineConfig.name}' is disabled and cannot be triggered") } - if (log.isInfoEnabled()) { - log.info('triggering dependent pipeline {}:{}', pipelineConfig.id, + log.info("triggering dependent pipeline ${pipelineConfig.id}") + if (log.isDebugEnabled()) { + log.debug('triggering dependent pipeline {}:{}', pipelineConfig.id, objectMapper.writeValueAsString(pipelineConfig)) } @@ -180,8 +181,9 @@ class DependentPipelineStarter implements ApplicationContextAware { def augmentedContext = [trigger: pipelineConfig.trigger] def processedPipeline = contextParameterProcessor.processPipeline(pipelineConfig, augmentedContext, false) - if (log.isInfoEnabled()) { - log.info('running pipeline {}:{}', pipelineConfig.id, objectMapper.writeValueAsString(processedPipeline)) + log.info("running pipeline ${pipelineConfig.id}") + if (log.isDebugEnabled()) { + log.debug('running pipeline {}:{}', pipelineConfig.id, objectMapper.writeValueAsString(processedPipeline)) } Callable callable