Make the defaultfstype to "" instead of ext4/none

Signed-off-by: Humble Chirammal <[email protected]>
kubernetes-csi · Jun 11, 2020 · fa371e7 · fa371e7
1 parent 6991438
commit fa371e7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cmd/csi-provisioner/csi-provisioner.go b/cmd/csi-provisioner/csi-provisioner.go
@@ -71,7 +71,7 @@ var (
 	metricsAddress = flag.String("metrics-address", "", "The TCP network address where the prometheus metrics endpoint will listen (example: `:8080`). The default is empty string, which means metrics endpoint is disabled.")
 	metricsPath    = flag.String("metrics-path", "/metrics", "The HTTP path where prometheus metrics will be exposed. Default is `/metrics`.")
 
-	defaultFSType = flag.String("default-fstype", "ext4", "Specify the filesystem type of the volume. If not specified,  external-provisioner will set default as `ext4`.")
+	defaultFSType = flag.String("default-fstype", "", "Specify the filesystem type of the volume. If not specified,  external-provisioner will not set any default filesystem type.")
 
 	featureGates        map[string]bool
 	provisionController *controller.ProvisionController

diff --git a/pkg/controller/controller.go b/pkg/controller/controller.go
@@ -519,7 +519,7 @@ func (p *csiProvisioner) ProvisionExt(options controller.ProvisionOptions) (*v1.
 	if fsTypesFound > 1 {
 		return nil, controller.ProvisioningFinished, fmt.Errorf("fstype specified in parameters with both \"fstype\" and \"%s\" keys", prefixedFsTypeKey)
 	}
-	if fsType == "" && p.defaultFSType != "none" {
+	if fsType == "" && p.defaultFSType != "" {
 		fsType = p.defaultFSType
 	}