Code review comment for lp:~sidnei/juju-core/lxc-mirror

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

LGTM, although I think I reviewed part of that as a previous CL
(AddFile, AddScripts, etc.) did that one get merged here?

https://codereview.appspot.com/12143043/diff/7001/container/lxc/lxc_test.go
File container/lxc/lxc_test.go (right):

https://codereview.appspot.com/12143043/diff/7001/container/lxc/lxc_test.go#newcode46
container/lxc/lxc_test.go:46: []byte(aptConfig), 0755)

Please put 0755 on a separate line, like the other args.

https://codereview.appspot.com/12143043/

« Back to merge proposal