From ca55d716e06996552accbe4ece02e1d112ee6afd Mon Sep 17 00:00:00 2001 From: Tony Mack Date: Wed, 5 Dec 2012 13:11:08 -0500 Subject: [PATCH 1/1] renew returns a list of structs --- sfa/openstack/nova_driver.py | 3 ++- sfa/planetlab/pldriver.py | 9 ++++----- 2 files changed, 6 insertions(+), 6 deletions(-) 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 -- 2.45.2