added CLI specific test

Author: jdnurme

pkg/sidecar_mounter/sidecar_mounter_config.go

@@ -206,7 +206,9 @@ func (mc *MountConfig) prepareMountArgs() {
 	for _, arg := range mc.Options {
 		klog.Infof("processing mount arg %v", arg)
 
-		if strings.Contains(arg, ":") && !strings.Contains(arg, "https") {
+		// Config file flags are identified by not containing '=' and containing ':'.
+		// e.g., "logging:severity:error"
+		if !strings.Contains(arg, "=") && strings.Contains(arg, ":") && !strings.Contains(arg, "https") {
 			var f, v string
 			if strings.HasPrefix(arg, "gcs-connection:custom-endpoint:") {
 				parts := strings.SplitN(arg, ":", 3)

pkg/sidecar_mounter/sidecar_mounter_config_test.go

@@ -318,6 +318,26 @@ func TestPrepareMountArgs(t *testing.T) {
 				"gcs-connection:custom-endpoint": "custom-service.my-system.svc.cluster.local:8080",
 			},
 		},
+		{
+			name: "should correctly parse custom-endpoint with a port in CLI format",
+			mc: &MountConfig{
+				BucketName: "test-bucket",
+				BufferDir:  "test-buffer-dir",
+				CacheDir:   "test-cache-dir",
+				ConfigFile: "test-config-file",
+				Options:    []string{"custom-endpoint=custom-service.my-system.svc.cluster.local:8080"},
+			},
+			expectedArgs: map[string]string{
+				"app-name":        GCSFuseAppName,
+				"temp-dir":        "test-buffer-dir/temp-dir",
+				"config-file":     "test-config-file",
+				"foreground":      "",
+				"uid":             "0",
+				"gid":             "0",
+				"custom-endpoint": "custom-service.my-system.svc.cluster.local:8080",
+			},
+			expectedConfigMapArgs: defaultConfigFileFlagMap,
+		},
 	}
 
 	testPrometheusPort := prometheusPort