Code review comment for lp:~thumper/juju-core/fast-lxc-no-apt-upgrade

Revision history for this message
Sidnei da Silva (sidnei) wrote :

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

https://codereview.appspot.com/74370044/diff/1/container/lxc/clonetemplate.go#newcode142
container/lxc/clonetemplate.go:142: if backingFilesystem == Btrfs {
I find it odd that backingFilesystem is special-cased for btrfs, since
the version of lxc in trusty has support for picking the right backing
filesystem if -Bbest is specified, which would work for either btrfs or
lvm thin provisioning.

https://codereview.appspot.com/74370044/

« Back to merge proposal