Code review comment for lp:~sidnei/juju-core/lxc-clone-with-overlayfs

William Reade (fwereade) wrote :

Yeah, definitely needs tests, otherwise very nice. I guess the notify
problems with overlayfs aren't a big deal if we're never updating the
template underneath a container?
File container/lxc/lxc.go (right):
container/lxc/lxc.go:134: if !template.IsConstructed() {
Would it be reasonable to extract this block as a method?

« Back to merge proposal