From: Joey Sullivan <joey.sullivan@amdocs.com>
Date: Tue, 11 Jun 2019 14:38:51 +0000 (-0400)
Subject: Increase helm deploy perf by reducing helm ls
X-Git-Tag: 4.0.0-ONAP~4^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?p=oom.git;a=commitdiff_plain;h=10341749c8db075fbd0d6b9684a1b7bf7aab6b33

Increase helm deploy perf by reducing helm ls

Helm ls takes a lot of time to execute.   This
code change makes sure it only executes once per
helm deploy instead of once per helm release.

Issue-ID: OOM-1923
Signed-off-by: Joey Sullivan <joey.sullivan@amdocs.com>
Change-Id: I6337c97380d9f583b33f55a2be1d41ac7ce350c8
---

diff --git a/kubernetes/helm/plugins/deploy/deploy.sh b/kubernetes/helm/plugins/deploy/deploy.sh
index 2bbae80a3e..3416a02596 100755
--- a/kubernetes/helm/plugins/deploy/deploy.sh
+++ b/kubernetes/helm/plugins/deploy/deploy.sh
@@ -206,6 +206,9 @@ deploy() {
 
   # upgrade/install each "enabled" subchart
   cd $CACHE_SUBCHART_DIR/
+  #“helm ls” is an expensive command in that it can take a long time to execute.
+  #So cache the results to prevent repeated execution.
+  ALL_HELM_RELEASES=$(helm ls -q)
   for subchart in * ; do
     SUBCHART_OVERRIDES=$CACHE_SUBCHART_DIR/$subchart/subchart-overrides.yaml
 
@@ -236,7 +239,7 @@ deploy() {
         fi
       fi
     else
-      array=($(helm ls -q | grep "${RELEASE}-${subchart}"))
+      array=($(echo "$ALL_HELM_RELEASES" | grep "${RELEASE}-${subchart}"))
       n=${#array[*]}
       for (( i = n-1; i >= 0; i-- )); do
         helm del "${array[i]}" --purge