adjusting variable names, fixing errors

tasks/pkgs/firewalld.yml

@@ -1,5 +1,40 @@
 # vim: set filetype=yaml.ansible :
+#
+## Package: firewalld
+## Description: zone based firewall
+## Version: system
+## Methods: system
+## Helpers: -
 ---
-- name: Append firewalld to pkg_sys
+- name: Set firewalld default facts # {{{
   ansible.builtin.set_fact:
-    pkg_sys: "{{ pkg_sys + ['firewalld'] }}"
+    firewalld:
+      methods:
+        - system
+      pkgname: firewalld
+# }}}
+- name: Configure firewalld
+  when:
+    - "'firewalld' not in __configured"
+  block:
+    - name: Set firewalld install method
+      when:
+        - firewalld_imethod is undefined
+      ansible.builtin.set_fact:
+        firewalld_imethod: "{{ imethod if imethod in firewalld.methods else firewalld.methods[0] }}"
+
+    - name: Configure firewalld system install
+      when:
+        - firewalld_imethod == 'system'
+      block:
+        - name: Append firewalld to pkg_sys
+          ansible.builtin.set_fact:
+            pkg_sys: "{{ pkg_sys + [firewalld.pkgname] }}"
+
+        - name: Finalise firewalld system instal
+          ansible.builtin.set_fact:
+            firewalld_install: "{{ firewalld_imethod }}={{ firewalld.pkgname }}"
+
+    - name: Finalise firewalld configuration
+      ansible.builtin.set_fact:
+        __configured: "{{ __configured | combine( { 'firewalld': firwalld_install } ) }}"