diff options
| author | Ahmed Abdelhalim <[email protected]> | 2026-06-12 12:15:41 +0200 |
|---|---|---|
| committer | Ahmed Abdelhalim <[email protected]> | 2026-06-12 12:17:03 +0200 |
| commit | e1616671e2d9e604b51464d9ab24ed66f9452f5b (patch) | |
| tree | 94ef823275900c20677407add0b60f133be1fdc0 | |
| parent | fc2e066d65cbbf8839ce9a101e1b44408090ff74 (diff) | |
Fix handler order issue in releasing port 53 bind
| -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 != "" |