sap_*_preconfigure, sap_ha_pacemaker_cluster: Vars loading reworked, Linting fixes for yes/no

[marcelmamula]

Description:

1. Add vars file loading in line with linux-system-roles to load in order and remove duplicated variables.
2. Correct linting errors of yes, no across preconfigure roles
3. Removed expect dependency from sap_ha_pacemaker_cluster role for Suse post tasks.
4. Ensure fapolicyd is installed only on RedHat in sap_hana_install

# Load variable files in order:
# Example for SUSE Linux Enterprise Server for SAP Applications 15 SP6:
# 1. Suse.yml - Specific to OS family.
# 2. SLES_15.yml - Specific to distribution (SLES and SLES_SAP) and major release.
# 3. SLES_15.6.yml - Specific to distribution (SLES and SLES_SAP) and minor release.
# 4. SLES_SAP_15.yml - Specific to distribution SLES_SAP and major release.
# 5. SLES_SAP_15.6.yml - Specific to distribution SLES_SAP and minor release.

Tested

Tested end to end SAP HANA HA and SAP ASCS/ERS cluster on SLES4SAP 15 SP6 on AWS and Google Cloud.

Next steps

• @berndfinger Please check RedHat var files to simplify them in separate PR as we discussed: Put shared parameters into RedHat.yml and version specific in their own files.

[berndfinger]

LGTM for the RHEL related and non-HA parts.

[ja9fuchs]

All changes look fine.
Cluster setup tested and all good.
Vars loading tested and the output is looking good now too.
👍

[sean-freeman]

Not that it's needed to be said, but - this is a very good cleanup commit.