app-registry / appr-helm-plugin

Apache License 2.0
56 stars 15 forks source link

trying to install non existing chart should show a proper message #12

Open rimusz opened 7 years ago

rimusz commented 7 years ago

when I try to install non-existing chart I get bunch of python errors:

$ helm registry install quay.io/charts/gcloud-sqlproxy
Traceback (most recent call last):
  File "bin/cnr", line 6, in <module>
  File "cnrclient/commands/cli.py", line 51, in cli
  File "cnrclient/commands/helm.py", line 32, in _install
  File "cnrclient/commands/helm.py", line 23, in _exec_cmd
  File "cnrclient/commands/pull.py", line 34, in _call
  File "cnrclient/client.py", line 115, in pull_json
  File "requests/models.py", line 909, in raise_for_status
requests.exceptions.HTTPError: 403 Client Error: FORBIDDEN for url: https://quay.io/cnr/api/v1/packages/charts/gcloud-sqlproxy/default/helm/pull?format=json
Failed to execute script cnr
Error: plugin "registry" exited with error

the plugin should show a proper error message