search

getporter / helm2-mixin

Helm mixin for Porter
https://porter.sh/mixins/helm
Apache License 2.0
13 stars 7 forks source link

Do not print helptext on runtime errors #27

Closed carolynvs closed 5 years ago

carolynvs commented 5 years ago

Here's an example of why we want this:

Uninstall MySQL
Error: uninstall.0.helm: releases is required
    * uninstall.0.helm: Additional property name is not allowed
Usage:
  helm uninstall [flags]

Flags:
  -h, --help   help for uninstall

Global Flags:
      --debug   Enable debug logging

err: uninstall.0.helm: releases is required
    * uninstall.0.helm: Additional property name is not allowed
Error: mixin execution failed: exit status 1

The helptext should only be printed when the command was called incorrectly. Otherwise it's just noise. We have this turned on for other commands (like porter) but just never got around flipping the switch for this one too.