]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Catch vol not found in SolidFire transfer
authorJohn Griffith <john.griffith8@gmail.com>
Fri, 15 Aug 2014 23:26:15 +0000 (17:26 -0600)
committerJohn Griffith <john.griffith8@gmail.com>
Sun, 17 Aug 2014 18:50:24 +0000 (12:50 -0600)
commit7a72303aff86ba2b06c41e73b35989cddd884652
treee7d38683bc81ac0f58c7015bf286f5dd0f3b0b54
parentda93b87b600db5a5f1ddc2d0f72c326a6d72ae4b
Catch vol not found in SolidFire transfer

The accept_transfer call in SolidFire wasn't catching
the volume_not found exception like it should be (and
like other methods do).

This patch just adds a check for volume is None on the
get_volume call during accept_transfer and raises an
appropriate exception if we run into that case.

While we're at it, add some more detail to the "NotFound"
error messages to include the driver operation as per
request of reviewers.

Change-Id: Ie6b56953028865eb06ebdaae3416339b8a371351
Closes-Bug: #1357261
cinder/tests/test_solidfire.py
cinder/volume/drivers/solidfire.py