cli: Add context to policy parsing errors #683
1 changed files with 2 additions and 2 deletions
|
@ -185,12 +185,12 @@ func parseContainerPolicy(cmd *cobra.Command, policyString string) (*netmap.Plac
|
||||||
return &result, nil
|
return &result, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
if err = result.UnmarshalJSON([]byte(policyString)); err == nil {
|
if err := result.UnmarshalJSON([]byte(policyString)); err == nil {
|
||||||
|
|||||||
common.PrintVerbose(cmd, "Parsed JSON encoded policy")
|
common.PrintVerbose(cmd, "Parsed JSON encoded policy")
|
||||||
return &result, nil
|
return &result, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
return nil, errors.New("can't parse placement policy")
|
return nil, fmt.Errorf("can't parse placement policy: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
func parseAttributes(dst *container.Container, attributes []string) error {
|
func parseAttributes(dst *container.Container, attributes []string) error {
|
||||||
|
|
Loading…
Add table
Reference in a new issue
It looks unnatural for Go language :)
Can you swap these lines please
It will become incorrect then -- we intentionally return an error from
DecodeString
ah, ok