Fixed ansible-lint problems

.ansible-lint

@@ -1,2 +1,4 @@
 ---
 strict: true
+exclude_paths:
+  - .github/

defaults/main.yml

@@ -1,14 +1,14 @@
 ---
 # Content for the /etc/iptables/rules.v4 file. If set to empty string, that file
 # won't be created
-rules_v4: ""
+iptables_rules_v4: ""
 
 # Content for the /etc/iptables/rules.v6 file. If set to empty string, that file
 # won't be created
-rules_v6: ""
+iptables_rules_v6: ""
 
 # Whether or not to reload the iptables rules if changed
-reload_rules_if_changed: true
+iptables_rules_reload_if_changed: true
 
 # List of services that must be restarted if the iptables rules change
-restart_services: []
+iptables_rules_restart_services: []

handlers/main.yml

@@ -1,6 +1,6 @@
 ---
 - name: Iptables IPv4 reset
-  when: reload_rules_if_changed
+  when: iptables_rules_reload_if_changed
   block:
     - name: Set the iptables policy for the filter:INPUT chain to ACCEPT
       ansible.builtin.iptables:
@@ -36,7 +36,7 @@
       listen: Reload iptables rules and restart services
 
 - name: Iptables IPv6 reset
-  when: reload_rules_if_changed
+  when: iptables_rules_reload_if_changed
   block:
     - name: Set the iptables IPv6 policy for the filter:INPUT chain to ACCEPT
       ansible.builtin.iptables:
@@ -81,11 +81,11 @@
     name: netfilter-persistent
     state: restarted
   listen: Reload iptables rules and restart services
-  when: reload_rules_if_changed
+  when: iptables_rules_reload_if_changed
 
 - name: Restart other services
   ansible.builtin.service:
     name: "{{ item }}"
     state: restarted
-  loop: "{{ restart_services }}"
+  loop: "{{ iptables_rules_restart_services }}"
   listen: Reload iptables rules and restart services

tasks/main.yml

@@ -28,15 +28,15 @@
     src: rules_v4.j2
     dest: /etc/iptables/rules.v4
     mode: "0644"
-  when: rules_v4 != ""
+  when: iptables_rules_v4 != ""
   notify: Reload iptables rules and restart services
 
 - name: Create the rules.v6 file
   ansible.builtin.template:
     src: rules_v6.j2
     dest: /etc/iptables/rules.v6
     mode: "0644"
-  when: rules_v6 != ""
+  when: iptables_rules_v6 != ""
   notify: Reload iptables rules and restart services
 
 - name: Force all notified handlers to run at this point

templates/rules_v4.j2

@@ -1 +1 @@
-{{ rules_v4 | regex_replace('\r\n', '\n') }}
+{{ iptables_rules_v4 | regex_replace('\r\n', '\n') }}

templates/rules_v6.j2

@@ -1 +1 @@
-{{ rules_v6 | regex_replace('\r\n', '\n') }}
+{{ iptables_rules_v6 | regex_replace('\r\n', '\n') }}

test/playbook.yml

@@ -6,8 +6,8 @@
   tasks:
     - name: Include the role from the parent directory
       ansible.builtin.include_role: { name: "{{ playbook_dir | dirname }}" }
-      vars:
+      vars: # noqa: var-naming[no-role-prefix]
         ansible_become: true
-        rules_v4: "{{ lookup('ansible.builtin.file', 'rules.v4') }}"
-        reload_rules_if_changed: false
-        restart_services: [docker]
+        iptables_rules_v4: "{{ lookup('ansible.builtin.file', 'rules.v4') }}"
+        iptables_rules_reload_if_changed: false
+        iptables_rules_restart_services: [docker]