Changed the default value of ha_cluster_manage_firewall to true.

README.md

@@ -838,15 +838,13 @@ You may take a look at
 
 #### `ha_cluster_manage_firewall`
 
-boolean, default: undefined
+boolean, default: true
 
 Manage the `firewall high-availability service` as well as the `fence-virt port`.
 When `ha_cluster_manage_firewall` is `true`, the `firewall high-availability
 service` and `fence-virt port` are enabled.
 When `ha_cluster_manage_firewall` is `false`, the `firewall high-availability
 service` and `fence-virt port` are disabled.
-When `ha_cluster_manage_firewall` is undefined and `firewalld.service`
-is in `ansible_facts.services`, `ha_cluster_manage_firewall` is set to true.
 
 ### Inventory
 

defaults/main.yml

@@ -48,3 +48,5 @@ ha_cluster_constraints_location: []
 ha_cluster_constraints_colocation: []
 ha_cluster_constraints_order: []
 ha_cluster_constraints_ticket: []
+
+ha_cluster_manage_firewall: true

tasks/firewall.yml

@@ -1,15 +1,5 @@
 # SPDX-License-Identifier: MIT
 ---
-- name: Get services status - detect firewall
-  service_facts:
-
-- name: Set ha_cluster_manage_firewall to true if the service is already enabled
-  set_fact:
-    ha_cluster_manage_firewall: true
-  when:
-    - '"firewalld.service" in ansible_facts.services'
-    - ha_cluster_manage_firewall is not defined
-
 - block:
     - name: Initialize firewall option
       set_fact:
@@ -39,5 +29,3 @@
   vars:
     _state_value: "{{ 'enabled' if ha_cluster_manage_firewall | d() |
                    bool else 'disabled' }}"
-  when:
-    - ha_cluster_manage_firewall is defined