Skip to content

Commit 7195c29

Browse files
committed
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...")
1 parent d80a1da commit 7195c29

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

core/environment/transition_deploy.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -342,9 +342,9 @@ func (t DeployTransition) do(env *Environment) (err error) {
342342
}
343343

344344
if err != nil {
345-
log.WithError(err).
345+
log.WithField("level", infologger.IL_Ops).
346346
WithField("partition", env.Id().String()).
347-
Error("deployment error")
347+
Error(err)
348348
return
349349
}
350350

core/task/manager.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1207,7 +1207,8 @@ func (m *Manager) handleMessage(tm *TaskmanMessage) error {
12071207
if err != nil {
12081208
log.WithError(err).
12091209
WithField("partition", tm.GetEnvironmentId().String()).
1210-
Errorf("Failed task creation and Mesos resources allocation during the deployment of the environment. For more details check Devel logs in Info Logger.")
1210+
WithField("level", infologger.IL_Support).
1211+
Errorf("failed task creation and Mesos resources allocation during the deployment of the environment. More details in Devel logs.")
12111212
}
12121213
}()
12131214
case taskop.ConfigureTasks:

0 commit comments

Comments
 (0)