redhat-sap / sap-hana-deployment

Deploys SAP HANA on the given hosts
Apache License 2.0
13 stars 20 forks source link

SAP hostname is set to ansible_hostname #16

Open rhmk opened 4 years ago

rhmk commented 4 years ago

This will fail if ansible runs through a didicated admin interface instead of the public interface where the system is accessed through or some other reason. sap-preconfigure uses sap_hostname for it and it should be used here as well and just default to ansible_hostname in all other cases. change template to sth like {{ sap_hostname | default(ansible_hostname) }}

rickgcv commented 4 years ago

This has been included in PR#20