One of the issues with this patch is that it only handles SCSI
attached disks, for which reason this issue continues to occur when
having generation 1 VMs booted from volume, in which case the disk
will be placed on the IDE controller.
In this case, one instance may end up booting from another tenant's
volume, which is a critical security issue.
Also, it assumes that the block device info volume order matches
the according disk controller slot order, which is wrong.
This patch fixes this issue, looping through the block device
info and fixing the disk path for all the passthrough attached
disks, regardless of the disk controller.
This requires the virtual disk resources attached to the instance
to be tagged with the serial number.
Reviewed: https:/ /review. openstack. org/258615 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=9de0bec4f9c df0cd6424aeb95d aabe657d4db2d4
Committed: https:/
Submitter: Jenkins
Branch: master
commit 9de0bec4f9cdf0c d6424aeb95daabe 657d4db2d4
Author: Lucian Petrut <email address hidden>
Date: Thu Dec 10 19:23:08 2015 +0200
HyperV: Fix vm disk path issue
As the disk number of iSCSI attached disks can change after host
reboot, passthrough attached volumes can get attached in this case.
This bug was partially fixed during Icehouse by this patch: /review. openstack. org/95356
https:/
One of the issues with this patch is that it only handles SCSI
attached disks, for which reason this issue continues to occur when
having generation 1 VMs booted from volume, in which case the disk
will be placed on the IDE controller.
In this case, one instance may end up booting from another tenant's
volume, which is a critical security issue.
Also, it assumes that the block device info volume order matches
the according disk controller slot order, which is wrong.
This patch fixes this issue, looping through the block device
info and fixing the disk path for all the passthrough attached
disks, regardless of the disk controller.
This requires the virtual disk resources attached to the instance
to be tagged with the serial number.
Co-Authored-By: Alin Balutoiu <email address hidden>
Closes-Bug: #1526831 9cab4b99fe7967c cb5f2181958 ec120fce9898661 eb9a20ade48
Depends-On: Ibe463ce9ffb912
Change-Id: Icf8697d917c814