feat: remove option to set volume snapshotter

Flintlock does not handle other snapshotters well, so we should be opinionated about this. There is no real reason to support others.
liquidmetal-dev · Dec 6, 2021 · d15012d · d15012d
1 parent 5fa3d5a
commit d15012d
Show file tree

Hide file tree

Showing 6 changed files with 8 additions and 13 deletions.
diff --git a/infrastructure/containerd/config.go b/infrastructure/containerd/config.go
@@ -4,7 +4,7 @@ const (
 	supportedSnapshotters = "overlayfs,native,devmapper"
 )
 
-// Config holds the contaierd configuration.
+// Config holds the containerd configuration.
 type Config struct {
 	// SnapshotterKernel is the name of the containerd snapshotter to use for kernel images.
 	SnapshotterKernel string

diff --git a/internal/command/flags/flags.go b/internal/command/flags/flags.go
@@ -18,7 +18,6 @@ const (
 	firecrackerBinFlag    = "firecracker-bin"
 	firecrackerDetachFlag = "firecracker-detach"
 	containerdSocketFlag  = "containerd-socket"
-	volSnapshotterFlag    = "containerd-volume-ss"
 	kernelSnapshotterFlag = "containerd-kernel-ss"
 	containerdNamespace   = "containerd-ns"
 	maximumRetryFlag      = "maximum-retry"
@@ -115,11 +114,6 @@ func AddContainerDFlagsToCommand(cmd *cobra.Command, cfg *config.Config) error {
 		defaults.ContainerdKernelSnapshotter,
 		"The name of the snapshotter to use with containerd for kernel/initrd images.")
 
-	cmd.Flags().StringVar(&cfg.CtrSnapshotterVolume,
-		volSnapshotterFlag,
-		defaults.ContainerdVolumeSnapshotter,
-		"The name of the snapshotter to use with containerd for volume images.")
-
 	cmd.Flags().StringVar(&cfg.CtrNamespace,
 		containerdNamespace,
 		defaults.ContainerdNamespace,

diff --git a/internal/config/config.go b/internal/config/config.go
@@ -26,8 +26,6 @@ type Config struct {
 	ParentIface string
 	// CtrSnapshotterKernel is the name of the containerd snapshotter to use for kernel images.
 	CtrSnapshotterKernel string
-	// CtrSnapshotterVolume is the name of the containerd snapshotter to use for volume (inc initrd) images.
-	CtrSnapshotterVolume string
 	// CtrSocketPath is the path to the containerd socket.
 	CtrSocketPath string
 	// CtrNamespace is the default containerd namespace to use

diff --git a/internal/inject/wire.go b/internal/inject/wire.go
@@ -19,6 +19,7 @@ import (
 	"github.com/weaveworks/flintlock/infrastructure/network"
 	"github.com/weaveworks/flintlock/infrastructure/ulid"
 	"github.com/weaveworks/flintlock/internal/config"
+	"github.com/weaveworks/flintlock/pkg/defaults"
 )
 
 func InitializePorts(cfg *config.Config) (*ports.Collection, error) {
@@ -58,7 +59,7 @@ func InitializeGRPCServer(app application.App) ports.MicroVMGRPCService {
 func containerdConfig(cfg *config.Config) *containerd.Config {
 	return &containerd.Config{
 		SnapshotterKernel: cfg.CtrSnapshotterKernel,
-		SnapshotterVolume: cfg.CtrSnapshotterVolume,
+		SnapshotterVolume: defaults.ContainerdVolumeSnapshotter,
 		SocketPath:        cfg.CtrSocketPath,
 		Namespace:         cfg.CtrNamespace,
 	}

diff --git a/internal/inject/wire_gen.go b/internal/inject/wire_gen.go
diff --git a/pkg/defaults/defaults.go b/pkg/defaults/defaults.go
@@ -14,7 +14,7 @@ const (
 	// ContainerdSocket is the defaults path for the containerd socket.
 	ContainerdSocket = "/run/containerd/containerd.sock"
 
-	// ContainerdVolumeSnapshotter is the name of the default snapshotter to use for volumes.
+	// ContainerdVolumeSnapshotter is the name of the snapshotter used for volumes.
 	ContainerdVolumeSnapshotter = "devmapper"
 
 	// ContainerdKernelSnapshotter is the name of the default snapshotter to use for kernek/initrd.