]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
VMware: Fixed upload-to-image for available volume
authorSubramanian Neelakantan <subbu@ubuntu.(none)>
Wed, 9 Oct 2013 10:53:52 +0000 (03:53 -0700)
committerSubramanian Neelakantan <subramanian.neelakantan@gmail.com>
Sat, 12 Oct 2013 04:21:24 +0000 (09:51 +0530)
commita2b774c0ce2a1aaa9cdfc7d7c4c4d642f1bfc3d1
tree6731eb78a5df3c44660cb31061b320cb746ed9dc
parent3da3a0e827ff0e099514702d7116084245f03e80
VMware: Fixed upload-to-image for available volume

Cinder generally does not allow upload-to-image operation on a volume
that is 'in-use'. This can however be over-ridden using the '--force True'
flag. The VMware driver cannot support upload-to-image when the volume is
in-use. This is a restriction for the VMware driver alone. When the user forces
an upload on a volume that is 'in-use' the Cinder api layer sets the volume
status to 'uploading' and calls into the driver code to upload. In this
scenario the VMware driver needs to fail the operation.

The current driver code here does this check wrongly by looking for volume
status of 'in-use'. Fixing this check to identify an in-use volume correctly.

Fixes bug: 1237338

Change-Id: If54edfdc242a7a1bff442b6bb4c5a9865eede1dc
cinder/tests/test_vmware_vmdk.py
cinder/volume/drivers/vmware/vmdk.py