commit 6e5b45ab81ee9bfba2a396b65ffdd999abffc4e4
author Samuli Silvius <s.silvius@partner.samsung.com> Mon Feb 04 12:52:20 2019 +0200
committer Samuli Silvius <s.silvius@partner.samsung.com> Tue Feb 12 09:35:15 2019 +0200
tree 20d05f12c2e76c8d7fdf62e18bf47efaaeb30bf6
parent 5f5aa01c0f757ff3a9eb55eb978b98e45f23a079

Helm charts dir commenting causes / dir copying

HELM_CHARTS_DIR in package.conf is unnecessarily mandatory.
When commenting out this variable, packaging script copies
machine's root directory content into package.

This commit will make HELM_CHARTS_DIR optional and its
existence is checked in package.sh script.

Issue-ID: OOM-1626

Change-Id: I7b5d135ab130a28491e7223b66f26b50ccce5fca
Signed-off-by: Samuli Silvius <s.silvius@partner.samsung.com>

build/package.conf

diff --git a/build/package.conf b/build/package.conf
index 1141798..c3bbeee 100644
--- a/build/package.conf
+++ b/build/package.conf
@@ -16,8 +16,7 @@
 ########################
 
 # directory with helm charts
-HELM_CHARTS_DIR="/root/oom"
-
+#HELM_CHARTS_DIR=~/myclones/casablanca_oom/
 
 ###################
 # Packages addons #

build/package.sh

diff --git a/build/package.sh b/build/package.sh
index 0e4f121..fd9b9e7 100755
--- a/build/package.sh
+++ b/build/package.sh
@@ -105,10 +105,13 @@
     done
 
     # Helm charts handling
-    echo "Helm charts handling"
-    # Copy charts available for ansible playbook to use/move them to target server/dir
-    mkdir -p "${pkg_root}"/ansible/application/helm_charts
-    cp -r "${HELM_CHARTS_DIR}"/* "${pkg_root}"/ansible/application/helm_charts
+    if [ ! -z "${HELM_CHARTS_DIR}" ];
+    then
+        echo "Helm charts handling"
+        # Copy charts available for ansible playbook to use/move them to target server/dir
+        mkdir -p "${pkg_root}"/ansible/application/helm_charts
+        cp -r "${HELM_CHARTS_DIR}"/* "${pkg_root}"/ansible/application/helm_charts
+    fi
 
     # Add metadata to the package
     add_metadata "${pkg_root}"/package.info