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
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 lxc/lxc_ test.go (right):
File container/
https:/ /codereview. appspot. com/12143043/ diff/7001/ container/ lxc/lxc_ test.go# newcode46 lxc/lxc_ test.go: 46: []byte(aptConfig), 0755)
container/
Please put 0755 on a separate line, like the other args.
https:/ /codereview. appspot. com/12143043/