From 6a2250bfd4ff33d95c9d3008dc7b89b7781fce42 Mon Sep 17 00:00:00 2001 From: Piotr Konopka Date: Thu, 12 Dec 2024 18:32:52 +0100 Subject: [PATCH] OCTRL-759 [core] clearer deployment failure logs for OPS - shortened one phrase and demoted the log to support, as a similar log is always produced for OPS anyway - removed redundant prefix "deployment error", which IMHO does not add anything to error message which follows (e.g "deployment error error=workflow deployment failed..." becomes "workflow deployment failed...") --- core/environment/transition_deploy.go | 4 ++-- core/task/manager.go | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/core/environment/transition_deploy.go b/core/environment/transition_deploy.go index 652b5325..ed7e6e45 100644 --- a/core/environment/transition_deploy.go +++ b/core/environment/transition_deploy.go @@ -342,9 +342,9 @@ func (t DeployTransition) do(env *Environment) (err error) { } if err != nil { - log.WithError(err). + log.WithField("level", infologger.IL_Ops). WithField("partition", env.Id().String()). - Error("deployment error") + Error(err) return } diff --git a/core/task/manager.go b/core/task/manager.go index 83139c69..d04964f2 100644 --- a/core/task/manager.go +++ b/core/task/manager.go @@ -1207,7 +1207,8 @@ func (m *Manager) handleMessage(tm *TaskmanMessage) error { if err != nil { log.WithError(err). WithField("partition", tm.GetEnvironmentId().String()). - Errorf("Failed task creation and Mesos resources allocation during the deployment of the environment. For more details check Devel logs in Info Logger.") + WithField("level", infologger.IL_Support). + Errorf("failed task creation and Mesos resources allocation during the deployment of the environment. More details in Devel logs.") } }() case taskop.ConfigureTasks: