From 3c55245c39d78dde94705f49ae1ec0d85c678190 Mon Sep 17 00:00:00 2001 From: Egor Tensin Date: Tue, 19 Sep 2023 15:03:30 +0200 Subject: systemd_depend_*: don't restart services First, I don't think it's even necessary. If the service wasn't running, then we don't need to restart it. If it was running, it'll continue running fine with a new dependency; it'll be picked up on a future restart. Second, all of that doesn't even work due to (another) weird way Ansible treats handlers: https://github.com/ansible/ansible/issues/81722 --- roles/systemd_depend_iface/handlers/main.yml | 13 +------------ 1 file changed, 1 insertion(+), 12 deletions(-) (limited to 'roles/systemd_depend_iface/handlers/main.yml') diff --git a/roles/systemd_depend_iface/handlers/main.yml b/roles/systemd_depend_iface/handlers/main.yml index 813a651..1449130 100644 --- a/roles/systemd_depend_iface/handlers/main.yml +++ b/roles/systemd_depend_iface/handlers/main.yml @@ -2,15 +2,4 @@ become: true ansible.builtin.systemd_service: daemon_reload: true - listen: systemd_depend_iface_restart - -- name: Collect installed services - ansible.builtin.service_facts: - listen: systemd_depend_iface_restart - -- name: Restart systemd services - ansible.builtin.include_tasks: service.yml - listen: systemd_depend_iface_restart - loop: "{{ systemd_depend_iface_result.results | selectattr('changed', 'equalto', true) | map(attribute='service') | list }}" - loop_control: - loop_var: service + listen: systemd_depend_iface_reload -- cgit v1.2.3