Code review comment for lp:~jameinel/juju-core/1.18-provisioner-no-tools-is-fatal-1311676

Revision history for this message
William Reade (fwereade) wrote :

LGTM if either use the existing method or have some good reason why it's
unsuitable.

https://codereview.appspot.com/93720044/diff/1/worker/provisioner/provisioner_task.go
File worker/provisioner/provisioner_task.go (right):

https://codereview.appspot.com/93720044/diff/1/worker/provisioner/provisioner_task.go#newcode423
worker/provisioner/provisioner_task.go:423: if err1 :=
machine.SetStatus(params.StatusError, err.Error(), nil); err1 != nil {
isn't there a setErrorStatus method already?

https://codereview.appspot.com/93720044/

« Back to merge proposal