diff options
| -rw-r--r-- | roles/pihole/handlers/main.yml | 6 | ||||
| -rw-r--r-- | roles/pihole/tasks/main.yml | 14 |
2 files changed, 14 insertions, 6 deletions
diff --git a/roles/pihole/handlers/main.yml b/roles/pihole/handlers/main.yml index 87e77475..1cad8eff 100644 --- a/roles/pihole/handlers/main.yml +++ b/roles/pihole/handlers/main.yml @@ -1,10 +1,4 @@ --- -- name: "Restart pihole" - become: true - ansible.builtin.systemd_service: - name: "pihole-FTL" - state: "restarted" - - name: "Restart systemd-resolved" become: true ansible.builtin.systemd_service: diff --git a/roles/pihole/tasks/main.yml b/roles/pihole/tasks/main.yml index 9757bd0e..3184860d 100644 --- a/roles/pihole/tasks/main.yml +++ b/roles/pihole/tasks/main.yml @@ -136,6 +136,20 @@ state: "link" force: true +- name: "Flush handlers before restarting pihole-FTL" + # systemd-resolved must release port 53 before pihole-FTL binds it. + # flush_handlers here makes that ordering a structural guarantee rather than + # relying on handler definition order. + ansible.builtin.meta: flush_handlers + +- name: "Restart pihole-FTL" + become: true + ansible.builtin.systemd_service: + name: "pihole-FTL" + state: "restarted" + tags: + - molecule-idempotence-notest + - name: "Create pihole nginx configurations" become: true when: pihole_by_nginx is defined and pihole_by_nginx != "" |