Don't use ansible_domain if empty string

Also removes `ci-rdo.local` added to `hostName` in nodeset spec.
openstack-k8s-operators · Aug 12, 2024 · befbd52 · befbd52
1 parent fb1634e
commit befbd52
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/hooks/playbooks/fetch_compute_facts.yml b/hooks/playbooks/fetch_compute_facts.yml
@@ -139,7 +139,7 @@
               {% for compute_node in groups['computes'] %}
                   - op: replace
                     path: /spec/nodes/edpm-{{ compute_node }}/hostName
-                    value: "{{compute_node}}.ci-rdo.local"
+                    value: "{{compute_node}}"
               {% endfor %}
                   - op: replace
                     path: /spec/nodeTemplate/ansible/ansibleVars/neutron_public_interface_name

diff --git a/playbooks/nfs.yml b/playbooks/nfs.yml
@@ -72,7 +72,7 @@
         PATH: "{{ cifmw_path }}"
       vars:
         _nfs_network_name: "{{ cifmw_nfs_network | default('storage') }}"
-        _nfs_host: "{{ [groups[cifmw_nfs_target | default('computes')][0], ansible_domain] | join('.') | default('') }}"
+        _nfs_host: "{{ [groups[cifmw_nfs_target | default('computes')][0], ansible_domain] | select() | join('.') | default('') }}"
         _ipset_namespace: "{{ cifmw_install_yamls_defaults['NAMESPACE'] | default('openstack') }}"
       ansible.builtin.command:
         cmd: oc get ipset {{ _nfs_host }} -n {{ _ipset_namespace }} -o jsonpath='{.status.reservations[?(@.network=="{{ _nfs_network_name }}")]}'