Merge pull request #1232 from hjensas/ironic-reset-deploy-img

Add deploy image reset step to Ironic adoption

Author: openshift-merge-bot[bot]

docs_user/modules/proc_deploying-the-bare-metal-provisioning-service.adoc

@@ -203,3 +203,18 @@ $ openstack endpoint list |grep ironic
 ----
 $ openstack baremetal node list
 ----
+
+. Reset the deploy images on all bare-metal nodes to use the new centrally configured images:
++
+[NOTE]
+After adoption, bare-metal nodes might still reference the old deployment's kernel and ramdisk
+images in their `driver_info` fields. Resetting these values causes the {bare_metal} to use the
+new centrally configured  `deploy_kernel` and `deploy_ramdisk` values from the `ironic.conf` file.
++
+----
+for node in $(openstack baremetal node list -c UUID -f value); do
+  openstack baremetal node set $node \
+    --driver-info deploy_ramdisk= \
+    --driver-info deploy_kernel=
+done
+----

tests/roles/ironic_adoption/tasks/main.yaml

@@ -100,3 +100,15 @@
     {{ oc_header }}
     alias openstack="oc exec -t openstackclient -- openstack"
     ${BASH_ALIASES[openstack]} subnet set --dns-nameserver {{ dns_server_provisioning_ip }} provisioning-subnet
+
+- name: Reset node driver_info to use new deploy images
+  ansible.builtin.shell: |
+    {{ shell_header }}
+    {{ oc_header }}
+    alias openstack="oc exec -t openstackclient -- openstack"
+
+    for node in $(${BASH_ALIASES[openstack]} baremetal node list -c UUID -f value); do
+      ${BASH_ALIASES[openstack]} baremetal node set $node \
+        --driver-info deploy_ramdisk= \
+        --driver-info deploy_kernel=
+    done