diff options
| author | Egor Tensin <Egor.Tensin@gmail.com> | 2023-09-19 15:03:30 +0200 |
|---|---|---|
| committer | Egor Tensin <Egor.Tensin@gmail.com> | 2023-09-19 15:03:33 +0200 |
| commit | 3c55245c39d78dde94705f49ae1ec0d85c678190 (patch) | |
| tree | d888d72c9bedac7b483787f304a332c9a8998d44 /roles/systemd_depend_iface/handlers/main.yml | |
| parent | apt: _actually_ don't reboot after cleaning up dependencies (diff) | |
| download | infra-ansible-3c55245c39d78dde94705f49ae1ec0d85c678190.tar.gz infra-ansible-3c55245c39d78dde94705f49ae1ec0d85c678190.zip | |
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
Diffstat (limited to 'roles/systemd_depend_iface/handlers/main.yml')
| -rw-r--r-- | roles/systemd_depend_iface/handlers/main.yml | 13 |
1 files changed, 1 insertions, 12 deletions
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 |