Code review comment for lp://staging/~jimmiebtlr/juju-core/add_machines_tag

Revision history for this message
Dave Cheney (dave-cheney) wrote :

https://codereview.appspot.com/98610044/diff/60001/cmd/juju/addmachine.go
File cmd/juju/addmachine.go (right):

https://codereview.appspot.com/98610044/diff/60001/cmd/juju/addmachine.go#newcode161
cmd/juju/addmachine.go:161: ctx.Infof("created container %v",
names.MachineTag(machineId))
isn't this now going to say "created container machine-0/lxc:1" or
"created machine machine-0".

To reduce the stuttering how about replacing line 160:165 with

ctx.Infof("created %v", names.MachineTag(machineId))

https://codereview.appspot.com/98610044/

« Back to merge proposal