DeleteSliver returns a boolean instead of an int
authorTony Mack <tmack@paris.CS.Princeton.EDU>
Fri, 31 Aug 2012 00:04:53 +0000 (20:04 -0400)
committerTony Mack <tmack@paris.CS.Princeton.EDU>
Fri, 31 Aug 2012 00:05:11 +0000 (20:05 -0400)
sfa/openstack/osaggregate.py
sfa/planetlab/pldriver.py

index 0ff4860..c5c4c6d 100644 (file)
@@ -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):
index 0951e8c..b5e7771 100644 (file)
@@ -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)