Code review comment for lp://staging/~waigani/juju-core/lxc-trusty-autostart

Revision history for this message
Tim Penhey (thumper) wrote :

https://codereview.appspot.com/56370043/diff/1/container/lxc/lxc.go
File container/lxc/lxc.go (right):

https://codereview.appspot.com/56370043/diff/1/container/lxc/lxc.go#newcode112
container/lxc/lxc.go:112: // Now symlink the config file into the
restart directory, if it exists
On 2014/01/24 02:19:17, axw wrote:
> I think it'd be good to add a comment as to when it does/doesn't
exist. i.e.
> that this is for backwards-compatibility, and the directory no longer
exists
> from trusty.

I agree that there should be more of a comment here to explain
intent.

https://codereview.appspot.com/56370043/

« Back to merge proposal