From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Wed, 5 Dec 2012 18:11:08 +0000 (-0500)
Subject: renew returns a list of structs
X-Git-Tag: sfa-3.0-0~62
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=ca55d716e06996552accbe4ece02e1d112ee6afd;p=sfa.git

renew returns a list of structs
---

diff --git a/sfa/openstack/nova_driver.py b/sfa/openstack/nova_driver.py
index b6595b2e..91de48bc 100644
--- a/sfa/openstack/nova_driver.py
+++ b/sfa/openstack/nova_driver.py
@@ -395,7 +395,8 @@ class NovaDriver(Driver):
         return True
 
     def renew (self, urns, expiration_time, options={}):
-        return True
+        description = self.describe(urns, None, options)
+        return description['geni_slivers']
 
     def perform_operational_action  (self, urns, action, options={}):
         aggregate = OSAggregate(self)
diff --git a/sfa/planetlab/pldriver.py b/sfa/planetlab/pldriver.py
index a1690f77..138e2549 100644
--- a/sfa/planetlab/pldriver.py
+++ b/sfa/planetlab/pldriver.py
@@ -710,11 +710,10 @@ class PlDriver (Driver):
         slice = slices[0]
         requested_time = utcparse(expiration_time)
         record = {'expires': int(datetime_to_epoch(requested_time))}
-        try:
-            self.shell.UpdateSlice(slice['slice_id'], record)
-            return True
-        except:
-            return False
+        self.shell.UpdateSlice(slice['slice_id'], record)
+        description = self.describe(urns, None, options)
+        return description['geni_slivers']
+            
 
     def perform_operational_action (self, urns, action, options={}):
         # MyPLC doesn't support operational actions. Lets pretend like it