Code review comment for lp:~jameinel/juju-core/api-connect-upgrade-1199915

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

LGTM

https://codereview.appspot.com/10986044/diff/1/cmd/jujud/machine.go
File cmd/jujud/machine.go (right):

https://codereview.appspot.com/10986044/diff/1/cmd/jujud/machine.go#newcode82
cmd/jujud/machine.go:82: log.Errorf("we were unable to install the lxc
package, unable to continue: %v", err)
On 2013/07/11 14:03:11, fwereade wrote:
> Quibble quibble waffle waffle, this will break on other platforms.
Unbreaks
> upgrade for now though, so fair enough.

Add TODO describing this please.

https://codereview.appspot.com/10986044/

« Back to merge proposal