]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Catch more general exception in manager's create_volume
authorMichal Dulko <michal.dulko@intel.com>
Wed, 1 Apr 2015 11:28:34 +0000 (13:28 +0200)
committerMichal Dulko <michal.dulko@intel.com>
Wed, 1 Apr 2015 11:28:34 +0000 (13:28 +0200)
commitb9e6af72029aebaa2b101cd9ade657a3c532b73d
tree48e3fb6d12692882ed6dffd3896274336f5d9d28
parente8652b1e924b3d8eccf00bc8767a77f8637233bb
Catch more general exception in manager's create_volume

When fixing issues with inaccurate allocated_capacity tracking when
create_volume flow fails (Icd6b04ac60c17cbda225d3be8bcc4435ea0cd7a8)
it was ignored that drivers may raise all kinds of exceptions and these
aren't repacked into CinderException in the flow. This commit broadens
except clause to catch Exception when deciding if rescheduling
occurred while creating a volume. Also regression unit test is added.

Change-Id: I36058eacd1eb6b42c6a4311bd3382ddc9f47550c
Closes-Bug: 1438744
cinder/tests/test_volume.py
cinder/volume/manager.py