Comment 2 for bug 1814245

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Lee Yarwood (<email address hidden>) on branch: master
Review: https://review.openstack.org/634398
Reason: As discussed this is wrong as instance.host isn't updated until later in the flow.

The actual issue is that the connection_info we have has just been pulled directly from Cinder API and doesn't include the multiattach key that Nova adds during the attach process:

https://github.com/openstack/nova/blob/5a4863aa152f58f6de426b3908a75c2cc1b2efca/nova/virt/libvirt/driver.py#L7970-L7976

https://github.com/openstack/nova/blob/5a4863aa152f58f6de426b3908a75c2cc1b2efca/nova/virt/block_device.py#L538-L549

Matt already has https://review.openstack.org/#/c/551302/ open that should correct this given it uses the existing connection_info for the volume that contains the multiattach key.