Fix uWSGI configuration for clean shutdown

As per investigation, we can remove exit-on-reload and use the array form of the command to ensure proper signal handling on reloads.
metabrainz · Jan 24, 2024 · a47c5bf · a47c5bf
1 parent c632ecf
commit a47c5bf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/docker/prod/consul-template-redirect.conf b/docker/prod/consul-template-redirect.conf
@@ -5,7 +5,7 @@ template {
 }
 
 exec {
-    command = "chpst -u art:art /home/art/.local/bin/uwsgi /etc/uwsgi/uwsgi.ini"
+    command = ["chpst", "-u", "art:art", "/home/art/.local/bin/uwsgi", "/etc/uwsgi/uwsgi.ini"]
     splay = "10s"
     reload_signal = "SIGHUP"
     kill_signal = "SIGTERM"

diff --git a/docker/prod/uwsgi.ini b/docker/prod/uwsgi.ini
@@ -9,3 +9,6 @@ processes = 10
 die-on-term = true
 worker-reload-mercy = 10
 reload-mercy = 25
+log-x-forwarded-for=true
+; quit uwsgi if the python app fails to load
+need-app = true