From cae84bf926ab97cc5e57aeac6b13b9087653117e Mon Sep 17 00:00:00 2001
From: Quentin Smith <quentin@mit.edu>
Date: Thu, 24 Nov 2011 20:42:19 -0500
Subject: [PATCH] Handle domains spinning in pygrub

---
 code/main.py             |    8 ++++++--
 code/templates/list.mako |   13 ++++++++-----
 debian/changelog         |    6 ++++++
 3 files changed, 20 insertions(+), 7 deletions(-)

diff --git a/code/main.py b/code/main.py
index 0edcb65..38b677c 100755
--- a/code/main.py
+++ b/code/main.py
@@ -605,8 +605,12 @@ def infoDict(username, state, machine):
     else:
         main_status = dict(status[1:])
         main_status['host'] = controls.listHost(machine)
-        start_time = float(main_status.get('start_time', 0))
-        uptime = datetime.timedelta(seconds=int(time.time()-start_time))
+        start_time = main_status.get('start_time')
+        if start_time is None:
+            uptime = "Still booting?"
+        else:
+            start_time = float(start_time)
+            uptime = datetime.timedelta(seconds=int(time.time()-start_time))
         cpu_time_float = float(main_status.get('cpu_time', 0))
         cputime = datetime.timedelta(seconds=int(cpu_time_float))
     display_fields = [('name', 'Name'),
diff --git a/code/templates/list.mako b/code/templates/list.mako
index 3d0c50c..e6ed39f 100644
--- a/code/templates/list.mako
+++ b/code/templates/list.mako
@@ -96,21 +96,24 @@ ${self.fn.cdromList(defaults.cdrom, "$('cd_or_auto_cd').checked = true;$('autoin
 </%def>
 
 <%def name="machineRow(machine, dark)">
+<%
+	on = (machine.uptime is not None)
+%>
       <tr\
 % if dark:
  class="stripedrow" \
 % endif
 > 
 	<td rowspan="2">
-	% if machine.uptime and installing[machine]:
+	% if on and installing[machine]:
 	<img src="static/power_installing.png" alt="Installing..." />
 	% else:
-	  <form action="machine/${machine.machine_id}/command/${'shutdown' if machine.uptime else 'create'}" method="post">
+	  <form action="machine/${machine.machine_id}/command/${'shutdown' if on else 'create'}" method="post">
 	    <input type="hidden" name="back" value="list"/>
 	    <input type="hidden" name="machine_id"
 		   value="${machine.machine_id}"/>
-<input type="submit" class="power ${'on' if machine.uptime else 'off'}" name="action" value="${'Shutdown' if machine.uptime else 'Power on'}"\
-% if machine.uptime:
+<input type="submit" class="power ${'on' if on else 'off'}" name="action" value="${'Shutdown' if on else 'Power on'}"\
+% if on:
  onclick="return confirm('Are you sure you want to power off this VM?');"
 % endif
 />
@@ -127,7 +130,7 @@ ${self.fn.cdromList(defaults.cdrom, "$('cd_or_auto_cd').checked = true;$('autoin
 	<td></td>
 % endif
 <td>\
-% if machine.uptime:
+% if on:
 ${datetime.timedelta(seconds=int(machine.uptime))}\
 % endif
 </td>
diff --git a/debian/changelog b/debian/changelog
index fbcd172..863f1c2 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+invirt-web (0.1.22) unstable; urgency=low
+
+  * Handle domains spinning in pygrub
+
+ -- Quentin Smith <quentin@mit.edu>  Thu, 24 Nov 2011 20:42:09 -0500
+
 invirt-web (0.1.21) unstable; urgency=low
 
   * Fix strict validation of description field
-- 
1.7.9.5