Cleaned up some old code added by the last merge

7 files changed, 1 insertions, 38 deletions

diff --git a/nova/compute/manager.py b/nova/compute/manager.py
index 4908cba97..ecfbd3908 100644
--- a/nova/compute/manager.py
+++ b/nova/compute/manager.py
@@ -1,4 +1,4 @@
-#: tabstop=4 shiftwidth=4 softtabstop=4
+# vim: tabstop=4 shiftwidth=4 softtabstop=4
 
 # Copyright 2010 United States Government as represented by the
 # Administrator of the National Aeronautics and Space Administration.
@@ -970,12 +970,6 @@ class ComputeManager(manager.SchedulerDependentManager):
         return self.driver.set_host_enabled(host, enabled)
 
     @exception.wrap_exception(notifier=notifier, publisher_id=publisher_id())
-    def set_power_state(self, context, instance_id=None, host=None,
-            power_state=None):
-        """Turns the specified host on/off, or reboots the host."""
-        return self.driver.set_power_state(host, power_state)
-
-    @exception.wrap_exception(notifier=notifier, publisher_id=publisher_id())
     def get_diagnostics(self, context, instance_id):
         """Retrieve diagnostics for an instance on this host."""
         instance_ref = self.db.instance_get(context, instance_id)
diff --git a/nova/virt/fake.py b/nova/virt/fake.py
index a811edf7a..89ad20494 100644
--- a/nova/virt/fake.py
+++ b/nova/virt/fake.py
@@ -519,7 +519,3 @@ class FakeConnection(driver.ComputeDriver):
     def set_host_enabled(self, host, enabled):
         """Sets the specified host's ability to accept new instances."""
         pass
-
-    def set_power_state(self, host, power_state):
-        """Reboots, shuts down or starts up the host."""
-        pass
diff --git a/nova/virt/hyperv.py b/nova/virt/hyperv.py
index 6524e1739..ae30c62f0 100644
--- a/nova/virt/hyperv.py
+++ b/nova/virt/hyperv.py
@@ -506,7 +506,3 @@ class HyperVConnection(driver.ComputeDriver):
     def set_host_enabled(self, host, enabled):
         """Sets the specified host's ability to accept new instances."""
         pass
-
-    def set_power_state(self, host, power_state):
-        """Reboots, shuts down or starts up the host."""
-        pass
diff --git a/nova/virt/libvirt/connection.py b/nova/virt/libvirt/connection.py
index af5221f55..7655bf386 100644
--- a/nova/virt/libvirt/connection.py
+++ b/nova/virt/libvirt/connection.py
@@ -1569,7 +1569,3 @@ class LibvirtConnection(driver.ComputeDriver):
     def set_host_enabled(self, host, enabled):
         """Sets the specified host's ability to accept new instances."""
         pass
-
-    def set_power_state(self, host, power_state):
-        """Reboots, shuts down or starts up the host."""
-        pass
diff --git a/nova/virt/vmwareapi_conn.py b/nova/virt/vmwareapi_conn.py
index cfa4cb418..aaa384374 100644
--- a/nova/virt/vmwareapi_conn.py
+++ b/nova/virt/vmwareapi_conn.py
@@ -199,10 +199,6 @@ class VMWareESXConnection(driver.ComputeDriver):
         """Sets the specified host's ability to accept new instances."""

         pass

 

-    def set_power_state(self, host, power_state):

-        """Reboots, shuts down or starts up the host."""

-        pass

-

     def plug_vifs(self, instance, network_info):

         """Plugs in VIFs to networks."""

         self._vmops.plug_vifs(instance, network_info)

diff --git a/nova/virt/xenapi/vmops.py b/nova/virt/xenapi/vmops.py
index 2a5cf8b5e..b549b33d1 100644
--- a/nova/virt/xenapi/vmops.py
+++ b/nova/virt/xenapi/vmops.py
@@ -1050,17 +1050,6 @@ class VMOps(object):
             return xenapi_resp.details[-1]
         return resp["status"]
 
-    def set_power_state(self, host, power_state):
-        """Reboots, shuts down or starts up the host."""
-        args = {"power_state": power_state}
-        xenapi_resp = self._call_xenhost("set_power_state", args)
-        try:
-            resp = json.loads(xenapi_resp)
-        except TypeError  as e:
-            # Already logged; return the message
-            return xenapi_resp.details[-1]
-        return resp["power_state"]
-
     def _call_xenhost(self, method, arg_dict):
         """There will be several methods that will need this general
         handling for interacting with the xenhost plugin, so this abstracts
diff --git a/nova/virt/xenapi_conn.py b/nova/virt/xenapi_conn.py
index 2a6a97faf..a1c9a1e30 100644
--- a/nova/virt/xenapi_conn.py
+++ b/nova/virt/xenapi_conn.py
@@ -349,10 +349,6 @@ class XenAPIConnection(driver.ComputeDriver):
         """Sets the specified host's ability to accept new instances."""
         return self._vmops.set_host_enabled(host, enabled)
 
-    def set_power_state(self, host, power_state):
-        """Reboots, shuts down or starts up the host."""
-        return self._vmops.set_power_state(host, power_state)
-
 
 class XenAPISession(object):
     """The session to invoke XenAPI SDK calls"""