Restructure L2 agent

Registered by Rossella Sblendido on 2014-11-28

Improvements for the L2 agent to pay down technical debt in Kilo

Blueprint information

Status:
Complete
Approver:
YAMAMOTO Takashi
Priority:
High
Drafter:
Rossella Sblendido
Direction:
Approved
Assignee:
Rossella Sblendido
Definition:
Approved
Series goal:
Accepted for mitaka
Implementation:
Implemented
Milestone target:
milestone icon mitaka-3
Started by
Kyle Mestery on 2015-01-05
Completed by
Armando Migliaccio on 2016-01-25

Related branches

Sprints

Whiteboard

Jan-24-2016(armax): All code targeting the bp looks merged [1]. To be marked complete, unless Rossella/Yamamoto confirms otherwise.

https://review.openstack.org/#/q/project:openstack/neutron+topic:bp/restructure-l2-agent

January-18-16 (rossella_s): Only 2 patches outstanding

September-13 (mestery): Moving to Mitaka per discussion with Rossella.

Sept-1-2015(armax): As of today I still see 9 outstanding patches. Is that all, to get this completed? Even if it did, this will never land in time to make L3. Optimistically move to L-rc1

March-18 (mestery): Moving this to Liberty-1, we won't land these today before FF.

February-2 (mestery): Moving to Kilo-3, this won't land in Kilo-2.

December-18 (mestery): Kilo-2.

Gerrit topic: https://review.openstack.org/#q,topic:restructure-l2-agent,n,z

Addressed by: https://review.openstack.org/137808 (Merged)
    Kilo L2 agent improvements

Gerrit topic: https://review.openstack.org/#q,topic:device_list,n,z

Addressed by: https://review.openstack.org/147435
    L2 agent RPC add bulk calls for device up and device down

Gerrit topic: https://review.openstack.org/#q,topic:seg-group,n,z

Addressed by: https://review.openstack.org/148235
    Add devices to update in RPC call security_groups_provider_updated

Gerrit topic: https://review.openstack.org/#q,topic:bp/restructure-l2-agent,n,z

Gerrit topic: https://review.openstack.org/#q,topic:port_update,n,z

Addressed by: https://review.openstack.org/155223
    Add state_or_sec_group_updated flag in port_update

Gerrit topic: https://review.openstack.org/#q,topic:sec-group-1,n,z

Gerrit topic: https://review.openstack.org/#q,topic:test,n,z

Addressed by: https://review.openstack.org/157839
    Introduce get_devices_details_list_and_failed_devices

Addressed by: https://review.openstack.org/157840
    Set port in error status if operation fails twice

Gerrit topic: https://review.openstack.org/#q,topic:l2-restr,n,z

Gerrit topic: https://review.openstack.org/#q,topic:sec_group_provider,n,z

Gerrit topic: https://review.openstack.org/#q,topic:flag-sec-group,n,z

Addressed by: https://review.openstack.org/161806
    Add get event to OVSDB monitor

Gerrit topic: https://review.openstack.org/#q,topic:(detached,n,z

Gerrit topic: https://review.openstack.org/#q,topic:ovs-event-last,n,z

Addressed by: https://review.openstack.org/164880
    OVS agent react to events instead of polling

Gerrit topic: https://review.openstack.org/#q,topic:ovs-last,n,z

Gerrit topic: https://review.openstack.org/#q,topic:l2agent1,n,z

Gerrit topic: https://review.openstack.org/#q,topic:l2agent,n,z

Gerrit topic: https://review.openstack.org/#q,topic:key-port,n,z

Gerrit topic: https://review.openstack.org/#q,topic:func-test1,n,z

Addressed by: https://review.openstack.org/140042
    Adds base in-tree functional testing of the ovs_neutron_agent

Gerrit topic: https://review.openstack.org/#q,topic:is_active,n,z

Gerrit topic: https://review.openstack.org/#q,topic:is_active2,n,z

Gerrit topic: https://review.openstack.org/#q,topic:marmella2,n,z

