]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
SolidFire provider_id for snapshots on init
authorJohn Griffith <john.griffith8@gmail.com>
Thu, 10 Sep 2015 00:59:38 +0000 (00:59 +0000)
committerJay S. Bryant <jsbryant@us.ibm.com>
Thu, 10 Sep 2015 18:41:45 +0000 (13:41 -0500)
commitccca5f5dacf3d2e36917d3005b0e536d1d235e5d
treeaeb9433a98e2422257beca4dff35c831a92df271
parentba0a12d52f7aa044d36f43dbe003858a29142bab
SolidFire provider_id for snapshots on init

We've added the provider_id field to the database for both
volumes and snapshots.  This however couldn't be used because
there was no upgrade/migration path included for backends to
populate these fields (ie upgrade from K to L).

This patch fixes the provider_id columns for snapshots and volumes
both, and updates these fields on backend init.  This fixes the
issue of not being able to "rely" on the database entries, and it
also fixes some issues with inefficiencies in how we were deleting
resources.

Now we can just use the provider_id info from the reference passed
in on the delete calls.

Change-Id: I9de02e9d12aa23ddf4e1fbb1e0efab2c1805fdb1
Closes-Bug: #bug/1491489
cinder/tests/unit/test_solidfire.py
cinder/volume/drivers/solidfire.py