From f90c1cf53714eb85783d791c578f35a1d0e713ae Mon Sep 17 00:00:00 2001
From: Leonardo Rossetti <lrossett@redhat.com>
Date: Thu, 9 May 2019 13:50:43 +0100
Subject: [PATCH] fix pv checks

---
 image/tools/lib/component/codeready_pv.sh | 25 ++++++++++++-----------
 image/tools/lib/component/enmasse_pv.sh   |  5 +++++
 2 files changed, 18 insertions(+), 12 deletions(-)

diff --git a/image/tools/lib/component/codeready_pv.sh b/image/tools/lib/component/codeready_pv.sh
index 0a737db..2ef6840 100755
--- a/image/tools/lib/component/codeready_pv.sh
+++ b/image/tools/lib/component/codeready_pv.sh
@@ -8,17 +8,18 @@ function dump_pod_data {
 }
 
 function component_dump_data {
-    workspace_pods=$(oc get pods -n ${PRODUCT_NAMESPACE_PREFIX}codeready | grep workspace | awk '{print $1}')
-
-    if [ "${#workspace_pods}" -eq "0" ]; then
-        echo "No workspaces found to backup"
-    else
-        archive_path="$1/archives"
-        dump_dest="/tmp/codeready-data"
-        mkdir -p $dump_dest
-        for i in $workspace_pods; do dump_pod_data $i $dump_dest; done
-        local ts=$(date '+%H_%M_%S')
-        tar -zcvf "$archive_path/codeready-pv-data-${ts}.tar.gz" -C $dump_dest .
-        rm -rf $dump_dest
+    local pods="$(oc get pods -n ${PRODUCT_NAMESPACE_PREFIX}codeready | grep workspace)"
+    if [ "${#pods}" -eq "0" ]; then
+        echo "=>> No workspaces found to backup"
+        exit 0
     fi
+
+    local workspace_pods="$(echo $pods | awk '{print $1}')"
+    local archive_path="$1/archives"
+    local dump_dest="/tmp/codeready-data"
+    mkdir -p $dump_dest
+    for i in $workspace_pods; do dump_pod_data $i $dump_dest; done
+    local ts=$(date '+%H_%M_%S')
+    tar -zcvf "$archive_path/codeready-pv-data-${ts}.tar.gz" -C $dump_dest .
+    rm -rf $dump_dest
 }
\ No newline at end of file
diff --git a/image/tools/lib/component/enmasse_pv.sh b/image/tools/lib/component/enmasse_pv.sh
index 3e78542..82dc769 100644
--- a/image/tools/lib/component/enmasse_pv.sh
+++ b/image/tools/lib/component/enmasse_pv.sh
@@ -19,6 +19,11 @@ function component_dump_data {
     local dump_dest="/tmp/enmasse-data"
     local pods=$(get_broker_pods)
 
+    if [ "${#pods}" -eq "0" ]; then
+        echo "=>> No broker pods found to backup"
+        exit 0
+    fi
+
     mkdir -p ${dump_dest}
 
     for pod in ${pods}; do