rename kube-apiserver-insurce to kube-insecure-skip-tls-verify

Signed-off-by: Dong Gang <dong.gang@daocloud.io>

pkg/cli/environment.go

@@ -56,8 +56,8 @@ type EnvSettings struct {
 	KubeAPIServer string
 	// Custom certificate authority file.
 	KubeCaFile string
-	// KubeInsecure indicates whether or not checking kubernetes apiserver's certificate
+	// KubeInsecureSkipTLSVerify indicates whether or not checking kubernetes apiserver's certificate
-	KubeInsecure bool
+	KubeInsecureSkipTLSVerify bool
 	// Debug indicates whether or not Helm is running in Debug mode.
 	Debug bool
 	// RegistryConfig is the path to the registry config file.
@@ -74,19 +74,19 @@ type EnvSettings struct {
 
 func New() *EnvSettings {
 	env := &EnvSettings{
-		namespace:        os.Getenv("HELM_NAMESPACE"),
+		namespace:                 os.Getenv("HELM_NAMESPACE"),
-		MaxHistory:       envIntOr("HELM_MAX_HISTORY", defaultMaxHistory),
+		MaxHistory:                envIntOr("HELM_MAX_HISTORY", defaultMaxHistory),
-		KubeContext:      os.Getenv("HELM_KUBECONTEXT"),
+		KubeContext:               os.Getenv("HELM_KUBECONTEXT"),
-		KubeToken:        os.Getenv("HELM_KUBETOKEN"),
+		KubeToken:                 os.Getenv("HELM_KUBETOKEN"),
-		KubeAsUser:       os.Getenv("HELM_KUBEASUSER"),
+		KubeAsUser:                os.Getenv("HELM_KUBEASUSER"),
-		KubeAsGroups:     envCSV("HELM_KUBEASGROUPS"),
+		KubeAsGroups:              envCSV("HELM_KUBEASGROUPS"),
-		KubeAPIServer:    os.Getenv("HELM_KUBEAPISERVER"),
+		KubeAPIServer:             os.Getenv("HELM_KUBEAPISERVER"),
-		KubeCaFile:       os.Getenv("HELM_KUBECAFILE"),
+		KubeCaFile:                os.Getenv("HELM_KUBECAFILE"),
-		KubeInsecure:     envBoolOr("HELM_KUBEAPISERVER_INSECURE", false),
+		KubeInsecureSkipTLSVerify: envBoolOr("HELM_KUBEAPISERVER_INSECURE", false),
-		PluginsDirectory: envOr("HELM_PLUGINS", helmpath.DataPath("plugins")),
+		PluginsDirectory:          envOr("HELM_PLUGINS", helmpath.DataPath("plugins")),
-		RegistryConfig:   envOr("HELM_REGISTRY_CONFIG", helmpath.ConfigPath("registry.json")),
+		RegistryConfig:            envOr("HELM_REGISTRY_CONFIG", helmpath.ConfigPath("registry.json")),
-		RepositoryConfig: envOr("HELM_REPOSITORY_CONFIG", helmpath.ConfigPath("repositories.yaml")),
+		RepositoryConfig:          envOr("HELM_REPOSITORY_CONFIG", helmpath.ConfigPath("repositories.yaml")),
-		RepositoryCache:  envOr("HELM_REPOSITORY_CACHE", helmpath.CachePath("repository")),
+		RepositoryCache:           envOr("HELM_REPOSITORY_CACHE", helmpath.CachePath("repository")),
 	}
 	env.Debug, _ = strconv.ParseBool(os.Getenv("HELM_DEBUG"))
 
@@ -100,7 +100,7 @@ func New() *EnvSettings {
 		KubeConfig:       &env.KubeConfig,
 		Impersonate:      &env.KubeAsUser,
 		ImpersonateGroup: &env.KubeAsGroups,
-		Insecure:         &env.KubeInsecure,
+		Insecure:         &env.KubeInsecureSkipTLSVerify,
 	}
 	return env
 }
@@ -115,7 +115,7 @@ func (s *EnvSettings) AddFlags(fs *pflag.FlagSet) {
 	fs.StringArrayVar(&s.KubeAsGroups, "kube-as-group", s.KubeAsGroups, "group to impersonate for the operation, this flag can be repeated to specify multiple groups.")
 	fs.StringVar(&s.KubeAPIServer, "kube-apiserver", s.KubeAPIServer, "the address and the port for the Kubernetes API server")
 	fs.StringVar(&s.KubeCaFile, "kube-ca-file", s.KubeCaFile, "the certificate authority file for the Kubernetes API server connection")
-	fs.BoolVar(&s.KubeInsecure, "kube-insecure-skip-tls-verify", s.KubeInsecure, "if true, the kubernetes apiserver's certificate will not be checked for validity. This will make your HTTPS connections insecure")
+	fs.BoolVar(&s.KubeInsecureSkipTLSVerify, "kube-insecure-skip-tls-verify", s.KubeInsecureSkipTLSVerify, "if true, the kubernetes apiserver's certificate will not be checked for validity. This will make your HTTPS connections insecure")
 	fs.BoolVar(&s.Debug, "debug", s.Debug, "enable verbose output")
 	fs.StringVar(&s.RegistryConfig, "registry-config", s.RegistryConfig, "path to the registry config file")
 	fs.StringVar(&s.RepositoryConfig, "repository-config", s.RepositoryConfig, "path to the file containing repository names and URLs")
@@ -176,13 +176,13 @@ func (s *EnvSettings) EnvVars() map[string]string {
 		"HELM_MAX_HISTORY":       strconv.Itoa(s.MaxHistory),
 
 		// broken, these are populated from helm flags and not kubeconfig.
-		"HELM_KUBECONTEXT":            s.KubeContext,
+		"HELM_KUBECONTEXT":                  s.KubeContext,
-		"HELM_KUBETOKEN":              s.KubeToken,
+		"HELM_KUBETOKEN":                    s.KubeToken,
-		"HELM_KUBEASUSER":             s.KubeAsUser,
+		"HELM_KUBEASUSER":                   s.KubeAsUser,
-		"HELM_KUBEASGROUPS":           strings.Join(s.KubeAsGroups, ","),
+		"HELM_KUBEASGROUPS":                 strings.Join(s.KubeAsGroups, ","),
-		"HELM_KUBEAPISERVER":          s.KubeAPIServer,
+		"HELM_KUBEAPISERVER":                s.KubeAPIServer,
-		"HELM_KUBECAFILE":             s.KubeCaFile,
+		"HELM_KUBECAFILE":                   s.KubeCaFile,
-		"HELM_KUBEAPISERVER_INSECURE": strconv.FormatBool(s.KubeInsecure),
+		"HELM_KUBEINSECURE_SKIP_TLS_VERIFY": strconv.FormatBool(s.KubeInsecureSkipTLSVerify),
 	}
 	if s.KubeConfig != "" {
 		envvars["KUBECONFIG"] = s.KubeConfig

pkg/cli/environment_test.go

@@ -34,13 +34,13 @@ func TestEnvSettings(t *testing.T) {
 		envvars map[string]string
 
 		// expected values
-		ns, kcontext string
+		ns, kcontext              string
-		debug        bool
+		debug                     bool
-		maxhistory   int
+		maxhistory                int
-		kAsUser      string
+		kAsUser                   string
-		kAsGroups    []string
+		kAsGroups                 []string
-		kCaFile      string
+		kCaFile                   string
-		kubeInsecure bool
+		kubeInsecureSkipTLSVerify bool
 	}{
 		{
 			name:       "defaults",