]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove force check from copy_volume_to_image
authorJohn Griffith <john.griffith8@gmail.com>
Wed, 22 Apr 2015 04:28:34 +0000 (04:28 +0000)
committerJohn Griffith <john.griffith@solidfire.com>
Thu, 23 Apr 2015 01:04:18 +0000 (19:04 -0600)
commitfb2c434c2461a25193067104de766fc1142a6024
tree918dfd3dc57b9acfd06161525f7ae58cc7e79d7c
parentcbcbc90cf64d91a33b7bcfb826c59d4b69697486
Remove force check from copy_volume_to_image

The upload_volume_to_image method allows a force parameter that will
upload a volume even though the volume is attached/in-use. A user can
get away with this with the LVM driver because the LVM backing is
local to the Cinder worker node that's pushing the bits to Glance.

The problem is, this only works for local storage, it won't work
with any iSCSI devices because they can't do multi-attach. Also,
the reason we required that a volume NOT be in-use for this
operation is because we have no way of keeping the guest Instance
from writing to the volume while we're uploading, and corrupting the data.

This has been exposed like this for several releases, so removing it
now likely would not be a good user experience.  Instead, this
patch add a config option to enable/disable it (default is to
disable), and deployers can choose whether they would like to
allow the use of --force True or not.

DocImpact Disables the --force option to copy-volume-to-image and
          introduces "allow_force_upload" boolean option to re-enable

Change-Id: I1210186689dea204601356f8d08805c6cb6f017c
Closes-Bug: 1446954
cinder/tests/unit/api/contrib/test_volume_actions.py
cinder/volume/api.py