sap_ha_pacemaker_cluster: fix: avoid optional parameter reference in …

…defaults
sap-linuxlab · Sep 26, 2023 · 80a4bf8 · 80a4bf8
1 parent e85c390
commit 80a4bf8
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/playbooks/sample-sap-nwas-ascs-ers-2-node-cluster.yml b/playbooks/sample-sap-nwas-ascs-ers-2-node-cluster.yml
@@ -27,6 +27,11 @@
     ############################################################################
 
     ### General minimum pacemaker cluster definitions.
+
+    # Cluster name is optional, the default of the 'ha_cluster' Linux System Role
+    # would be applied when missing.
+    # However, for convenience it is defined along with the rest of the mandatory
+    # parameters in this example.
     sap_ha_pacemaker_cluster_cluster_name: ascs-ers-cluster
     sap_ha_pacemaker_cluster_hacluster_password: my-hacluster-vault-password
 

diff --git a/roles/sap_ha_pacemaker_cluster/defaults/main.yml b/roles/sap_ha_pacemaker_cluster/defaults/main.yml
@@ -21,7 +21,7 @@ sap_ha_pacemaker_cluster_system_roles_collection: 'fedora.linux_system_roles'
 # config file.
 # Useful for parameter review or re-use with the 'ha_cluster' LSR.
 sap_ha_pacemaker_cluster_create_config_varfile: false
-sap_ha_pacemaker_cluster_create_config_dest: "{{ sap_ha_pacemaker_cluster_cluster_name }}_resource_config.yml"
+sap_ha_pacemaker_cluster_create_config_dest: "review_resource_config.yml"
 
 # Inherit SAP common (global synonyms) parameters when defined.