diff --git a/internals/overlord/state/taskrunner.go b/internals/overlord/state/taskrunner.go index cfed27bd..0d162dc8 100644 --- a/internals/overlord/state/taskrunner.go +++ b/internals/overlord/state/taskrunner.go @@ -291,7 +291,7 @@ func (r *TaskRunner) run(t *Task) { t.SetStatus(ErrorStatus) t.Errorf("%s", err) // ensure the error is available in the global log too - logger.Noticef("[change %s %q task] failed: %v", t.Change().ID(), t.Summary(), err) + logger.Noticef("Change %s task (%s) failed: %v", t.Change().ID(), t.Summary(), err) if r.taskErrorCallback != nil { r.taskErrorCallback(err) } diff --git a/internals/overlord/state/taskrunner_test.go b/internals/overlord/state/taskrunner_test.go index 979d05e9..241aec37 100644 --- a/internals/overlord/state/taskrunner_test.go +++ b/internals/overlord/state/taskrunner_test.go @@ -1290,7 +1290,7 @@ func (ts *taskRunnerSuite) TestErrorCallbackCalledOnError(c *C) { c.Check(strings.Join(t1.Log(), ""), Matches, `.*handler error for "foo"`) c.Check(called, Equals, true) - c.Check(logbuf.String(), Matches, `(?m).*: \[change 1 "task summary" task\] failed: handler error for "foo".*`) + c.Check(logbuf.String(), Matches, `(?m).*: Change 1 task \(task summary\) failed: handler error for "foo".*`) } func (ts *taskRunnerSuite) TestErrorCallbackNotCalled(c *C) {