error msg fixes

Miscellaneous error message tweaks.
Makes the error message displayed in `kubectl get` more visible.
Returns exit code 1 if a patch fails.
pull/6/head
juanvallejo 2017-09-13 17:55:26 -04:00
parent a416534744
commit dfef8574cf
No known key found for this signature in database
GPG Key ID: 7D2C958002D6448D
2 changed files with 10 additions and 1 deletions

View File

@ -239,7 +239,8 @@ __custom_func() {
* services (aka 'svc')
* statefulsets
* storageclasses
`
`
)
var (

View File

@ -183,6 +183,7 @@ func RunPatch(f cmdutil.Factory, out io.Writer, cmd *cobra.Command, args []strin
if !options.Local {
dataChangedMsg := "not patched"
didPatch := false
helper := resource.NewHelper(client, mapping)
patchedObj, err := helper.Patch(namespace, name, patchType, patchBytes)
if err != nil {
@ -213,6 +214,7 @@ func RunPatch(f cmdutil.Factory, out io.Writer, cmd *cobra.Command, args []strin
return err
}
if !reflect.DeepEqual(oldData, newData) {
didPatch = true
dataChangedMsg = "patched"
}
@ -229,6 +231,12 @@ func RunPatch(f cmdutil.Factory, out io.Writer, cmd *cobra.Command, args []strin
return err
}
cmdutil.PrintSuccess(mapper, options.OutputFormat == "name", out, info.Mapping.Resource, info.Name, false, dataChangedMsg)
// if object was not successfully patched, exit with error code 1
if !didPatch {
return cmdutil.ErrExit
}
return nil
}