openstack/ansible-collection-kolla
Code Issues Proposed changes

Merge "Allow creating /etc/hosts without Ansible facts"

Browse Source
This commit is contained in:
Zuul
2025-09-26 17:22:09 +00:00
committed by Gerrit Code Review
parent 53f62d9976 138418d7b7
commit cebacc6019
2 changed files with 20 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
releasenotes/notes/etc-hosts-no-facts-0b3a1a6fa15d3b2f.yaml Normal file
View File

@@ -0,0 +1,9 @@
---
features:
- |
Allow creating ``/etc/hosts`` on all hosts when Ansible facts are not
gathered for all hosts. The IP address for a host in ``/etc/hosts``
may be hard-coded by setting ``etc_hosts_api_address`` in the inventory
for each host, and when there are no facts available for a host,
the FQDN and short hostname are generated from the Ansible
magic-variables ``inventory_hostname`` and ``inventory_hostname_short``.

13
roles/etc_hosts/tasks/etc-hosts.yml
View File

@@ -30,8 +30,17 @@
{% for host in groups['baremetal'] %}
{% set api_interface = hostvars[host]['api_interface'] | replace('-', '_') %}
{% if host not in groups['bifrost'] or api_interface in hostvars[host].ansible_facts %}
{% set hostnames = [hostvars[host].ansible_facts.nodename, hostvars[host].ansible_facts.hostname] %}
{{ 'api' | kolla_address(host) }} {{ hostnames | unique | join(' ') }}
{% set hostnames =
[
hostvars[host].ansible_facts.nodename
if hostvars[host].ansible_facts.nodename is defined
else hostvars[host].inventory_hostname,
hostvars[host].ansible_facts.hostname
if hostvars[host].ansible_facts.hostname is defined
else hostvars[host].inventory_hostname_short
]
%}
{{ 'api' | kolla_address(host, override_var="etc_hosts_api_address") }} {{ hostnames | unique | join(' ') }}
{% endif %}
{% endfor %}
become: true