Merge pull request #4651 from edx/jibsheet/exec-worker

Exec so that bash is replaced by celery

Merge pull request #4651 from edx/jibsheet/exec-worker
Exec so that bash is replaced by celery
e261e0b5 · Kevin Falcone · GitHub · a8ea0248 · f65b4a40 · e261e0b5
Unverified Commit e261e0b5 authored Jun 27, 2018 by Kevin Falcone Committed by GitHub Jun 27, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletions

playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2
+2 -1

No files found.
--- a/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2
+++ b/playbooks/roles/edxapp/templates/edx/app/edxapp/worker.sh.j2
@@ -15,4 +15,5 @@ fi
 {% set executable = edxapp_venv_bin + '/python' %}
 {% endif %}

-{{ executable }} {{ edxapp_code_dir }}/manage.py $@
+# We exec so that celery is the child of supervisor and can be managed properly
+exec {{ executable }} {{ edxapp_code_dir }}/manage.py $@