]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
NetApp: E-Series fix deletion of missing volume
authorMichael Price <michael.price@netapp.com>
Wed, 30 Sep 2015 18:56:56 +0000 (13:56 -0500)
committerMichael Price <michael.price@netapp.com>
Fri, 16 Oct 2015 15:44:16 +0000 (15:44 +0000)
commit5cbf00b624e6c2bb8597d82640a1cb61b9405ba6
treec6958a0b81557bf859388060284a0aee72ddf0d9
parentc9eef31820dc385a2c9f4ba24dd1d194f9e7d088
NetApp: E-Series fix deletion of missing volume

When a Cinder volume is deleted but the backend volume is missing,
an exception was being thrown, rather than swallowed and being logged.
A KeyError was being expected, but a VolumeNotFound exception was
actually being raised.

This patch catches the VolumeNotFound exception, and logs a debug
message if the volume could not be deleted from the backend (because
it no longer exists). This patch restores compatibility with older
versions where an exception would not be thrown.

Closes-Bug: 1502166
Change-Id: I47a8d9b408fc4ac7fae30f043b0dfca0bb763d8d
cinder/tests/unit/test_netapp_eseries_iscsi.py
cinder/volume/drivers/netapp/eseries/library.py