misc: Minor improvements

assets/runtime/functions.sh

@@ -217,6 +217,7 @@ function configure_salt_master()
 #----------------------------------------------------------------------------------------------------------------------
 function configure_salt_api()
 {
+  rm -f /etc/supervisor/conf.d/salt-api.conf
   [[ ${SALT_API_SERVICE_ENABLED} == true ]] || return 0
 
   if [[ -n "${SALT_API_USER}" ]]; then

entrypoint.sh

@@ -3,7 +3,9 @@
 set -e
 set -o pipefail
 
-source "${SALT_RUNTIME_DIR}/functions.sh"
+# shellcheck source=assets/runtime/functions.sh
+FUNCTIONS_FILE="${SALT_RUNTIME_DIR}/functions.sh"
+source "${FUNCTIONS_FILE}"
 
 [[ "${DEBUG}" == true ]] && set -x
 
@@ -28,7 +30,7 @@ case "${1}" in
     case "${1}" in
       salt-master|salt-api)
         echo "Restarting ${1} service ..."
-        exec pkill "${1}"
+        exec supervisorctl restart "${1}"
         ;;
       *)
         log_error "Unable to restart ${1} serice. Service is unknown"

tests/gitfs/test.sh

@@ -13,7 +13,7 @@ COMMON_FILE="${SCRIPT_PATH}/../lib/common.sh"
 source "${COMMON_FILE}"
 trap cleanup EXIT
 
-export GITFS_KEYS_DIR=${GITFS_KEYS_DIR:-tests/gitfs/data/keys/gitfs}
+export GITFS_KEYS_DIR=${GITFS_KEYS_DIR:-"${SCRIPT_PATH}/data/keys/gitfs"}
 
 # Check gitfs keys are present
 echo "==> Checking gitfs keys are present ..."
@@ -25,8 +25,8 @@ ok "gitfs keys"
 # Run test instance
 echo "==> Starting docker-salt-master (${PLATFORM}) with RSA 4096 ssh key ..."
 start_container_and_wait \
-  --volume "$(pwd)/tests/gitfs/config":/home/salt/data/config:ro \
-  --volume "$(pwd)/${GITFS_KEYS_DIR%%/gitfs}":/home/salt/data/keys \
+  --volume "${SCRIPT_PATH}/config":/home/salt/data/config:ro \
+  --volume "${GITFS_KEYS_DIR%%/gitfs}":/home/salt/data/keys \
 || error "container started"
 ok "container started"
 

tests/lib/common.sh

@@ -30,7 +30,7 @@ export BOOTUP_WAIT_SECONDS=${BOOTUP_WAIT_SECONDS:-60}
 #----------------------------------------------------------------------------------------------------------------------
 function cleanup()
 {
-  echo "==> Removing ${CONTAINER_NAME} ..."
+  echo "🧹 Removing ${CONTAINER_NAME} ..."
   docker container rm --force "${CONTAINER_NAME}"
 }
 

tests/salt-api/test.sh

@@ -31,7 +31,7 @@ external_auth:
       - '@wheel'
       - '@jobs'
 EOF
-echo "salt-api config created ✅"
+ok "salt-api config created"
 
 # Run test instance
 echo "==> Starting docker-salt-master (${PLATFORM}) with salt-api config ..."