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:
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.
Change abandoned by Lee Yarwood (<email address hidden>) on branch: master /review. openstack. org/634398
Review: https:/
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/ 5a4863aa152f58f 6de426b3908a75c 2cc1b2efca/ nova/virt/ libvirt/ driver. py#L7970- L7976
https:/ /github. com/openstack/ nova/blob/ 5a4863aa152f58f 6de426b3908a75c 2cc1b2efca/ 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.