setting *_installed facts for successful runs with --skip-tags deploy

playbooks/roles/certs/tasks/deploy.yml

@@ -66,3 +66,13 @@
     config={{ supervisor_cfg }}
     state=started
   sudo_user: "{{ supervisor_service_user }}"
+
+- name: certs | create a symlink for venv python
+  file: >
+    src="{{ certs_venv_bin }}/{{ item }}"
+    dest={{ COMMON_BIN_DIR }}/{{ item }}.certs
+    state=link
+  notify: certs | restart certs
+  with_items:
+  - python
+  - pip

playbooks/roles/certs/tasks/main.yml

@@ -78,13 +78,3 @@
   notify: certs | restart certs
 
 - include: deploy.yml tags=deploy
-
-- name: certs | create a symlink for venv python
-  file: >
-    src="{{ certs_venv_bin }}/{{ item }}"
-    dest={{ COMMON_BIN_DIR }}/{{ item }}.certs
-    state=link
-  notify: certs | restart certs
-  with_items:
-  - python
-  - pip

playbooks/roles/discern/handlers/main.yml

@@ -5,6 +5,7 @@
     supervisorctl_path={{ supervisor_ctl }}
     config={{ supervisor_cfg }}
     state=restarted
+  when: discern_installed is defined
   with_items:
   - discern
   - discern_celery

playbooks/roles/discern/tasks/deploy.yml

@@ -120,3 +120,6 @@
     src="{{ discern_venv_bin }}/python"
     dest={{ COMMON_BIN_DIR }}/python.discern
     state=link
+
+- set_fact: discern_installed=true
+

playbooks/roles/edxapp/handlers/main.yml

@@ -5,7 +5,7 @@
     supervisorctl_path={{ supervisor_ctl }}
     config={{ supervisor_cfg }}
     name="edxapp:{{ item }}"
-  when: celery_worker is not defined and not devstack
+  when: edxapp_installed is defined and celery_worker is not defined and not devstack
   sudo_user: "{{ supervisor_service_user }}"
   with_items: service_variants_enabled
 
@@ -15,6 +15,6 @@
     supervisorctl_path={{ supervisor_ctl }}
     config={{ supervisor_cfg }}
     state=restarted
-  when: celery_worker is defined and not devstack
+  when: edxapp_installed is defined and celery_worker is defined and not devstack
   with_items: edxapp_workers
   sudo_user: "{{ common_web_user }}"

playbooks/roles/edxapp/tasks/deploy.yml

@@ -295,3 +295,5 @@
   when: celery_worker is defined and not devstack
   with_items: edxapp_workers
   sudo_user: "{{ supervisor_service_user }}"
+
+- set_fact: edxapp_installed=true

playbooks/roles/forum/handlers/main.yml

@@ -5,4 +5,4 @@
     supervisorctl_path={{ supervisor_ctl }}
     config={{ supervisor_cfg }}
     state=restarted
-  when: not devstack
+  when: forum_installed is defined and not devstack

playbooks/roles/forum/tasks/deploy.yml

@@ -50,3 +50,5 @@
     config={{ supervisor_cfg }}
     state=started
   when: not devstack
+
+- set_fact: forum_installed=true