Commit ecda7c60 authored by Davanum Srinivas's avatar Davanum Srinivas

Allow empty value for env in kubectl run

Fixes #28734
parent 7f528c67
...@@ -934,7 +934,7 @@ func parseEnvs(envArray []string) ([]api.EnvVar, error) { ...@@ -934,7 +934,7 @@ func parseEnvs(envArray []string) ([]api.EnvVar, error) {
} }
name := env[:pos] name := env[:pos]
value := env[pos+1:] value := env[pos+1:]
if len(name) == 0 || len(value) == 0 { if len(name) == 0 {
return nil, fmt.Errorf("invalid env: %v", env) return nil, fmt.Errorf("invalid env: %v", env)
} }
if len(validation.IsCIdentifier(name)) != 0 { if len(validation.IsCIdentifier(name)) != 0 {
...@@ -955,7 +955,7 @@ func parseV1Envs(envArray []string) ([]v1.EnvVar, error) { ...@@ -955,7 +955,7 @@ func parseV1Envs(envArray []string) ([]v1.EnvVar, error) {
} }
name := env[:pos] name := env[:pos]
value := env[pos+1:] value := env[pos+1:]
if len(name) == 0 || len(validation.IsCIdentifier(name)) != 0 || len(value) == 0 { if len(name) == 0 || len(validation.IsCIdentifier(name)) != 0 {
return nil, fmt.Errorf("invalid env: %v", env) return nil, fmt.Errorf("invalid env: %v", env)
} }
envVar := v1.EnvVar{Name: name, Value: value} envVar := v1.EnvVar{Name: name, Value: value}
......
...@@ -858,8 +858,13 @@ func TestParseEnv(t *testing.T) { ...@@ -858,8 +858,13 @@ func TestParseEnv(t *testing.T) {
envArray: []string{ envArray: []string{
"WITH_OUT_VALUES=", "WITH_OUT_VALUES=",
}, },
expected: []api.EnvVar{}, expected: []api.EnvVar{
expectErr: true, {
Name: "WITH_OUT_VALUES",
Value: "",
},
},
expectErr: false,
test: "test case 3", test: "test case 3",
}, },
{ {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment