lxd container type: use ZFS by default

Tycho Andersen · Tycho Andersen · commit 65d1076f1c61 · 2016-03-28T16:37:24.000-06:00
Signed-off-by: Tycho Andersen &lt;tycho.andersen@canonical.com&gt;
diff --git a/container/lxd/initialisation.go b/container/lxd/initialisation.go
@@ -28,6 +28,10 @@ var requiredPackages = []string{
 	"lxd",
 }
 
+var xenialPackages = []string{
+	"zfsutils-linux",
+}
+
 type containerInitialiser struct {
 	series string
 }
@@ -48,7 +52,16 @@ func (ci *containerInitialiser) Initialise() error {
 		return err
 	}
 
-	return configureLXDBridge()
+	err = configureLXDBridge()
+	if err != nil {
+		return err
+	}
+
+	if ci.series >= "xenial" {
+		configureZFS()
+	}
+
+	return nil
 }
 
 // getPackageManager is a helper function which returns the
@@ -63,6 +76,24 @@ func getPackagingConfigurer(series string) (config.PackagingConfigurer, error) {
 	return config.NewPackagingConfigurer(series)
 }
 
+func configureZFS() {
+	/* create a 100 GB pool by default (sparse, so it won't actually fill
+	 * that immediately)
+	*/
+	output, err := exec.Command(
+		"lxd",
+		"init",
+		"--auto",
+		"--storage-backend", "zfs",
+		"--storage-pool", "lxd",
+		"--storage-create-loop", "100",
+	).CombinedOutput()
+
+	if err != nil {
+		logger.Warningf("configuring zfs failed with %s: %s", err, string(output))
+	}
+}
+
 func configureLXDBridge() error {
 	f, err := os.OpenFile(lxdBridgeFile, os.O_RDWR, 0777)
 	if err != nil {
@@ -219,5 +250,11 @@ func ensureDependencies(series string) error {
 		}
 	}
 
+	if series >= "xenial" {
+		for _, pack := range xenialPackages {
+			pacman.Install(fmt.Sprintf("--no-install-recommends %s", pack))
+		}
+	}
+
 	return err
 }
