address pairs in update request always trigger agent notification

Bug #1255145 reported by Salvatore Orlando
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Salvatore Orlando

Bug Description

currently some plugins always notify the agent of a port_update even if the allowed address pairs attribute is specified: https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/plugin.py#L600

However, if there's no change in allowed address pairs there is no need for triggering a notification

Affected plugins:
ml2
openvswitch
nec

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/58566

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/58566
Committed: http://github.com/openstack/neutron/commit/eb23b345876796f4002cab3632bad1c840289d04
Submitter: Jenkins
Branch: master

commit eb23b345876796f4002cab3632bad1c840289d04
Author: Salvatore Orlando <email address hidden>
Date: Tue Nov 26 08:18:17 2013 -0800

    Rebind allowed address pairs only if they changed

    This patch ensures allowed address pairs bindings are refreshed
    only when they actually change.
    This will also avoid sending a notification to the agent if no
    change actually occured.

    Closes-Bug: #1255145
    Partial blueprint neutron-tempest-parallel

    Change-Id: Iac2502586a0d215a29194590c16c2e1a064f943b

Changed in neutron:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: icehouse-1 → 2014.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.