Fix kubectl Pod Disruption Budget assertion error messages

Signed-off-by: Ferran Rodenas <frodenas@gmail.com>
This commit is contained in:
Ferran Rodenas 2017-10-23 12:42:29 +02:00 committed by Ferran Rodenas
parent 2936479ea1
commit dc5852448e

View File

@ -50,15 +50,15 @@ func (s PodDisruptionBudgetV1Generator) Generate(params map[string]interface{})
} }
name, isString := params["name"].(string) name, isString := params["name"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, saw %v for 'name'", name) return nil, fmt.Errorf("expected string, found %v for 'name'", params["name"])
} }
minAvailable, isString := params["min-available"].(string) minAvailable, isString := params["min-available"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, found %v", minAvailable) return nil, fmt.Errorf("expected string, found %v for 'min-available'", params["min-available"])
} }
selector, isString := params["selector"].(string) selector, isString := params["selector"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, found %v", selector) return nil, fmt.Errorf("expected string, found %v for 'selector'", params["selector"])
} }
delegate := &PodDisruptionBudgetV1Generator{Name: name, MinAvailable: minAvailable, Selector: selector} delegate := &PodDisruptionBudgetV1Generator{Name: name, MinAvailable: minAvailable, Selector: selector}
return delegate.StructuredGenerate() return delegate.StructuredGenerate()
@ -134,22 +134,22 @@ func (s PodDisruptionBudgetV2Generator) Generate(params map[string]interface{})
name, isString := params["name"].(string) name, isString := params["name"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, saw %v for 'name'", name) return nil, fmt.Errorf("expected string, found %v for 'name'", params["name"])
} }
minAvailable, isString := params["min-available"].(string) minAvailable, isString := params["min-available"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, found %v", minAvailable) return nil, fmt.Errorf("expected string, found %v for 'min-available'", params["min-available"])
} }
maxUnavailable, isString := params["max-unavailable"].(string) maxUnavailable, isString := params["max-unavailable"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, found %v", maxUnavailable) return nil, fmt.Errorf("expected string, found %v for 'max-unavailable'", params["max-unavailable"])
} }
selector, isString := params["selector"].(string) selector, isString := params["selector"].(string)
if !isString { if !isString {
return nil, fmt.Errorf("expected string, found %v", selector) return nil, fmt.Errorf("expected string, found %v for 'selector'", params["selector"])
} }
delegate := &PodDisruptionBudgetV2Generator{Name: name, MinAvailable: minAvailable, MaxUnavailable: maxUnavailable, Selector: selector} delegate := &PodDisruptionBudgetV2Generator{Name: name, MinAvailable: minAvailable, MaxUnavailable: maxUnavailable, Selector: selector}
return delegate.StructuredGenerate() return delegate.StructuredGenerate()