'GlusterfsDriver' object has no attribute 'migrate_volume'

[Bharat Kumar Kobagana]

When I am trying to migrate cinder volume between two gluster volumes I am getting below error:

2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher File "/home/kevin/code/openstack/cinder/cinder/volume/manager.py", line 143, in lvo_inner2
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher return f(*_args, **_kwargs)
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher File "/usr/lib/python2.7/site-packages/oslo_utils/excutils.py", line 119, in __exit__
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher six.reraise(self.type_, self.value, self.tb)
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher File "/home/kevin/code/openstack/cinder/cinder/volume/manager.py", line 556, in migrate_volume
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher self.driver.migrate_volume(volume_ref)
2015-08-21 10:13:24.654 TRACE oslo_messaging.rpc.dispatcher AttributeError: 'GlusterfsDriver' object has no attribute 'migrate_volume'

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/221251

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/221425

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/221445

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/221425
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=223fbd36b7ec433040f5b95dcbaacf46f6939b79
Submitter: Jenkins
Branch: master

commit 223fbd36b7ec433040f5b95dcbaacf46f6939b79
Author: Eric Harney <email address hidden>
Date: Tue Sep 8 13:31:31 2015 -0400

    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

[OpenStack Infra (hudson-openstack)]

Change abandoned by Bharat Kumar Kobagana (BharatK) (<email address hidden>) on branch: master
Review: https://review.openstack.org/221251
Reason: This patch is no longer required. As this issue fixed by Eric here: https://review.openstack.org/#/c/221425/

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/221445
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=7edc87ff0e92043f5a21d630bbb9b966f5e34535
Submitter: Jenkins
Branch: master

commit 7edc87ff0e92043f5a21d630bbb9b966f5e34535
Author: Eric Harney <email address hidden>
Date: Tue Sep 8 14:10:55 2015 -0400

    Add manage_existing and unmanage to BaseVD

    manage_existing() and unmanage() only exist in the deprecated
    VolumeDriver() class and not BaseVD. This means that drivers
    inheriting from *VD hit attribute errors when trying to call
    either of these methods.

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

    Change-Id: Ie8aba971b51e1f5eaac5a4e681ea06b358ebb7b8