]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix Pure get pgroup volume snapshot name
authorPatrick East <patrick.east@purestorage.com>
Fri, 11 Sep 2015 21:12:03 +0000 (14:12 -0700)
committerPatrick East <patrick.east@purestorage.com>
Fri, 11 Sep 2015 21:59:59 +0000 (14:59 -0700)
commitd2170a9e13047ea9a05ed78c4e6da61a5dfc46e2
tree22dda48b0ae20066e99ff332f1744d098696c3b5
parent39ae7a23eb39c52c2f29a15aeaf5570fb5dabe59
Fix Pure get pgroup volume snapshot name

When creating a volume from a consistency group snapshot we have to go
look up the volume level snapshot from the group. Previously we could
just directly get to the cgsnapshot model from the snapshot. After we
switched to using the snapshot objects however we cannot do that (at
least not until we have cgsnapshot objects too).

As an interim solution we did a query to the REST API on the array to
just list all of the snapshots. Unfortunately that doesn’t work on all
versions of the Purity REST API. To have a more usable solution we need
to do a db lookup to get the pgsnapshot.

It’s not ideal, and will be removed ASAP, but for now we need to at
least not regress in functionality.

Change-Id: I73a4ad5883bf33ce282b5a49aa8acba66ffcb84f
Closes-Bug: #1494934
cinder/tests/unit/test_pure.py
cinder/volume/drivers/pure.py