From e9fab281bd738cc2271f7719389474fd9c9157d4 Mon Sep 17 00:00:00 2001 From: Dmitriy Rabotyagov Date: Mon, 10 Apr 2023 16:05:42 +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: Ia9d1164e1e38201244a062be95f936b314c5c56b --- handlers/main.yml | 1 + vars/main.yml | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/handlers/main.yml b/handlers/main.yml index 44d4d41c..9cfd70d2 100644 --- a/handlers/main.yml +++ b/handlers/main.yml @@ -22,6 +22,7 @@ with_list: "{{ filtered_ironic_services }}" listen: - "venv changed" + - "systemd service changed" - name: Restart tftpd service: diff --git a/vars/main.yml b/vars/main.yml index 208704cc..4097ad3d 100644 --- a/vars/main.yml +++ b/vars/main.yml @@ -262,8 +262,8 @@ filtered_ironic_services: |- {% set _ = value.update( { 'service_key': key, - 'enabled': 'yes', - 'state': 'started', + 'enabled': value['enabled'] | default(True), + 'state': value['state'] | default('started'), 'config_overrides': value.init_config_overrides } )