Skip to content

sap*preconfigure, sap_hana_install, sap_maintain_etc_hosts and sap_install_media_detect: Replace injected vars with ansible_facts#1182

Draft
berndfinger wants to merge 9 commits intosap-linuxlab:devfrom
berndfinger:INJECT_FACTS_AS_VARS
Draft

sap*preconfigure, sap_hana_install, sap_maintain_etc_hosts and sap_install_media_detect: Replace injected vars with ansible_facts#1182
berndfinger wants to merge 9 commits intosap-linuxlab:devfrom
berndfinger:INJECT_FACTS_AS_VARS

Conversation

@berndfinger
Copy link
Member

Roles:

  • sap_general_preconfigure
  • sap_netweaver_preconfigure
  • sap_hana_preconfigure
  • sap_maintain_etc_hosts
  • sap_install_media_detect
  • sap_hana_install

Changes:

  • ansible_architecture -> ansible_facts['architecture']
  • ansible_os_family -> ansible_facts['os_family']
  • ansible_distribution_major_version -> ansible_facts['distribution_major_version']
  • ansible_distribution_version -> ansible_facts['distribution_version']
  • ansible_distribution -> ansible_facts['distribution']
  • ansible_memtotal_mb -> ansible_facts['memtotal_mb']
  • ansible_swaptotal_mb -> ansible_facts['swaptotal_mb']
  • ansible_interfaces -> ansible_facts['interfaces']
  • ansible_all_ipv4_addresses -> ansible_facts['all_ipv4_addresses']
  • ansible_default_ipv4 -> ansible_facts['default_ipv4']
  • ansible_hostname -> ansible_facts['hostname']
  • ansible_domain -> ansible_facts['domain']
  • ansible_fqdn -> ansible_facts['fqdn']
  • ansible_mounts -> ansible_facts['mounts']
  • ansible_selinux -> ansible_facts['selinux']

plus some necessary follow-up replacements (quotes/double quotes)

Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Signed-off-by: Bernd Finger <bfinger@redhat.com>
@berndfinger berndfinger marked this pull request as draft March 17, 2026 10:59
Signed-off-by: Bernd Finger <bfinger@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant