Merge pull request #1603 from edx/e0d/fix-notifier

Fixing process management

playbooks/roles/notifier/tasks/main.yml

@@ -90,7 +90,15 @@
   file:
     path="{{ NOTIFIER_HOME }}/.ssh" mode=2700 state=directory owner={{ NOTIFIER_USER }} group={{ NOTIFIER_USER }}
 
-
+- name: create service log dir
+  file: >
+    path="{{ item }}"
+    state=directory
+    owner="syslog"
+    group="syslog"
+  with_items:
+    - "{{ COMMON_LOG_DIR }}/notifier"
+    
 - name: write supervisord wrapper for celery workers
   template: >
     src=notifier-celery-workers-supervisor.sh.j2

playbooks/roles/notifier/templates/edx/app/supervisor/conf.d/notifier-celery-workers.conf.j2

@@ -7,6 +7,7 @@ command={{ NOTIFIER_HOME }}/notifier-celery-workers-supervisor.sh
 
 process_name=%(program_name)s
 numprocs=1
+stopasgroup=true
 directory={{ NOTIFIER_CODE_DIR }}
 umask=022
 autostart=true

playbooks/roles/notifier/templates/edx/app/supervisor/conf.d/notifier-scheduler.conf.j2

@@ -7,6 +7,7 @@ command={{ NOTIFIER_HOME }}/notifier-scheduler-supervisor.sh
 
 process_name=%(program_name)s
 numprocs=1
+stopasgroup=true
 directory={{ NOTIFIER_CODE_DIR }}
 umask=022
 autostart=true