From 47a0a37e1d10958213b621172a22f0e04610c5b4 Mon Sep 17 00:00:00 2001
From: Baptiste DE RENZO <baptiste.derenzo@ubicast.eu>
Date: Tue, 11 Apr 2023 08:31:13 +0000
Subject: [PATCH] Fix lint, Refs #37692

---
 roles/lxc/tasks/main.yml            | 2 +-
 roles/mediaserver/handlers/main.yml | 4 ----
 roles/mediaserver/tasks/main.yml    | 3 ++-
 roles/postfix/handlers/main.yml     | 8 ++++++++
 roles/sysconfig/handlers/main.yml   | 2 ++
 5 files changed, 13 insertions(+), 6 deletions(-)

diff --git a/roles/lxc/tasks/main.yml b/roles/lxc/tasks/main.yml
index 406bc194..c758b5e4 100644
--- a/roles/lxc/tasks/main.yml
+++ b/roles/lxc/tasks/main.yml
@@ -21,7 +21,7 @@
         src: lxc-net.host_bridge
         dest: /etc/default/lxc-net
         mode: "644"
- 
+
     - name: Ask confirmation
       ansible.builtin.pause:
         prompt: |
diff --git a/roles/mediaserver/handlers/main.yml b/roles/mediaserver/handlers/main.yml
index 93ab5aea..b7d5c45a 100644
--- a/roles/mediaserver/handlers/main.yml
+++ b/roles/mediaserver/handlers/main.yml
@@ -1,8 +1,4 @@
 ---
-- name: mscontroller restart
-  ansible.builtin.command:
-    cmd: mscontroller.py restart
-
 - name: restart nginx
   ansible.builtin.systemd:
     name: nginx
diff --git a/roles/mediaserver/tasks/main.yml b/roles/mediaserver/tasks/main.yml
index 08990cb0..09abea00 100644
--- a/roles/mediaserver/tasks/main.yml
+++ b/roles/mediaserver/tasks/main.yml
@@ -116,7 +116,8 @@
   tags: mediaserver-synchronize
 
 - name: configure email sender address
-  notify: mscontroller restart
+  notify:
+    - restart mediaserver
   ansible.builtin.lineinfile:
     path: /etc/mediaserver/msconf.py
     backup: true
diff --git a/roles/postfix/handlers/main.yml b/roles/postfix/handlers/main.yml
index 56cbbb41..a92f8494 100644
--- a/roles/postfix/handlers/main.yml
+++ b/roles/postfix/handlers/main.yml
@@ -1,15 +1,23 @@
 ---
 - name: postmap sasl
   ansible.builtin.command: postmap hash:/etc/postfix/sasl-passwords
+  # we want to generate db file at every execution
+  changed_when: false
 
 - name: postmap generic
   ansible.builtin.command: postmap hash:/etc/postfix/generic
+  # we want to generate db file at every execution
+  changed_when: false
 
 - name: postmap virtual
   ansible.builtin.command: postmap hash:/etc/postfix/virtual
+  # we want to generate db file at every execution
+  changed_when: false
 
 - name: newaliases
   ansible.builtin.command: newaliases
+  # we want to generate aliases at every execution
+  changed_when: false
 
 - name: restart postfix
   ansible.builtin.service:
diff --git a/roles/sysconfig/handlers/main.yml b/roles/sysconfig/handlers/main.yml
index a565a463..fbb96d46 100644
--- a/roles/sysconfig/handlers/main.yml
+++ b/roles/sysconfig/handlers/main.yml
@@ -14,6 +14,8 @@
 
 - name: update locale
   ansible.builtin.command: locale-gen
+  # we want to execute locale generation at every execution
+  changed_when: false
 
 - name: restart cron
   ansible.builtin.service:
-- 
GitLab