fixing discern service management

playbooks/roles/devpi/tasks/main.yml

@@ -95,6 +95,17 @@
     dest={{ COMMON_CFG_DIR }}/supervisord.conf.devpi
     state=link
 
+  # call supervisorctl update. this reloads
+  # the supervisorctl config and restarts
+  # the services if any of the configurations
+  # have changed.
+  #
+- name: devpi | update devpi supervisor configuration
+  shell:  "{{ devpi_supervisor_ctl }} -c {{ devpi_supervisor_cfg }} update"
+  register: supervisor_update
+  changed_when: supervisor_update.stdout != ""
+  tags: deploy
+
 - name: devpi | ensure devpi is started
   supervisorctl: >
     state=started

playbooks/roles/discern/tasks/deploy.yml

@@ -107,36 +107,6 @@
   tags:
     - deploy
 
-  # call supervisorctl update. this reloads
-  # the supervisorctl config and restarts
-  # the services if any of the configurations
-  # have changed.
-  #
-  # we don't use notifications for supervisor because
-  # they don't work well with parameterized roles.
-  # See https://github.com/ansible/ansible/issues/4853
-- name: discern | update supervisor configuration
-  shell:  "{{ supervisor_ctl }} -c {{ supervisor_cfg }} update"
-  register: supervisor_update
-  sudo_user: "{{ supervisor_service_user }}"
-  changed_when: supervisor_update.stdout != ""
-
-- name: discern | ensure discern has started
-  supervisorctl: >
-    name=discern
-    supervisorctl_path={{ supervisor_ctl }}
-    config={{ supervisor_cfg }}
-    state=started
-  sudo_user: "{{ supervisor_service_user }}"
-
-- name: discern_celery | ensure discern_celery has started
-  supervisorctl: >
-    name=discern_celery
-    supervisorctl_path={{ supervisor_ctl }}
-    config={{ supervisor_cfg }}
-    state=started
-  sudo_user: "{{ supervisor_service_user }}"
-
 
   # call supervisorctl update. this reloads
   # the supervisorctl config and restarts
@@ -148,6 +118,7 @@
   register: supervisor_update
   sudo_user: "{{ supervisor_service_user }}"
   changed_when: supervisor_update.stdout != ""
+  tags: deploy
 
 - name: discern | ensure discern, discern_celery has started
   supervisorctl: >
@@ -158,9 +129,4 @@
   with_items:
   - discern
   - discern_celery
-
-- name: discern | create a symlink for venv python
-  file: >
-    src="{{ discern_venv_bin }}/python"
-    dest={{ COMMON_BIN_DIR }}/python.discern
-    state=link
+  tags: deploy

playbooks/roles/discern/tasks/main.yml

@@ -54,3 +54,9 @@
     - discern | restart discern
 
 - include: deploy.yml
+
+- name: discern | create a symlink for venv python
+  file: >
+    src="{{ discern_venv_bin }}/python"
+    dest={{ COMMON_BIN_DIR }}/python.discern
+    state=link