Addressed by: https://review.openstack.org/177028
    Remove is_active property from SimpleInterfaceMonitor

Gerrit topic: https://review.openstack.org/#q,topic:ovs,n,z

Gerrit topic: https://review.openstack.org/#q,topic:restr1,n,z

Gerrit topic: https://review.openstack.org/#q,topic:unit-test,n,z

Addressed by: https://review.openstack.org/188380
    Make _val_to_py not private

Addressed by: https://review.openstack.org/190638
    OVSNeutronAgent pass the config as paramter

Gerrit topic: https://review.openstack.org/#q,topic:failed_dev,n,z

Gerrit topic: https://review.openstack.org/#q,topic:bug/1468998,n,z

Addressed by: https://review.openstack.org/196097
    Merge remote-tracking branch 'origin/master' into merge-qos

Gerrit topic: https://review.openstack.org/#q,topic:bug/1468828,n,z

Addressed by: https://review.openstack.org/196701
    Merge remote-tracking branch 'origin/master' into merge-branch

Addressed by: https://review.openstack.org/196920
    Merge remote-tracking branch 'origin/master' into merge-branch

Addressed by: https://review.openstack.org/197751
    Merge remote-tracking branch 'origin/master' into merge-branch

Gerrit topic: https://review.openstack.org/#q,topic:bug/1472304,n,z

Addressed by: https://review.openstack.org/200163
    Merge remote-tracking branch 'origin/master' into merge-branch

Addressed by: https://review.openstack.org/202098
    Don't sync all devices when some fail

Addressed by: https://review.openstack.org/200279
    Introduce get_ports_attributes in OVSBridge

Gerrit topic: https://review.openstack.org/#q,topic:port-update,n,z

Gerrit topic: https://review.openstack.org/#q,topic:bp/restructure-l2-agent1,n,z

Gerrit topic: https://review.openstack.org/#q,topic:bug/1478512,n,z

Addressed by: https://review.openstack.org/207903
    Merge remote-tracking branch 'origin/master' into merge-branch

Gerrit topic: https://review.openstack.org/#q,topic:ancy_port1,n,z

Addressed by: https://review.openstack.org/211192
    Remove cache of registered ports in the OVS agent

Addressed by: https://review.openstack.org/211191
    check_changed_vlans doesn't need registered_ports as param

Addressed by: https://review.openstack.org/211586
    OVS agent set max number of attempts to sync failed devices

Addressed by: https://review.openstack.org/205442
    OVS agent add functional tests of OVS status

Addressed by: https://review.openstack.org/213436
    SimpleInterfaceMonitor handle case when ofport is an empty set

Addressed by: https://review.openstack.org/165023
    ovs agent resync may miss port remove event

Addressed by: https://review.openstack.org/217272
    _bind_devices query only existing ports

Gerrit topic: https://review.openstack.org/#q,topic:bug/1478847,n,z

Addressed by: https://review.openstack.org/224334
    Merge remote-tracking branch 'origin/master' into merge-pecan-resolve-conflict

Gerrit topic: https://review.openstack.org/#q,topic:bug/1486039,n,z

Addressed by: https://review.openstack.org/224357
    Merge remote-tracking branch 'origin/master' into walnut

Gerrit topic: https://review.openstack.org/#q,topic:puzza,n,z

Addressed by: https://review.openstack.org/243808
    Revert "OVS agent reacts to events instead of polling"

Addressed by: https://review.openstack.org/245105
    Revert "Revert "OVS agent reacts to events instead of polling""

Addressed by: https://review.openstack.org/251972
    Trivial fix for test test_noresync_after_port_gone

Addressed by: https://review.openstack.org/257284
    Check if polling manager is active before stopping it

Gerrit topic: https://review.openstack.org/#q,topic:bug/1528895,n,z

Addressed by: https://review.openstack.org/277279
    Revert use of update_device_list

Gerrit topic: https://review.openstack.org/#q,topic:bug/1430999,n,z

Addressed by: https://review.openstack.org/284535
    Faster _get_devices_details_list_and_failed_devices

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.