]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
Fix 500 raised on disassociate_floatingips when out of sync
authorAaron Rosen <arosen@nicira.com>
Tue, 7 May 2013 03:21:57 +0000 (20:21 -0700)
committerAaron Rosen <arosen@nicira.com>
Tue, 7 May 2013 04:27:51 +0000 (21:27 -0700)
If one deletes NAT rules directly from NVP and not through quantum,
an error will be raised when deleting a port that is associated with
the floating ip. This patch catches the NotFound exception and logs
it so that the port can successfully be deleted.

Fixes bug: 1177176

Change-Id: Ia7d0b94957d0c6b014d242439245b0e4fa9de40d

quantum/plugins/nicira/QuantumPlugin.py

index 52879c2a21069632096548d88037eb2a96939069..80591dde5261aacc588ac8247f5361a11564345b 100644 (file)
@@ -1856,6 +1856,9 @@ class NvpPluginV2(db_base_plugin_v2.QuantumDbPluginV2,
         except sa_exc.NoResultFound:
             LOG.debug(_("The port '%s' is not associated with floating IPs"),
                       port_id)
+        except q_exc.NotFound:
+            LOG.warning(_("Nat rules not found in nvp for port: %s"), id)
+
         super(NvpPluginV2, self).disassociate_floatingips(context, port_id)
 
     def create_network_gateway(self, context, network_gateway):