update to use bash shell

Author: Tristen Harr

CHANGELOG.md

@@ -4,8 +4,11 @@ This changelog documents the changes between release versions.
 ## [Unreleased]
 Changes to be included in the next upcoming release
 
-## [0.0.31] - 2024-07-22
-* Update workflow to create multi-platform build
+## [0.0.32] - 2024-07-29
+* Fix scripts and base image to use bash instead of sh
+
+## [0.0.31] - 2024-07-29
+* Fix base image to Python 3.11
 
 ## [0.0.30] - 2024-07-22
 * Update workflow to create multi-platform build

Dockerfile

@@ -1,12 +1,20 @@
 FROM python:3.11
 
-COPY /docker /scripts
+# Install curl for healthcheck
+RUN apt-get update && apt-get install -y curl && rm -rf /var/lib/apt/lists/*
 
+COPY /docker /scripts
 COPY /functions /functions
+
+# Ensure scripts are executable
+RUN chmod +x /scripts/package-restore.sh /scripts/start.sh
+
+# Run the package-restore script
 RUN /scripts/package-restore.sh
 
 EXPOSE 8080
 
-HEALTHCHECK --interval=5s --timeout=10s --start-period=1s --retries=3 CMD [ "sh", "-c", "exec curl -f http://localhost:${HASURA_CONNECTOR_PORT:-8080}/health" ]
+HEALTHCHECK --interval=5s --timeout=10s --start-period=1s --retries=3 \
+    CMD [ "bash", "-c", "exec curl -f http://localhost:${HASURA_CONNECTOR_PORT:-8080}/health" ]
 
-CMD [ "sh", "/scripts/start.sh" ]
+CMD [ "/scripts/start.sh" ]

docker/package-restore.sh

@@ -1,5 +1,5 @@
-#!/usr/bin/env sh
-set -eu -o pipefail
+#!/usr/bin/env bash
+set -eu
 
 cd /functions
 
@@ -13,4 +13,4 @@ then
   fi
 else
   . venv/bin/activate
-fi
+fi

docker/start.sh

@@ -1,5 +1,5 @@
-#!/usr/bin/env sh
-set -eu -o pipefail
+#!/usr/bin/env bash
+set -eu
 
 /scripts/package-restore.sh
 
@@ -11,4 +11,4 @@ if [ -f "venv/bin/activate" ]; then
 fi
 
 # Run the Python script with the serve command
-python3 functions.py serve --configuration ./
+exec python3 functions.py serve --configuration ./