commit c1665b13b4cdd40641525996899ca46bbd3993b5
author Tony Mack <tmack@paris.CS.Princeton.EDU> Mon Apr 29 16:45:14 2013 -0400
committer Tony Mack <tmack@paris.CS.Princeton.EDU> Mon Apr 29 16:45:14 2013 -0400
tree 449875f5c5e64bd6171d9399a1efb5892f2a9bd3
parent 9bcbe4f97aefc3e4679bafe1f37ac5e049f56621

fix bug in delete_network_ports()

plstackapi/openstack/driver.py

diff --git a/plstackapi/openstack/driver.py b/plstackapi/openstack/driver.py
index 3522b8c..b03669d 100644
--- a/plstackapi/openstack/driver.py
+++ b/plstackapi/openstack/driver.py
@@ -103,6 +103,7 @@
             router = routers[0]
         else:
             router = self.shell.quantum.create_router({'router': {'name': name}})['router']
+        # add router to external network
         if set_gateway:
             nets = self.shell.quantum.list_networks()['networks']
             for net in nets:
@@ -116,6 +117,13 @@
         routers = self.shell.quantum.list_routers(id=id)['routers']
         for router in routers:
             self.shell.quantum.delete_router(router['id'])
+            
+            # remove router form external network
+            nets = self.shell.quantum.list_networks()['networks']
+            for net in nets:
+                if net['router:external'] == True:
+                    self.shell.quantum.add_gateway_router(router['id'],
+                                                          {'network_id': net['id']})
 
     def add_router_interface(self, router_id, subnet_id):
         router = self.shell.quantum.show_router(router_id)['router']
@@ -152,7 +160,7 @@
     def delete_network_ports(self, network_id):
         ports = self.shell.quantum.list_ports()['ports']
         for port in ports:
-            if port['network_id'] == 'network_id':
+            if port['network_id'] == network_id:
                 self.shell.quantum.delete_port(port['id'])
         return 1