]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Dell SC: init_volume stale volume info fix
authorTom Swanson <tom_swanson@dell.com>
Tue, 25 Aug 2015 20:13:31 +0000 (15:13 -0500)
committerTom Swanson <tom_swanson@dell.com>
Thu, 27 Aug 2015 16:09:11 +0000 (11:09 -0500)
commite66b0856a5eeabc78c356067f60c927ac1d47d8a
treef2bd033586d3ab814adce542bdb2b347022dd945
parent25d90467b176463b609834ceb5851920896c6975
Dell SC: init_volume stale volume info fix

In init_volume the volume information becomes stale after the volume
is mapped to a server.  The volume should be updated before being sent
to unmap.

The unmap function was testing for the volume being active.  This
should not be required.  It is unlikely an inactive volume will have
mappings but not impossible.  This check has been removed.

Removed LOG.error from _find_mapping_profiles as the API returning
an error is fine if the volume is inactive.  Plus we were not failing
on it.  The LOG.debug is sufficient.

Change-Id: I3ff8447360cea134e1008fa8db5cd912aa5effd3
cinder/tests/unit/test_dellscapi.py
cinder/volume/drivers/dell/dell_storagecenter_api.py