This commit aims to correct problems with the resize_up codebase that allows
the snapshot and migration of volume backed VDI/VHDs. Since these are empty
stub disks, and the XenAPI does not allow these VDIs to be snapped, this results
in an SR_OPERATION_NOT_ALLOWED or similar error on attempt.
This change adds a check into the _process_ephemeral_chain_recursive method to
run the current userdevice through volume_utils.is_booted_from_volume. To
achieve this, the method has been opened in scope to accept custom user_device
objects. In a future commit we will need to rename this method for clarity
and correct its dependancies that call it. I have added a TODO for this to be
done by myself. The check will ensure that the userdevice is not volume backed
and then continue to snapshot and migrate the disk as needed, else increment
and move on.
Closes-Bug: #1745072
Change-Id: I7cd2977c8268c1f73062b5d0b2b68ea686db99fe
(cherry picked from commit eefb20e4658e17f91fa76b74fef6ff899babe51b)
Reviewed: https:/ /review. openstack. org/604203 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=dba00dbe113 61a12d2602dd960 1ee365a0755051
Committed: https:/
Submitter: Zuul
Branch: stable/rocky
commit dba00dbe11361a1 2d2602dd9601ee3 65a0755051
Author: Brooks Kaminski <email address hidden>
Date: Fri Jan 12 06:05:36 2018 -0600
XenAPI/Stops the migration of volume backed VHDS
This commit aims to correct problems with the resize_up codebase that allows NOT_ALLOWED or similar error on attempt.
the snapshot and migration of volume backed VDI/VHDs. Since these are empty
stub disks, and the XenAPI does not allow these VDIs to be snapped, this results
in an SR_OPERATION_
This change adds a check into the _process_ ephemeral_ chain_recursive method to utils.is_ booted_ from_volume. To
run the current userdevice through volume_
achieve this, the method has been opened in scope to accept custom user_device
objects. In a future commit we will need to rename this method for clarity
and correct its dependancies that call it. I have added a TODO for this to be
done by myself. The check will ensure that the userdevice is not volume backed
and then continue to snapshot and migrate the disk as needed, else increment
and move on.
Closes-Bug: #1745072 f73062b5d0b2b68 ea686db99fe 91fa76b74fef6ff 899babe51b)
Change-Id: I7cd2977c8268c1
(cherry picked from commit eefb20e4658e17f