Merge pull request #5 from mysteriumnetwork/fix/backup-script

Cleanup Dockerfile
mysteriumnetwork · Mar 31, 2023 · d622ade · d622ade
2 parents 8ab7bbd + 0423caa
commit d622ade
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Dockerfile b/Dockerfile
@@ -30,4 +30,4 @@ RUN chmod 600 /root/.pgpass
 #Copying script file
 USER root 
 #switching the user to give elevated access to the commands being executed from the k8s cron job
-CMD sh /home/root/backup.sh
+CMD bash /home/root/backup.sh
diff --git a/backup.sh b/backup.sh
@@ -102,7 +102,7 @@ if [ -n "$HEARTBEAT_URL" ]; then
   # shellcheck disable=SC2034
   MAX_RETRIES=5
   CURRENT=0
-  while [[ -z "$HTTP_CODE" || $CURRENT -lt $MAX_RETRIES ]]; do
+  while [[ -z "$HTTP_CODE" || "$CURRENT" -lt "$MAX_RETRIES" ]]; do
     HTTP_CODE=$(curl --silent --write-out "%{http_code}" --output /dev/null "$HEARTBEAT_URL")
     if [ "$HTTP_CODE" -eq "$SUCCESS_HTTP_CODE" ]; then exit 0; fi
     sleep $((2 ** "$CURRENT"))