lib.sh: Fix stop condition in wait_for 83/50183/1
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Sun, 7 Jan 2018 18:22:18 +0000 (19:22 +0100)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Sun, 7 Jan 2018 23:43:52 +0000 (00:43 +0100)
Change-Id: Ida693b6dd328db283d6992ac33500f4dd1a73eb8
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
(cherry picked from commit 717cb4e0f8d9e430eb31ba643f95682f1592e509)

mcp/scripts/lib.sh

index 2102880..1c51ece 100644 (file)
@@ -436,7 +436,7 @@ function wait_for {
     local total_attempts=$1; shift
     local cmdstr=$*
     local sleep_time=10
-    echo "[wait_for] Waiting for cmd to return success: ${cmdstr}"
+    echo -e "\n[wait_for] Waiting for cmd to return success: ${cmdstr}"
     # shellcheck disable=SC2034
     for attempt in $(seq "${total_attempts}"); do
       echo "[wait_for] Attempt ${attempt}/${total_attempts%.*} for: ${cmdstr}"
@@ -444,11 +444,11 @@ function wait_for {
         # shellcheck disable=SC2015
         eval "${cmdstr}" && echo "[wait_for] OK: ${cmdstr}" && return 0 || true
       else
-        ( eval "${cmdstr}" || echo __fuel_wf_failure__ ) |& tee /dev/stderr | \
-          grep -Eq '(Not connected|No response|__fuel_wf_failure__)' || \
-          echo "[wait_for] OK: ${cmdstr}" && return 0
+        !(eval "${cmdstr}" || echo __fuel_wf_failure__) |& tee /dev/stderr | \
+          grep -Eq '(Not connected|No response|__fuel_wf_failure__)' && \
+          echo "[wait_for] OK: ${cmdstr}" && return 0 || true
       fi
-      echo -n '.'; sleep "${sleep_time}"
+      sleep "${sleep_time}"
     done
     echo "[wait_for] ERROR: Failed after max attempts: ${cmdstr}"
     return 1