github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-23 11:50:44 +00:00
Code Issues Projects Releases Wiki Activity

Report validation errors when command-line flag parsing fails.

Browse Source 
Before this, I was stumped with

 invalid argument "federation=kubernetes-federation.test." for --federations=federation=kubernetes-federation.test.: federation not a valid federation name

but now

  invalid argument "federation=kubernetes-federation.test." for --federations=federation=kubernetes-federation.test.: "kubernetes-federation.test." not a valid domain name: ["must match the regex [a-z0-9]([-a-z0-9]*[a-z0-9])?(\\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)* (e.g. 'example.com')"]
This commit is contained in:
Matt Liggett 2016-06-21 15:49:56 -07:00
parent 8b817c4a0d
commit 62c052bb6b
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
cmd/kube-dns/app/options/options.go
View File

@ -112,13 +112,13 @@ func (fv federationsVar) Set(keyVal string) error {
splits := strings.SplitN(strings.TrimSpace(val), "=", 2) splits := strings.SplitN(strings.TrimSpace(val), "=", 2)
name := strings.TrimSpace(splits[0]) name := strings.TrimSpace(splits[0])
domain := strings.TrimSpace(splits[1]) domain := strings.TrimSpace(splits[1])
if len(validation.IsDNS1123Label(name)) != 0 { if errs := validation.IsDNS1123Label(name); len(errs) != 0 {
return fmt.Errorf("%s not a valid federation name", name) return fmt.Errorf("%q not a valid federation name: %q", name, errs)
} }
// The federation domain name need not strictly be domain names, we // The federation domain name need not strictly be domain names, we
// accept valid dns names with subdomain components. // accept valid dns names with subdomain components.
if len(validation.IsDNS1123Subdomain(domain)) != 0 { if errs := validation.IsDNS1123Subdomain(domain); len(errs) != 0 {
return fmt.Errorf("%s not a valid federation name", name) return fmt.Errorf("%q not a valid domain name: %q", domain, errs)
} }
fv.nameDomainMap[name] = domain fv.nameDomainMap[name] = domain
} }