]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commitdiff
fix update_port to get tenant_id from db rather than request
authorAaron Rosen <arosen@nicira.com>
Wed, 27 Feb 2013 23:00:41 +0000 (15:00 -0800)
committerAaron Rosen <arosen@nicira.com>
Fri, 1 Mar 2013 18:42:21 +0000 (10:42 -0800)
This patch changes the behavor of nvp's update_port() to how it previously
worked by retrieving the value from the db and not the request.

Fixes bug 1138330

Change-Id: I49682db392954b3cfaeb4ad4b8acf95113dcae7e

quantum/plugins/nicira/nicira_nvp_plugin/QuantumPlugin.py

index 8fb8aee1938ac4ceaea8b0c6f53f0eef31a741b3..a20be4d532f1ecedd282493e631bc236bf59fb21 100644 (file)
@@ -1358,7 +1358,6 @@ class NvpPluginV2(db_base_plugin_v2.QuantumDbPluginV2,
         if attr.is_attr_set(port['port'].get(psec.PORTSECURITY)):
             self._enforce_set_auth(context, port,
                                    self.port_security_enabled_update)
-        tenant_id = self._get_tenant_id_for_create(context, port['port'])
         delete_security_groups = self._check_update_deletes_security_groups(
             port)
         has_security_groups = self._check_update_has_security_groups(port)
@@ -1368,7 +1367,7 @@ class NvpPluginV2(db_base_plugin_v2.QuantumDbPluginV2,
                 context, id, port)
             # copy values over
             ret_port.update(port['port'])
-
+            tenant_id = self._get_tenant_id_for_create(context, ret_port)
             # populate port_security setting
             if psec.PORTSECURITY not in port['port']:
                 ret_port[psec.PORTSECURITY] = self._get_port_security_binding(