From b6a54ca4d13ef2cf2817d7766d75dfc85ab728c9 Mon Sep 17 00:00:00 2001 From: Alan Orth Date: Thu, 2 Nov 2017 18:58:15 +0200 Subject: [PATCH] roles/common: Reload firewalld instead of restart There is no need to bounce the service, just reload it. --- roles/common/handlers/main.yml | 4 ++-- roles/common/tasks/firewall_Debian.yml | 2 +- roles/common/tasks/firewall_Ubuntu.yml | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/roles/common/handlers/main.yml b/roles/common/handlers/main.yml index c9b14fa..25f49b8 100644 --- a/roles/common/handlers/main.yml +++ b/roles/common/handlers/main.yml @@ -13,5 +13,5 @@ - name: reload sysctl command: sysctl -p /etc/sysctl.conf -- name: restart firewalld - systemd: name=firewalld state=restarted +- name: reload firewalld + systemd: name=firewalld state=reloaded diff --git a/roles/common/tasks/firewall_Debian.yml b/roles/common/tasks/firewall_Debian.yml index 89c2fdd..6fba26d 100644 --- a/roles/common/tasks/firewall_Debian.yml +++ b/roles/common/tasks/firewall_Debian.yml @@ -15,4 +15,4 @@ when: ansible_distribution_major_version | version_compare('8', '>=') shell: tidy -xml -iq -m -w 0 /etc/firewalld/zones/public.xml notify: - - restart firewalld + - reload firewalld diff --git a/roles/common/tasks/firewall_Ubuntu.yml b/roles/common/tasks/firewall_Ubuntu.yml index 39f7a9a..4f953b4 100644 --- a/roles/common/tasks/firewall_Ubuntu.yml +++ b/roles/common/tasks/firewall_Ubuntu.yml @@ -32,4 +32,4 @@ when: ansible_distribution_version | version_compare('15.04', '>=') shell: tidy -xml -iq -m -w 0 /etc/firewalld/zones/public.xml notify: - - restart firewalld + - reload firewalld