]> 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: 8e63f4d) | patch
Add migrate_volume to BaseVD
authorEric Harney <eharney@redhat.com>
Tue, 8 Sep 2015 17:31:31 +0000 (13:31 -0400)
committerEric Harney <eharney@redhat.com>
Tue, 8 Sep 2015 18:12:08 +0000 (14:12 -0400)
commit223fbd36b7ec433040f5b95dcbaacf46f6939b79
treefe881b95cd9eb938a6e5ac9704636c1f5102b563tree | snapshot
parent8e63f4da54bd28a837f2feee115b6c3db3262a2dcommit | diff
Add migrate_volume to BaseVD

migrate_volume() only exists in the deprecated VolumeDriver()
class and not BaseVD.  This means that drivers inheriting from
*VD hit attribute errors when trying to call driver.migrate_volume().

I suspect this indicates that we should rework some of our
abc class structure a little bit, but this seems like a
reasonable fix for now.

Related-Bug: #1478987
Related-Bug: #1471807
Closes-Bug: #1493286

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