]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix SolidFire inaccurate model on migrated vols
authorJohn Griffith <john.griffith8@gmail.com>
Thu, 23 Oct 2014 16:37:11 +0000 (16:37 +0000)
committerJohn Griffith <john.griffith8@gmail.com>
Sun, 26 Oct 2014 14:34:49 +0000 (08:34 -0600)
commit6338b76a2de7d50d4fc47d8d88f2ba60cf15d939
tree29395b51e1c6eecd9dc9a8bac3790c487edaf3c4
parentc10525973651d488f5d7d83446561406742f61d4
Fix SolidFire inaccurate model on migrated vols

The general migration impl in Cinder works
by creating a new volume, transfering the data
from the original volume to the new volume, and
then deleting the original and flipping the ID
of the new volume.

Turns out we missed the fact that this creates a
mismatch between the volume Cinder will later ask
for and what the volumes identity is on the backend
device.

This change adds a check on create_volume at the drivers
level to see if it's part of a migration and is infact
going to get renamed.  If so, just use the new name
and avoid all the headaches that come later with updating
provider auth and location.

The model info won't change in this case and is accessible
independent of the ID field in the Cinder base and the
crazy change that's going to take place on that value
in the Cinder DB.

Change-Id: If916df5fd986352ffb20f2dd74552e4d7dad4f60
Closes-Bug: #1381943
cinder/tests/test_solidfire.py
cinder/volume/drivers/solidfire.py
cinder/volume/manager.py