]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
VMware: Remove redundant extend disk API call
authorVipin Balachandran <vbala@vmware.com>
Wed, 20 Aug 2014 11:40:39 +0000 (17:10 +0530)
committerVipin Balachandran <vbala@vmware.com>
Wed, 3 Sep 2014 06:29:33 +0000 (11:59 +0530)
commit1fdb38fd4b43c2685658c1a7aa6a56c92ac31585
tree225da174d70f5fb0bf0afb96d418fb55aefd4f83
parentbaa34a301b1e46d860038b4901b8061bb7101d4f
VMware: Remove redundant extend disk API call

Extend virtual disk API is called during volume creation from image if
the image size is less than volume size. In the case of streamOptimized
and sparse vmdk images, the size of the virtual disk created from the
image can be greater than image size and equal to the volume size. For
example, streamOptimized image created from a new 1GB volume has size
69120 bytes and the size of the virtual disk created from this image
is 1GB. Therefore, relying on image size to invoke extend API might
result in VIM API fault, if the virtual disk size is same as the target
size (volume size). The fix is to read the current virtual disk size to
decide whether extend disk needs to be invoked or not.

Closes-Bug: #1301854
Change-Id: I990ac0b9aef68b3ef8b6d67188db8d44b5c29599
cinder/tests/test_vmware_vmdk.py
cinder/tests/test_vmware_volumeops.py
cinder/volume/drivers/vmware/error_util.py
cinder/volume/drivers/vmware/vmdk.py
cinder/volume/drivers/vmware/volumeops.py