From cfd0fcc90fa3a3c67289ca4fba26e725f317e4c2 Mon Sep 17 00:00:00 2001 From: Alexandru Avadanii Date: Sat, 14 Oct 2017 19:18:51 +0200 Subject: Drop infinite loops in favor of finite wait_for While at it, fix some shellcheck warnings, and s/fgrep/grep -F/g. Change-Id: I093b7b4c196731b1ecc0c27a4111955b2e412762 Signed-off-by: Alexandru Avadanii (cherry picked from commit 089585ad5370109d4c2a476b94b0017153d7713d) --- mcp/scripts/lib.sh | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'mcp/scripts/lib.sh') diff --git a/mcp/scripts/lib.sh b/mcp/scripts/lib.sh index 748c6dde1..3a6deec21 100644 --- a/mcp/scripts/lib.sh +++ b/mcp/scripts/lib.sh @@ -144,13 +144,13 @@ function start_vms { function check_connection { local total_attempts=60 local sleep_time=5 - local attempt=1 set +e echo '[INFO] Attempting to get into Salt master ...' # wait until ssh on Salt master is available - while ((attempt <= total_attempts)); do + # shellcheck disable=SC2034 + for attempt in $(seq "${total_attempts}"); do # shellcheck disable=SC2086 ssh ${SSH_OPTS} "ubuntu@${SALT_MASTER}" uptime case $? in @@ -158,7 +158,6 @@ function check_connection { *) echo "${attempt}/${total_attempts}> ssh server ain't ready yet, waiting for ${sleep_time} seconds ..." ;; esac sleep $sleep_time - ((attempt+=1)) done set -e } -- cgit 1.2.3-korg