diff options
| author |   Bartek Grzybowski <b.grzybowski@partner.samsung.com> | 2021-02-11 13:20:53 +0100 |
|---|---|---|
| committer | Bartek Grzybowski <b.grzybowski@partner.samsung.com> | 2021-02-11 13:20:53 +0100 |
| commit | 9c096c92d3623f5d8717efbf63e9d607369f06a8 (patch) | |
| tree | deded161ebfe765881bf5b3fe25d4d8b8b1321ca | |
| parent | 5041b168ab3326aa829dbd5dd950dbde5e002acf (diff) | |
[GENERAL] Fix failed chart list reporting in deploy plugin
Helm's v3 'list' subcommand syntax has changed and without
[-A | --all-namespaces] flag it's output will contain only
charts from default namespace, not the one ONAP was deployed
in. Hence if any chart fail to deploy the output of "helm ls"
would be still empty without --all-namespaces flag.
Change-Id: I425a6ac8823b69e64f2f68f4475c51691239ad33
Issue-ID: OOM-2677
Signed-off-by: Bartek Grzybowski <b.grzybowski@partner.samsung.com>
| -rwxr-xr-x | kubernetes/helm/plugins/deploy/deploy.sh | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/kubernetes/helm/plugins/deploy/deploy.sh b/kubernetes/helm/plugins/deploy/deploy.sh index 40338b9485..3bcb404032 100755 --- a/kubernetes/helm/plugins/deploy/deploy.sh +++ b/kubernetes/helm/plugins/deploy/deploy.sh @@ -266,7 +266,11 @@ deploy() { done # report on success/failures of installs/upgrades - helm ls | grep FAILED | grep $RELEASE + if [[ $HELM_VER == "v3."* ]]; then + helm ls --all-namespaces | grep -i FAILED | grep $RELEASE + else + helm ls | grep FAILED | grep $RELEASE + fi } HELM_VER=$(helm version --template "{{.Version}}") echo $HELM_VER |