move firewall install tasks

ansible · Oct 2, 2023 · d428dc7 · d428dc7
1 parent b2e9999
commit d428dc7
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 11 deletions.
diff --git a/collections/ansible_collections/demo/patching/roles/report_server/tasks/apache.yml b/collections/ansible_collections/demo/patching/roles/report_server/tasks/apache.yml
@@ -2,6 +2,20 @@
 - name: Include system variables
   ansible.builtin.include_vars: "{{ ansible_system }}.yml"
 
+- name: Install firewall dependencies
+  ansible.builtin.dnf:
+    name:
+      - firewalld
+      - python3-firewall
+    state: present
+  check_mode: false
+
+- name: Start firewalld
+  ansible.builtin.service:
+    name: firewalld
+    state: started
+  check_mode: false
+
 - name: Permit traffic in default zone for http service
   ansible.posix.firewalld:
     service: http

diff --git a/linux/patching.yml b/linux/patching.yml
@@ -32,17 +32,6 @@
       delegate_to: "{{ report_server }}"
       run_once: true  # noqa: run-once[task]
       block:
-        - name: Install firewall dependencies
-          ansible.builtin.dnf:
-            name:
-              - firewalld
-              - python3-firewall
-            state: present
-
-        - name: Start firewalld
-          ansible.builtin.service:
-            name: firewalld
-            state: started
 
         - name: Build report server
           ansible.builtin.include_role: