Skip to content
This repository was archived by the owner on Feb 13, 2023. It is now read-only.

Commit a096a56

Browse files
committed
Update Firewall role to latest version - fix idempotence.
1 parent 6c5bc1f commit a096a56

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

provisioning/requirements.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@
2626
- src: geerlingguy.elasticsearch
2727
version: 2.1.3
2828
- src: geerlingguy.firewall
29-
version: 2.1.1
29+
version: 2.1.2
3030
- src: geerlingguy.git
3131
version: 1.2.1
3232
- src: geerlingguy.java

provisioning/roles/geerlingguy.firewall/tasks/disable-other-firewalls.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
shell: yum list installed firewalld
44
register: firewalld_installed
55
ignore_errors: true
6+
changed_when: false
67
when: ansible_os_family == "RedHat" and firewall_disable_firewalld
78

89
- name: Disable the firewalld service (on RHEL, if configured).
@@ -16,6 +17,7 @@
1617
shell: dpkg -l ufw
1718
register: ufw_installed
1819
ignore_errors: true
20+
changed_when: false
1921
when: ansible_distribution == "Ubuntu" and firewall_disable_ufw
2022

2123
- name: Disable the ufw firewall (on Ubuntu, if configured).

0 commit comments

Comments
 (0)