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
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 provisioner/ provisioner_ task.go (right):
File worker/
https:/ /codereview. appspot. com/93720044/ diff/1/ worker/ provisioner/ provisioner_ task.go# newcode423 provisioner/ provisioner_ task.go: 423: if err1 := SetStatus( params. StatusError, err.Error(), nil); err1 != nil {
worker/
machine.
isn't there a setErrorStatus method already?
https:/ /codereview. appspot. com/93720044/