From 7dbc61553b3bf819e21c5f1aaad51bc8b0757a0c Mon Sep 17 00:00:00 2001 From: Lee Armbuster <80494138+larmbuster@users.noreply.github.com> Date: Wed, 22 May 2024 16:41:01 -0400 Subject: [PATCH 1/2] Update README.md Fixes connection error to ansible-1 due to firewalld being installed and started without proper configuration. Added when conditional to run all the firewall tasks only on the web group. --- exercises/ansible_rhel/1.5-handlers/README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/exercises/ansible_rhel/1.5-handlers/README.md b/exercises/ansible_rhel/1.5-handlers/README.md index b6a5bd0c9..6aaf7b9f9 100644 --- a/exercises/ansible_rhel/1.5-handlers/README.md +++ b/exercises/ansible_rhel/1.5-handlers/README.md @@ -89,12 +89,14 @@ Let's say we want to ensure the firewall is configured correctly on all web serv ansible.builtin.dnf: name: firewalld state: present + when: inventory_hostname in groups['web'] - name: Ensure firewalld is running ansible.builtin.service: name: firewalld state: started enabled: true + when: inventory_hostname in groups['web'] - name: Allow HTTPS traffic on web servers ansible.posix.firewalld: From 466299c5718a1fe96244526c71ffd8288854ac33 Mon Sep 17 00:00:00 2001 From: Lee Armbuster <80494138+larmbuster@users.noreply.github.com> Date: Wed, 22 May 2024 16:43:45 -0400 Subject: [PATCH 2/2] Update README.md Update example playbook output to reflect playbook changes --- exercises/ansible_rhel/1.5-handlers/README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/exercises/ansible_rhel/1.5-handlers/README.md b/exercises/ansible_rhel/1.5-handlers/README.md index 6aaf7b9f9..25bd4d1a4 100644 --- a/exercises/ansible_rhel/1.5-handlers/README.md +++ b/exercises/ansible_rhel/1.5-handlers/README.md @@ -146,14 +146,14 @@ ok: [node1] ok: [node3] TASK [Install firewalld] ******************************************************* -changed: [ansible-1] +skipping: [ansible-1] changed: [node2] changed: [node1] changed: [node3] TASK [Ensure firewalld is running] ********************************************* +skipping: [ansible-1] changed: [node3] -changed: [ansible-1] changed: [node2] changed: [node1]