[GENERAL] Fix failed chart list reporting in deploy plugin 43/117743/1
authorBartek Grzybowski <b.grzybowski@partner.samsung.com>
Thu, 11 Feb 2021 12:20:53 +0000 (13:20 +0100)
committerBartek Grzybowski <b.grzybowski@partner.samsung.com>
Thu, 11 Feb 2021 12:20:53 +0000 (13:20 +0100)
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>
kubernetes/helm/plugins/deploy/deploy.sh

index 40338b9..3bcb404 100755 (executable)
@@ -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