Comment 4 for bug 1381943

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/130595
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=6338b76a2de7d50d4fc47d8d88f2ba60cf15d939
Submitter: Jenkins
Branch: master

commit 6338b76a2de7d50d4fc47d8d88f2ba60cf15d939
Author: John Griffith <email address hidden>
Date: Thu Oct 23 16:37:11 2014 +0000

    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