From: Zhiteng Huang <zhiteng.huang@intel.com>
Date: Fri, 9 Nov 2012 06:37:21 +0000 (+0800)
Subject: Remove unused volume API method - remove_from_compute()
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=f70cc1aceb8abab6de69ac970fcc35ea23f70c25;p=openstack-build%2Fcinder-build.git

Remove unused volume API method - remove_from_compute()

remove_from_compute() method in cinder.volume.api is never used
by others and it is also broken since the interface in Nova has
been changed to remove_volume_connection(), thus it should be
removed from Cinder.

Change-Id: Ibcb2cf7f88606d9a540cc9a03b5c95990fca8c3e
---

diff --git a/cinder/volume/api.py b/cinder/volume/api.py
index c4bc66e67..b0af133f4 100644
--- a/cinder/volume/api.py
+++ b/cinder/volume/api.py
@@ -368,14 +368,6 @@ class API(base.Base):
             msg = _("already detached")
             raise exception.InvalidVolume(reason=msg)
 
-    def remove_from_compute(self, context, volume, instance_id, host):
-        """Remove volume from specified compute host."""
-        rpc.call(context,
-                 rpc.queue_get_for(context, FLAGS.compute_topic, host),
-                 {"method": "remove_volume_connection",
-                  "args": {'instance_id': instance_id,
-                           'volume_id': volume['id']}})
-
     @wrap_check_policy
     def reserve_volume(self, context, volume):
         self.update(context, volume, {"status": "attaching"})