It looks like os-cloud-config is mapping pm_addr to ucs_hostname:

https://github.com/openstack/os-cloud-config/blob/master/os_cloud_config/nodes.py#L84

However, in ironic, the ucs_helper requires ucs_address to be set. There doesn't seem to be a param called ucs_hostname anymore:

https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ucs/helper.py#L97

Got a report of this happening in https://bugzilla.redhat.com/show_bug.cgi?id=1274579

(In reply to Dan Macpherson from comment #2)
> It looks like os-cloud-config is mapping pm_addr to ucs_hostname:
> 
> https://github.com/openstack/os-cloud-config/blob/master/os_cloud_config/
> nodes.py#L84
> 
> However, in ironic, the ucs_helper requires ucs_address to be set. There
> doesn't seem to be a param called ucs_hostname anymore:
> 
> https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/ucs/
> helper.py#L97
> 
> Got a report of this happening in
> https://bugzilla.redhat.com/show_bug.cgi?id=1274579

Right, that seems to be the problem indeed. I'm reassign this bug to the os-cloud-config component since the error is there.

This bug did not make the OSP 8.0 release.  It is being deferred to OSP 10.

FWIW I've upload a fix for this upstream: https://review.openstack.org/#/c/370773/

Backported to rhosp8 and 9 as well

https://github.com/openstack/os-cloud-config/blob/stable/newton/os_cloud_config/nodes.py

Newton still seems to be ucs_hostname and not ucs_address in spite of https://review.openstack.org/#/c/370773/2/os_cloud_config/nodes.py

(Not in master either) , seems the patch did not make it so we should re-open this BZ.

Moving to 8. I've prepared downstream backports for both 8 and 9, but I will need OtherQA on them.

*** Bug 1417302 has been marked as a duplicate of this bug. ***