get_instance_disk_info() now takes an additional, optional argument
of block_device_info. When passed in, we use it to filter out
volumes, such that they are not returned. Previously, any devices
classified as "file" as parsed from the instance XML were considered
local disks. This assumption is incorrect, since some volumes are
simply files, rather than block devices.
Added a new test, based almost entirely off the existing test case
for this function, which represents an XML file with two fake
file-based (NFS-like) volumes and associated block_device_info to
claim those volumes.
Change-Id: I4809a8fd68eb0f709347aa7cad0fa85389ade41c
(cherry picked from commit e71081b856b06513cc1e56a9adb2a619967f0eae)
Reviewed: https:/ /review. openstack. org/35529 github. com/openstack/ nova/commit/ eff629fa4cfdc68 e670d70a4a38810 c4935f81af
Committed: http://
Submitter: Jenkins
Branch: stable/grizzly
commit eff629fa4cfdc68 e670d70a4a38810 c4935f81af
Author: Rafi Khardalian <email address hidden>
Date: Thu Jun 13 05:09:45 2013 +0000
Fix resizes with attached file-based volumes
Bug 1190364
get_ instance_ disk_info( ) now takes an additional, optional argument
of block_device_info. When passed in, we use it to filter out
volumes, such that they are not returned. Previously, any devices
classified as "file" as parsed from the instance XML were considered
local disks. This assumption is incorrect, since some volumes are
simply files, rather than block devices.
Added a new test, based almost entirely off the existing test case
for this function, which represents an XML file with two fake
file-based (NFS-like) volumes and associated block_device_info to
claim those volumes.
Change-Id: I4809a8fd68eb0f 709347aa7cad0fa 85389ade41c 3cc1e56a9adb2a6 19967f0eae)
(cherry picked from commit e71081b856b0651