Merge pull request #11 from roumano/upstart

Fix #10 : restart node exporter : failed when ansible_service_mgr: upstart
UnderGreen · Apr 27, 2017 · d8f8516 · d8f8516
2 parents f43ca83 + f1c767f
commit d8f8516
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/tasks/main.yml b/tasks/main.yml
@@ -42,6 +42,11 @@
     - service reenable
     - restart node exporter
 
+- name: Looking which version of upstart is install
+  command: initctl version
+  when: ansible_service_mgr == 'upstart'
+  register: upstart_version
+
 - name: create init service unit
   template:
     src: etc/init/prometheus-node-exporter.conf.j2

diff --git a/templates/etc/init/prometheus-node-exporter.conf.j2 b/templates/etc/init/prometheus-node-exporter.conf.j2
@@ -3,8 +3,10 @@ start on (local-filesystems and net-device-up IFACE!=lo)
 stop on runlevel [016]
 
 respawn
+{% if upstart_version.stdout | replace("init (upstart ", "") |replace(")","") | version_compare('1.4', '>=') %}
 setuid root
 setgid {{ prometheus_exporters_common_group }}
+{% endif %}
 
 script
   exec >> "{{ prometheus_exporters_common_log_dir }}/node-exporter.log"