Code review comment for lp:~wallyworld/juju-core/retryprovisioning-command

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

I'll add to the chorus: LGTM

https://codereview.appspot.com/80410044/diff/1/cmd/juju/resolveprovisioningerror.go
File cmd/juju/resolveprovisioningerror.go (right):

https://codereview.appspot.com/80410044/diff/1/cmd/juju/resolveprovisioningerror.go#newcode23
cmd/juju/resolveprovisioningerror.go:23: Name:
"resolve-provisioning-error",
retry-provisioning, as discussed

https://codereview.appspot.com/80410044/diff/1/cmd/juju/resolveprovisioningerror.go#newcode25
cmd/juju/resolveprovisioningerror.go:25: Purpose: "marks provisioning
errors resolved",
"retries provisioning for a failed machine"?

https://codereview.appspot.com/80410044/

« Back to merge proposal