Configure ironic Inspection hooks to manage port and node attributes

After out-of-band inspection we run port-bios-name.

The interface names returned by the BMC are generally more meaningful to
the data center than the linux names we get in the agent inspection data

After Agent Inspection we run update-baremetal-port.

This consumes the LLDP information, so it must run after agent
inspection.  Out-of-band inspection doesn't give us the full LLDP data.

The default ironic local-link-connection is removed because the new hook
updates the same fields.

We add "validate-interfaces" because that populates data we consume.

Author: stevekeay

components/ironic/values.yaml

@@ -92,7 +92,10 @@ conf:
       # Default hooks include: ramdisk-error,validate-interfaces,ports,architecture
       # We override to exclude 'ports' from the default hooks
       default_hooks: "ramdisk-error,validate-interfaces,architecture"
-      hooks: "$default_hooks,pci-devices,parse-lldp,local-link-connection,resource-class"
+      hooks: "$default_hooks,pci-devices,validate-interfaces,parse-lldp,resource-class,update-baremetal-port"
+    redfish:
+      inspection_hooks: "ramdisk-error,validate-interfaces,ports,port-bios-name,architecture,pci-devices"
+      add_ports: "all"
     # enable sensors and metrics for redfish metrics - https://docs.openstack.org/ironic/latest/admin/drivers/redfish/metrics.html
     sensor_data:
       send_sensor_data: true