Merge pull request #9 from EDITD/8-12.04

8 12.04

README.md

@@ -2,6 +2,8 @@
 
 A role for creating supervisor tasks.
 
+**NOTE**: This role depends on facts being gathered and will fail when `gather_facts` is set to false/no.
+
 
 ## Actions
 
@@ -52,6 +54,10 @@ to gracefully restart their workers. An example of how to do that would be:
       signal_task: HUP
 ```
 
+Unfortunately versions 3.2.0+ of supervisor are not compatible with Ubuntu 12.04,
+in which case an older supervisor version is installed instead, without signaling support.
+In these systems tasks will always be restarted, even when `restart: no` is passed to the role.
+
 ## License
 
 MIT

tasks/main.yml

@@ -51,10 +51,10 @@
 
 - name: Restart task {{ name }}
   command: "{{ supervisorctl_command }} restart {{ name }}:*"
-  when: restart_task
+  when: ansible_distribution_version == "12.04" or restart_task
   sudo: yes
 
-- name: Send signal to task {{ name }}
+- name: Send {{ signal_task }} signal to task {{ name }}
   command: "{{ supervisorctl_command }} signal {{ signal_task }} {{ name }}:*"
-  when: signal_task is defined
+  when: ansible_distribution_version != "12.04" and signal_task is defined
   sudo: yes

vars/12.04.yml

@@ -1,2 +1,2 @@
 supervisor_deb_file: "supervisor_3.0a8-1.1_all.deb"
-supervisor_version: "3.0a8-1.1"
+supervisor_version: "3.0a8-1.1"