sap_maintain_etc_hosts: Wrong assert message

[berndfinger]

In

community.sap_install/roles/sap_maintain_etc_hosts/tasks/update_host_present.yml

Line 22 in 5a790ac

- name: Verify that variable domain_name is set

, the following message is displayed:

community.sap_install/roles/sap_maintain_etc_hosts/tasks/update_host_present.yml

Line 28 in 5a790ac

msg: "Variable 'domain_name' is undefined or empty. Please define it your host list"

However, the assertion checks for the variable named __sap_maintain_etc_hosts_domain. This is set in:

community.sap_install/roles/sap_maintain_etc_hosts/tasks/update_host_present.yml

Line 19 in 5a790ac

__sap_maintain_etc_hosts_domain: "{{ thishost.node_domain | default(sap_domain) | default(ansible_domain) }}"

to be one of:

• thishost.node_domain
• sap_domain
• ansible_domain

but not domain_name.

An updated message should take into account the case of this role being called from another role, so maybe it needs to be more generic.

[berndfinger]

Solved by #668.