]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Openvswitch update_port should return updated port info
authorJon Grimm <jgrimm@linux.vnet.ibm.com>
Wed, 27 Nov 2013 19:10:33 +0000 (13:10 -0600)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:36 +0000 (15:20 +0800)
commited822fdcdc4ac67a387bb7d081ebba86a8c2c91a
tree8ab38ef0e9d4f365e345fada1e06ccf09853c22a
parent13223da1182b1a31162d71cd9e2e28932b7f497c
Openvswitch update_port should return updated port info

Found when I enabled test_extension_allowedaddress_pairs, where
test_create_port_removed_allowed_address_pairs would fail due to the
returned port still containing the original addresspair.  The cause is
ovs simply not updating the port info being returned.

This patch additionally enables test_extension_allowedaddress_pairs for
openvswitch.

Moved checks and updating into method similar to what we do for
extradhcpopts and security_groups.

Additionally, this required fixing is_address_pairs_attribute_updated() as
it was passing (non-hashable) dicts to utils.compare_elements.

Change-Id: Ic871fea68fb9fcc862b1fd5ae5fe7aec540e4a30
Partial-Bug: #1255150
neutron/db/allowedaddresspairs_db.py
neutron/plugins/openvswitch/ovs_neutron_plugin.py
neutron/tests/unit/openvswitch/test_openvswitch_plugin.py