Merge "Fixes for baremetal compute commands"

ansible/baremetal-compute-inspect.yml

@@ -8,7 +8,7 @@
 
 - name: Ensure dependencies are installed and the virtual environment is activated
   hosts: controllers[0]
-  gather_facts: False
+  gather_facts: true
   vars:
     venv: "{{ virtualenv_path }}/shade"
   roles:

ansible/baremetal-compute-manage.yml

@@ -8,7 +8,7 @@
 
 - name: Ensure baremetal compute nodes are available in ironic
   hosts: controllers[0]
-  gather_facts: False
+  gather_facts: true
   vars:
     venv: "{{ virtualenv_path }}/shade"
   roles:

ansible/baremetal-compute-provide.yml

@@ -8,7 +8,7 @@
 
 - name: Ensure baremetal compute nodes are available in ironic
   hosts: controllers[0]
-  gather_facts: False
+  gather_facts: true
   vars:
     venv: "{{ virtualenv_path }}/shade"
   roles:

ansible/baremetal-compute-rename.yml

@@ -54,5 +54,8 @@
         # be respected when using delegate_to.
         ansible_host: "{{ hostvars[controller_host].ansible_host | default(controller_host) }}"
         ipmi_address: "{{ hostvars[inventory_hostname].ipmi_address }}"
-        node: "{{ (nodes.stdout | from_json) | selectattr('Driver Info.ipmi_address', 'equalto', ipmi_address) | first }}"
-      when: node['Name'] != inventory_hostname
+        matching_nodes: "{{ (nodes.stdout | from_json) | selectattr('Driver Info.ipmi_address', 'defined') | selectattr('Driver Info.ipmi_address', 'equalto', ipmi_address) | list }}"
+        node: "{{ matching_nodes | first }}"
+      when:
+        - matching_nodes | length > 0
+        - node['Name'] != inventory_hostname