Add flag to disable creation of VSL on install

Fixes #1453

Signed-off-by: Nolan Brubaker <brubakern@vmware.com>

pkg/cmd/cli/install/install.go

@@ -50,6 +50,7 @@ type InstallOptions struct {
 	VolumeSnapshotConfig flag.Map
 	UseRestic            bool
 	Wait                 bool
+	UseVolumeSnapshots   bool
 }
 
 // BindFlags adds command line values to the options struct.
@@ -62,6 +63,7 @@ func (o *InstallOptions) BindFlags(flags *pflag.FlagSet) {
 	flags.StringVar(&o.Namespace, "namespace", o.Namespace, "namespace to install Velero and associated data into. Optional.")
 	flags.Var(&o.BackupStorageConfig, "backup-location-config", "configuration to use for the backup storage location. Format is key1=value1,key2=value2")
 	flags.Var(&o.VolumeSnapshotConfig, "snapshot-location-config", "configuration to use for the volume snapshot location. Format is key1=value1,key2=value2")
+	flags.BoolVar(&o.UseVolumeSnapshots, "use-volume-snapshots", o.UseVolumeSnapshots, "whether or not to create snapshot location automatically. Set to false if you do not plan to create volume snapshots via a storage provider.")
 	flags.BoolVar(&o.RestoreOnly, "restore-only", o.RestoreOnly, "run the server in restore-only mode. Optional.")
 	flags.BoolVar(&o.DryRun, "dry-run", o.DryRun, "generate resources, but don't send them to the cluster. Use with -o. Optional.")
 	flags.BoolVar(&o.UseRestic, "use-restic", o.UseRestic, "create restic deployment. Optional.")
@@ -75,6 +77,8 @@ func NewInstallOptions() *InstallOptions {
 		Image:                install.DefaultImage,
 		BackupStorageConfig:  flag.NewMap(),
 		VolumeSnapshotConfig: flag.NewMap(),
+		// Default to creating a VSL unless we're told otherwise
+		UseVolumeSnapshots: true,
 	}
 }
 
@@ -89,16 +93,17 @@ func (o *InstallOptions) AsVeleroOptions() (*install.VeleroOptions, error) {
 		return nil, err
 	}
 	return &install.VeleroOptions{
-		Namespace:    o.Namespace,
-		Image:        o.Image,
-		ProviderName: o.ProviderName,
-		Bucket:       o.BucketName,
-		Prefix:       o.Prefix,
-		SecretData:   secretData,
-		RestoreOnly:  o.RestoreOnly,
-		UseRestic:    o.UseRestic,
-		BSLConfig:    o.BackupStorageConfig.Data(),
-		VSLConfig:    o.VolumeSnapshotConfig.Data(),
+		Namespace:          o.Namespace,
+		Image:              o.Image,
+		ProviderName:       o.ProviderName,
+		Bucket:             o.BucketName,
+		Prefix:             o.Prefix,
+		SecretData:         secretData,
+		RestoreOnly:        o.RestoreOnly,
+		UseRestic:          o.UseRestic,
+		UseVolumeSnapshots: o.UseVolumeSnapshots,
+		BSLConfig:          o.BackupStorageConfig.Data(),
+		VSLConfig:          o.VolumeSnapshotConfig.Data(),
 	}, nil
 }
 

pkg/install/resources.go

@@ -175,16 +175,17 @@ func appendUnstructured(list *unstructured.UnstructuredList, obj runtime.Object)
 }
 
 type VeleroOptions struct {
-	Namespace    string
-	Image        string
-	ProviderName string
-	Bucket       string
-	Prefix       string
-	SecretData   []byte
-	RestoreOnly  bool
-	UseRestic    bool
-	BSLConfig    map[string]string
-	VSLConfig    map[string]string
+	Namespace          string
+	Image              string
+	ProviderName       string
+	Bucket             string
+	Prefix             string
+	SecretData         []byte
+	RestoreOnly        bool
+	UseRestic          bool
+	UseVolumeSnapshots bool
+	BSLConfig          map[string]string
+	VSLConfig          map[string]string
 }
 
 // AllResources returns a list of all resources necessary to install Velero, in the appropriate order, into a Kubernetes cluster.
@@ -213,8 +214,11 @@ func AllResources(o *VeleroOptions) (*unstructured.UnstructuredList, error) {
 	bsl := BackupStorageLocation(o.Namespace, o.ProviderName, o.Bucket, o.Prefix, o.BSLConfig)
 	appendUnstructured(resources, bsl)
 
-	vsl := VolumeSnapshotLocation(o.Namespace, o.ProviderName, o.VSLConfig)
-	appendUnstructured(resources, vsl)
+	// A snapshot location may not be desirable for users relying on restic
+	if o.UseVolumeSnapshots {
+		vsl := VolumeSnapshotLocation(o.Namespace, o.ProviderName, o.VSLConfig)
+		appendUnstructured(resources, vsl)
+	}
 
 	deploy := Deployment(o.Namespace,
 		WithImage(o.Image),