]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 126e4f2) | patch
Add unmanage default implementation to VolumeDriver
authorEric Harney <eharney@redhat.com>
Tue, 8 Sep 2015 20:23:45 +0000 (16:23 -0400)
committerEric Harney <eharney@redhat.com>
Tue, 8 Sep 2015 20:38:00 +0000 (16:38 -0400)
commitbf22aeb185afe8de0db526c83e50c15113f84d88
tree8132e279b8021acc08ac2aafbd1ccb7cd96520b9tree | snapshot
parent126e4f215e5fc0275a3c660990504b4e758de099commit | diff
Add unmanage default implementation to VolumeDriver

Currently, only drivers that inherit from ManageableVD
get the default unmanage() implementation.

This is not correct, because drivers can support unmanage
without driver code.  Fix unmanage() in VolumeDriver to
support the LVM driver's unmanage operation.

(Regression from the refactoring done when introducing
abc classes.)

Closes-Bug: #1493544

Change-Id: Iaacf10141559a2c88b5873722d1e237dc9223f27
cinder/tests/unit/test_volume.py diff | blob | history
cinder/volume/driver.py diff | blob | history