Add signal handling to agbot_start.sh

When run in a container anax is run via a shell script that does not handle SIGTERM and hence when the container is stopped this signal does not arrive to the binary for graceful exit. A consequence of this is when run via docker compose up, docker compose down will have to wait until the timeout in order to kill the process. This change adds this to agbot_start.sh Signed-off-by: Leonardo Bergesio <[email protected]>
open-horizon · Apr 16, 2024 · efe8e4c · efe8e4c
1 parent 91401ee
commit efe8e4c
Showing 1 changed file with 11 additions and 2 deletions.
diff --git a/anax-in-container/script/agbot_start.sh b/anax-in-container/script/agbot_start.sh
@@ -2,10 +2,19 @@
 
 # Script to start agbot from inside the container
 
-if [ -z "$ANAX_LOG_LEVEL" ]; then
+if [ -z "${ANAX_LOG_LEVEL}" ]; then
 	ANAX_LOG_LEVEL=3
 fi
 
 /usr/bin/envsubst < /etc/horizon/anax.json.tmpl > /etc/horizon/anax.json
-/usr/horizon/bin/anax -v $ANAX_LOG_LEVEL -logtostderr -config /etc/horizon/anax.json
+/usr/horizon/bin/anax -v "${ANAX_LOG_LEVEL}" -logtostderr -config /etc/horizon/anax.json &
+ANAX_PID=$!
 
+# If we receive SIGTERM, forward it to anax to start graceful termination
+send_sigterm() {
+        kill "${ANAX_PID}" 2>/dev/null
+}
+trap send_sigterm TERM
+
+# Wait for anax termination
+wait "${ANAX_PID}"