From a0ec6a6aa5ebdde1d099c5f6c03cf1dbd28441fa Mon Sep 17 00:00:00 2001
From: Ed Leafe <ed@leafe.com>
Date: Wed, 3 Aug 2011 00:52:15 +0000
Subject: Removed duplicate methods created by previous merge.

---
 .../xenserver/xenapi/etc/xapi.d/plugins/xenhost    | 27 ----------------------
 1 file changed, 27 deletions(-)

(limited to 'plugins')

diff --git a/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost b/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
index 5169aeb12..c29d57717 100755
--- a/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
+++ b/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
@@ -142,33 +142,6 @@ def host_start(self, arg_dict):
     return _powerstate("startup")
 
 
-@jsonify
-def set_power_state(self, arg_dict):
-    """Reboots or powers off this host. Ideally, we would also like to be
-    able to power *on* a host, but right now this is not technically
-    feasible.
-    """
-    power_state = arg_dict.get("power_state")
-    if power_state is None:
-        raise pluginlib.PluginError(
-                _("Missing 'power_state' argument to set_power_state"))
-    # Host must be disabled first
-#    result = _run_command("xe host-disable")
-#    if result:
-#        raise pluginlib.PluginError(result)
-#    # All running VMs must be shutdown
-#    result = _run_command("xe vm-shutdown --multiple power-state=running")
-#    if result:
-#        raise pluginlib.PluginError(result)
-#    cmds = {"reboot": "xe host-reboot", "on": "xe host-power-on",
-#            "off": "xe host-shutdown"}
-#    result = _run_command(cmds[power_state])
-#    # Should be empty string
-#    if result:
-#        raise pluginlib.PluginError(result)
-    return {"power_state": power_state}
-
-
 @jsonify
 def host_data(self, arg_dict):
     """Runs the commands on the xenstore host to return the current status
-- 
cgit 


From 75b110aa451382cce94f10a392597b40df97839c Mon Sep 17 00:00:00 2001
From: Ed Leafe <ed@leafe.com>
Date: Thu, 4 Aug 2011 20:49:21 +0000
Subject: Changed all references to 'power state' to 'power action' as
 requested by review.

---
 plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

(limited to 'plugins')

diff --git a/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost b/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
index c29d57717..f6a9ac8d8 100755
--- a/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
+++ b/plugins/xenserver/xenapi/etc/xapi.d/plugins/xenhost
@@ -103,7 +103,7 @@ def set_host_enabled(self, arg_dict):
     return {"status": status}
 
 
-def _powerstate(state):
+def _power_action(action):
     host_uuid = _get_host_uuid()
     # Host must be disabled first
     result = _run_command("xe host-disable")
@@ -115,23 +115,23 @@ def _powerstate(state):
         raise pluginlib.PluginError(result)
     cmds = {"reboot": "xe host-reboot", "startup": "xe host-power-on",
             "shutdown": "xe host-shutdown"}
-    result = _run_command(cmds[state])
+    result = _run_command(cmds[action])
     # Should be empty string
     if result:
         raise pluginlib.PluginError(result)
-    return {"powerstate": state}
+    return {"power_action": action}
 
 
 @jsonify
 def host_reboot(self, arg_dict):
     """Reboots the host."""
-    return _powerstate("reboot")
+    return _power_action("reboot")
 
 
 @jsonify
 def host_shutdown(self, arg_dict):
     """Reboots the host."""
-    return _powerstate("shutdown")
+    return _power_action("shutdown")
 
 
 @jsonify
@@ -139,7 +139,7 @@ def host_start(self, arg_dict):
     """Starts the host. NOTE: Currently not feasible, since the host
     runs on the same machine as Xen.
     """
-    return _powerstate("startup")
+    return _power_action("startup")
 
 
 @jsonify
-- 
cgit