From 9c096c92d3623f5d8717efbf63e9d607369f06a8 Mon Sep 17 00:00:00 2001 From: Bartek Grzybowski Date: Thu, 11 Feb 2021 13:20:53 +0100 Subject: [PATCH] [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 --- kubernetes/helm/plugins/deploy/deploy.sh | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) 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 -- 2.16.6