Use systemd for service control

[SuperQ]

Since the role already assumes systemd service units, it would be better to declare them with the systemd rather than service. The service module does not understand or automatically run daemon-reload.

[paulfantom]

We are doing it, look here: https://github.com/cloudalchemy/ansible-prometheus/blob/master/handlers/main.yml
However currently there is a bug in ansible systemd module, which causes failures in idempotency testing when enabling service, that's why we use service module here: https://github.com/cloudalchemy/ansible-prometheus/blob/master/tasks/main.yml#L8

[SuperQ]

I haven't noticed any issues with:

- name: Enable and start prometheus service
  systemd:
    name: prometheus.service
    state: started
    enabled: yes
    daemon_reload: yes

Is there an upstream bug?

[paulfantom]

We had some issues related to systemd module in our fluentd role, I need to consult it with @jkrol2 since he said there is something wrong.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.