X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=PLC%2FMethods%2FRebootNodeWithPCU.py;h=751f9fd2af47344e5e723253a4d788a418228e90;hb=83fe6b6b3571d190b525e46d30b0df05e552d76e;hp=ac5b6b6506df3be74c380e4146e2fafb8900b657;hpb=b0f3bc65926002bffd013f8201b6200c9e53db79;p=plcapi.git

diff --git a/PLC/Methods/RebootNodeWithPCU.py b/PLC/Methods/RebootNodeWithPCU.py
index ac5b6b6..751f9fd 100644
--- a/PLC/Methods/RebootNodeWithPCU.py
+++ b/PLC/Methods/RebootNodeWithPCU.py
@@ -11,22 +11,22 @@ from PLC.Nodes import Node, Nodes
 from PLC.PCUs import PCU, PCUs
 
 try:
-	from pcucontrol import reboot
-	external_dependency = True
+    from pcucontrol import reboot
+    external_dependency = True
 except:
-	external_dependency = False
+    external_dependency = False
 
 class RebootNodeWithPCU(Method):
     """
-	Uses the associated PCU to attempt to reboot the given Node.
+        Uses the associated PCU to attempt to reboot the given Node.
 
     Admins can reboot any node. Techs and PIs can only reboot nodes at
     their site.
 
     Returns 1 if the reboot proceeded without error (Note: this does not guarantee
-	that the reboot is successful).
-	Returns -1 if external dependencies for this call are not available.
-	Returns "error string" if the reboot failed with a specific message.
+        that the reboot is successful).
+        Returns -1 if external dependencies for this call are not available.
+        Returns "error string" if the reboot failed with a specific message.
     """
 
     roles = ['admin', 'pi', 'tech']
@@ -41,7 +41,7 @@ class RebootNodeWithPCU(Method):
     returns = Parameter(int, '1 if successful')
 
     def call(self, auth, node_id_or_hostname, testrun=None):
-        # Get account information
+    # Get account information
         nodes = Nodes(self.api, [node_id_or_hostname])
         if not nodes:
             raise PLCInvalidArgument, "No such node"
@@ -60,22 +60,24 @@ class RebootNodeWithPCU(Method):
             if node['site_id'] not in self.caller['site_ids']:
                 raise PLCPermissionDenied, "Not allowed to reboot nodes from specified site"
 
-	# Verify that the node has pcus associated with it.
-	pcus = PCUs(self.api, {'pcu_id' : node['pcu_ids']} )
+        # Verify that the node has pcus associated with it.
+        pcus = PCUs(self.api, {'pcu_id' : node['pcu_ids']} )
         if not pcus:
             raise PLCInvalidArgument, "No PCUs associated with Node"
 
-	pcu = pcus[0]
+        pcu = pcus[0]
 
-	if not external_dependency:
+        if not external_dependency:
             raise PLCNotImplemented, "Could not load external module to attempt reboot"
 
-	# model, hostname, port, 
-	# i = pcu['node_ids'].index(node['node_id'])
-	# p = pcu['ports'][i]
-	ret = reboot.reboot_api(node, pcu, testrun)
+        # model, hostname, port,
+        # i = pcu['node_ids'].index(node['node_id'])
+        # p = pcu['ports'][i]
+        ret = reboot.reboot_api(node, pcu, testrun)
+
+        node.update_last_pcu_reboot(commit=True) # commits new timestamp to node 
 
         self.event_objects = {'Node': [node['node_id']]}
-        self.message = "RebootNodeWithPCU called"
-	        
+        self.message = "RebootNodeWithPCU %s with %s returned %s" % (node['node_id'], pcu['pcu_id'], ret)
+
         return ret