fix code rabbit comments

jembi · Jul 17, 2024 · fcbbb89 · fcbbb89
1 parent 4a8090d
commit fcbbb89
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 3 deletions.
diff --git a/devops/linux/docker/backup_restore/dgraph-backup-api.sh b/devops/linux/docker/backup_restore/dgraph-backup-api.sh
@@ -3,6 +3,11 @@
 set -e
 set -u
 pushd .
+    if [ -z "$1" ]; then
+        echo "Error: No backup folder name provided."
+        echo "Usage: $0 <backup_folder_name>"
+        exit 1
+    fi
     BACKUP_DATE_TIME=$1
     SCRIPT_DIR=$(cd -P -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd -P)
     cd "${SCRIPT_DIR}/.." || exit

diff --git a/devops/linux/docker/backup_restore/dgraph-restore-api.sh b/devops/linux/docker/backup_restore/dgraph-restore-api.sh
@@ -1,6 +1,11 @@
 #!/bin/bash
 source ../conf.env
 #Backup Folder Name
+if [ -z "$1" ]; then
+    echo "Error: No backup folder name provided."
+    echo "Usage: $0 <backup_folder_name>"
+    exit 1
+fi
     BACKUP_FOLDER_NAME=$1
     SCRIPT_DIR=$(cd -P -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd -P)
     cd ${SCRIPT_DIR}/..

diff --git a/devops/linux/docker/backup_restore/postgres-backup.sh b/devops/linux/docker/backup_restore/postgres-backup.sh
@@ -1,6 +1,11 @@
 #!/bin/bash
 source ../conf.env
 # Load Database Credentials from Environment Variables
+if [ -z "$1" ]; then
+    echo "Error: No backup folder name provided."
+    echo "Usage: $0 <backup_folder_name>"
+    exit 1
+fi
 if [ -z "$1" ]; then
     # Argument is empty, so set current datetime
     BACKUP_DATE_TIME=$(date +%Y%m%d_%H%M%S)

diff --git a/devops/linux/docker/backup_restore/postgres-restore.sh b/devops/linux/docker/backup_restore/postgres-restore.sh
@@ -1,5 +1,10 @@
 #!/bin/bash
 source ../conf.env
+if [ -z "$1" ]; then
+    echo "Error: No backup folder name provided."
+    echo "Usage: $0 <backup_folder_name>"
+    exit 1
+fi
 BACKUP_FOLDER_NAME=$1
 
 # PostgreSQL settings from environment variables

diff --git a/devops/linux/docker/backup_restore/restore-dgraph-postgres.sh b/devops/linux/docker/backup_restore/restore-dgraph-postgres.sh
@@ -2,7 +2,7 @@
 source ../conf.env
 #Backup Folder Name
 SCRIPT_DIR=$(cd -P -- "$(dirname -- "${BASH_SOURCE[0]}")" && pwd -P)
-cd ${SCRIPT_DIR}/..
+cd ${SCRIPT_DIR}/.. || exit
 JEMPI_DOCKER_HOME=$PWD
 # JEMPI_HOME = $1
 down_dir="$JEMPI_DOCKER_HOME/deployment/down"
@@ -16,10 +16,10 @@ python_cmd=$(which python3 || which python)
 echo $python_cmd
 Function to stop services
 stop_services() {
-    pushd "$down_dir"
+    pushd "$down_dir" || exit
         echo "Stopping API service"
         source d-stack-stop-services.sh
-    popd
+    popd || exit
 }
 start_services() {
     pushd "$reboot_dir" || exit