From: Tony Mack <tmack@paris.CS.Princeton.EDU>
Date: Fri, 31 Aug 2012 00:04:53 +0000 (-0400)
Subject: DeleteSliver returns a boolean instead of an int
X-Git-Tag: sfa-2.1-14~33
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=4889e9bac6b0f5d3704ee0318d378d33fce5fb40;p=sfa.git

DeleteSliver returns a boolean instead of an int
---

diff --git a/sfa/openstack/osaggregate.py b/sfa/openstack/osaggregate.py
index 0ff48609..c5c4c6dd 100644
--- a/sfa/openstack/osaggregate.py
+++ b/sfa/openstack/osaggregate.py
@@ -325,7 +325,7 @@ class OSAggregate:
             self.driver.shell.nova_manager.servers.delete(instance)
             # deleate this instance's security groups
             thread_manager.run(_delete_security_group, instance)
-        return 1
+        return True
 
 
     def stop_instances(self, instance_name, tenant_name):
diff --git a/sfa/planetlab/pldriver.py b/sfa/planetlab/pldriver.py
index 0951e8c0..b5e77710 100644
--- a/sfa/planetlab/pldriver.py
+++ b/sfa/planetlab/pldriver.py
@@ -772,7 +772,7 @@ class PlDriver (Driver):
         slicename = hrn_to_pl_slicename(slice_hrn)
         slices = self.shell.GetSlices({'name': slicename})
         if not slices:
-            return 1
+            return True
         slice = slices[0]
     
         # determine if this is a peer slice
@@ -787,7 +787,7 @@ class PlDriver (Driver):
         finally:
             if peer:
                 self.shell.BindObjectToPeer('slice', slice['slice_id'], peer, slice['peer_slice_id'])
-        return 1
+        return True
     
     def renew_sliver (self, slice_urn, slice_hrn, creds, expiration_time, options):
         slicename = hrn_to_pl_slicename(slice_hrn)