From 3aa5aefb1bddbbac7fbb22ce11fd7426859c628c Mon Sep 17 00:00:00 2001
From: Dmitriy Rabotyagov <dmitriy.rabotyagov@cleura.com>
Date: Tue, 11 Apr 2023 12:50:24 +0200
Subject: [PATCH] Ensure service is restarted on unit file changes

At the moment we don't restart services if systemd unit file is changed.

We knowingly prevent systemd_service role handlers to execute
by providing `state: started` as otherwise service will be restarted twice.
With that now  we ensure that role handlers will also listen for systemd
unit changes.

Change-Id: I4ebae4853fc0bc2840d3ea79546f10a12051bea9
---
 defaults/main.yml | 1 +
 handlers/main.yml | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/defaults/main.yml b/defaults/main.yml
index ad8f516..a14fccc 100644
--- a/defaults/main.yml
+++ b/defaults/main.yml
@@ -142,6 +142,7 @@ tacker_services:
     group: tacker_server
     service_name: "{{ tacker_conductor_program_name }}"
     enabled: yes
+    state: started
     execstarts: "{{ tacker_bin }}/{{ tacker_conductor_program_name }} {{ tacker_conductor_config_options|default('') }}"
     config_overrides: "{{ tacker_conductor_init_config_overrides }}"
 
diff --git a/handlers/main.yml b/handlers/main.yml
index 6cad2bd..87bb815 100644
--- a/handlers/main.yml
+++ b/handlers/main.yml
@@ -17,9 +17,13 @@
 
 - name: Restart tacker services
   service:
-    name: "{{ tacker_service_name }}"
+    name: "{{ item }}"
     enabled: yes
     state: restarted
     daemon_reload: yes
+  with_items:
+    - "{{ tacker_service_name }}"
+    - "{{ tacker_conductor_program_name }}"
   listen:
     - "venv changed"
+    - "systemd service changed"