Difference between revisions of "Meetings/ML2"

Revision as of 15:55, 26 March 2014

Weekly on Wednesdays at 1600 UTC on #openstack-meeting-alt

Agenda

Please note the deadline for Juno design summit session proposals is April 20th: http://lists.openstack.org/pipermail/openstack-dev/2014-March/029319.html (banix)

Meeting March 26, 2014

Action items from last week:
- None
Migration BP
- migration - FFE - https://review.openstack.org/#/c/76533/ - merged
Other Bugs:
Bugs for icehouse
- Port binding details - https://bugs.launchpad.net/neutron/+bug/1276395
  - partial fix (add new properties) - https://review.openstack.org/#/c/76363/ - merged
  - remainder (binding details in delete_port_*commit() ops) - https://review.openstack.org/#/c/79511/ - merged
- bind_port() called in transaction - https://bugs.launchpad.net/neutron/+bug/1276391
  - partial fix (remove validate_port_binding() and unbind_port() ops) - https://review.openstack.org/#/c/79511/ - merged
  - remainder (move binding outside transaction) - https://review.openstack.org/#/c/82945/ - WIP
    - Required minor fixes to mechanism drivers and tests - see REVISIT comments
    - Update later today with cleanups, record locking, etc.
    - Not clear if it will make icehouse
- Error handling for update ops - https://bugs.launchpad.net/neutron/+bug/1227336
  - https://review.openstack.org/#/c/69792/ - in review
  - Needs ML2 and core reviews
  - Consensus that undoing operation makes sense if postcommit() ops fail?
  - Do only drivers already called before failure see undo?
  - Defer to juno?
- Binding failed - https://bugs.launchpad.net/neutron/+bug/1244255 - closed
- VIF security - https://bugs.launchpad.net/nova/+bug/1112912 - in review
  - Resistance to merging nova side - just control via nova config?
  - Email thread and IRC discussion
  - Nachi suggested discussion on #openstack-neutron after this meeting

Meeting March 19, 2014

Action items from last week:
- None
Migration BP
- migration - FFE - https://review.openstack.org/#/c/76533/
  - (marun) Updated to support LB, please review.
Other Bugs:
Bugs for icehouse
- Port binding details - https://bugs.launchpad.net/neutron/+bug/1276395
  - partial fix (add new properties) - https://review.openstack.org/#/c/76363/ - merged
  - remainder (binding details in delete_port_*commit() ops) - https://review.openstack.org/#/c/79511/ - approved
- bind_port() called in transaction - https://bugs.launchpad.net/neutron/+bug/1276391
  - partial fix (remove validate_port_binding() and unbind_port() ops) - https://review.openstack.org/#/c/79511/ - approved
  - remainder (move binding outside transaction) - in progress, hopefully in review today
- Error handling for update ops - https://bugs.launchpad.net/neutron/+bug/1227336
  - https://review.openstack.org/#/c/69792/ - in review
  - Needs ML2 and core reviews
  - Consensus that undoing operation makes sense if postcommit() ops fail?
- VIF security - https://bugs.launchpad.net/nova/+bug/1112912 - in review
- Binding failed - https://bugs.launchpad.net/neutron/+bug/1244255 - unassigned?
  - Will https://review.openstack.org/#/c/80829/ help?
Final mechanism driver status for icehouse:
- cisco apic - deferred - https://review.openstack.org/#/c/73355/, https://review.openstack.org/#/c/73372/
- cisco ucs - deferred
- freescale - deferred - https://review.openstack.org/#/c/78092/ (former reviews: https://review.openstack.org/#/c/71868/, https://review.openstack.org/#/c/73985/, https://review.openstack.org/#/c/70089/)
- huawei - deferred
- SRIOV mech driver - deferred - https://review.openstack.org/#/c/74464/

Meeting March 12, 2014

Action items from last week:
- None
Final mechanism driver status for icehouse
- bigswitch - merged - https://review.openstack.org/#/c/64944/
- brocade - merged - https://review.openstack.org/#/c/60129/
- cisco apic - deferred - https://review.openstack.org/#/c/73355/, https://review.openstack.org/#/c/73372/
- cisco ucs - deferred
- freescale - deferred - https://review.openstack.org/#/c/78092/ (former reviews: https://review.openstack.org/#/c/71868/, https://review.openstack.org/#/c/73985/, https://review.openstack.org/#/c/70089/)
- huawei - deferred
- mellanox - merged - https://review.openstack.org/#/c/72854/
- open daylight (odl) - merged - https://review.openstack.org/#/c/69775/
- openflow agent - merged - https://review.openstack.org/#/c/71791/
- SRIOV mech driver - deferred - https://review.openstack.org/#/c/74464/
Other BPs for icehouse
- migration - FFE - https://review.openstack.org/#/c/76533/
  - (marun) Updated to support LB, please review.
- partial specs - deferred - https://review.openstack.org/#/c/71904/, https://review.openstack.org/#/c/74055/
Bugs for icehouse
- Port binding details - https://bugs.launchpad.net/neutron/+bug/1276395
  - partial fix (add new properties) - https://review.openstack.org/#/c/76363/ - merged
  - remainder (binding details in delete_port_*commit() ops) - will be addressed in update to https://review.openstack.org/#/c/79511/ today - in progress
- bind_port() called in transaction - https://bugs.launchpad.net/neutron/+bug/1276391
  - partial fix (remove validate_port_binding() and unbind_port() ops) - https://review.openstack.org/#/c/79511/ - in review, update today
  - remainder (move binding outside transaction) - in progress, hopefully in review today
- Error handling for update ops - https://bugs.launchpad.net/neutron/+bug/1227336
  - https://review.openstack.org/#/c/69792/ - in review
  - Needs ML2 and core reviews
  - Consensus that undoing operation makes sense if postcommit() ops fail?
- VIF security - https://bugs.launchpad.net/nova/+bug/1112912 - in review
- Binding failed - https://bugs.launchpad.net/neutron/+bug/1244255 - unassigned?
- Agent model migration - https://bugs.launchpad.net/neutron/+bug/1260224 - in review
- L2-pop live migration - https://bugs.launchpad.net/neutron/+bug/1237841 - review abandoned?
  - here is the good review : https://review.openstack.org/#/c/61767/ (will be rebased asap)
Distributed Virtual Router
- L2 plugin changes needed to enable DVR. See the DVR section of L3 subteam page for details.
- Here is the document specifically about L2 changes
Juno design summit session proposals
- Submit and read proposals at http://summit.openstack.org/
- I'll submit a general ML2 update/planning/catch-all session
- Unfinished items from icehouse?
- Brainstorm?
Open Discussion

Meeting March 5, 2014

Action items from last week:
- None
Mechanism driver status for icehouse
- bigswitch - merged - https://review.openstack.org/#/c/64944/
- brocade - approved - https://review.openstack.org/#/c/60129/
- cisco apic - deferred? - https://review.openstack.org/#/c/73355/, https://review.openstack.org/#/c/73372/
- cisco ucs - deferred
- freescale - deferred - https://review.openstack.org/#/c/78092/ (former reviews: https://review.openstack.org/#/c/71868/, https://review.openstack.org/#/c/73985/, https://review.openstack.org/#/c/70089/)
- huawei - deferred
- mellanox - merged - https://review.openstack.org/#/c/72854/
- open daylight (odl) - FFE - https://review.openstack.org/#/c/69775/
- openflow agent - merged - https://review.openstack.org/#/c/71791/
- SRIOV mech driver: https://review.openstack.org/#/c/74464/
Other BPs for icehouse
- migration - FFE - https://review.openstack.org/#/c/76533/
  - (marun) Updated to support LB, please review.
- partial specs - deferred? - https://review.openstack.org/#/c/71904/, https://review.openstack.org/#/c/74055/
Bugs for icehouse
- https://bugs.launchpad.net/neutron/+bug/1276395, https://review.openstack.org/#/c/76363/ - approved
- https://bugs.launchpad.net/neutron/+bug/1276391 - in progress
- https://review.openstack.org/#/c/69792/ Deals with fails in update-*-postcommit ops
Distributed Virtual Router
- L2 plugin changes needed to enable DVR. See the DVR section of L3 subteam page for details.
- Here is the document specifically about L2 changes
Open Discussion

Meeting February 26, 2014

Action items from last week:
- ML2 team to review Port Binding document and comment inline
  - https://docs.google.com/document/d/1k8tAqfQr8Ujzx5TzpXTYEUaym-0U-pE3YL9wjrS8cDg/edit#heading=h.cpck9ip3dbdv
- marun and rkukura to proceed with migration BP and get an extension from markmcclain
  - Initial patch posted for review, please take a look! https://review.openstack.org/#/c/76533/
    - [marun]: I'm afraid I can't attend the meeting but will be back online later today.
- All ML2 MechanismDrivers which bind ports should ensure they look to make sure the binding:vnic_type attribute is something they handle.
Port binding bug fixes which may affect MDs:
- https://bugs.launchpad.net/neutron/+bug/1276395
- https://bugs.launchpad.net/neutron/+bug/1276391
Open Discussion

Meeting February 19, 2014

Action items from last week:
- rkukura to move port binding discussion into Google Doc
- matrohon to file BP for multi-node testing and bring this up in the infra meeting
Migration tool
- BP: https://blueprints.launchpad.net/neutron/+spec/ml2-deprecated-plugin-migration
- marun volunteered to own this working with rkukura
SR-IOV BPs in review:
Port binding bug fixes which may affect MDs:
- https://bugs.launchpad.net/neutron/+bug/1276395
- https://bugs.launchpad.net/neutron/+bug/1276391
Proposed patch to ML2 to move away from providernet extension into the multi-provider extension with arbitrary attributes
- https://review.openstack.org/#/c/37893/

Meeting February 12, 2014

Action items from last week:
- rkukura to put result of binding changes email discussion into a wiki or google doc
- rkukura to change priorities of all new 3rd party MD BPs to low if they do not already have jenkins job in place
  - https://blueprints.launchpad.net/neutron/+spec/brocade-ml2-mechanism-driver - medium, approved, in review, have jenkins
  - https://blueprints.launchpad.net/neutron/+spec/bsn-ml2-mechanism-driver - medium, approved, no review, no jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/huawei-ml2-mechanism-driver - medium, approved, in review, no jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/ml2-cisco-apic-mechanism-driver - medium, approved, no review, have jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/ml2-opendaylight-mechanism-driver - medium, approved, in review, have jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/ml2-ucs-manager-mechanism-driver - medium, approved, no review, have jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/mlnx-ml2-support - medium, approved, in review, have jenkins
  - https://blueprints.launchpad.net/neutron/+spec/ryu-ml2-driver - medium, approved, in review, have jenkins
  - https://blueprints.launchpad.net/neutron/+spec/bigswitch-ml2-driver - low, approved, in review, no jenkins?, relationship to bsn-ml2-mechanism-driver?
  - https://blueprints.launchpad.net/neutron/+spec/fsl-sdn-os-mech-driver - low, approved, in review, no jenkins?
  - https://blueprints.launchpad.net/neutron/+spec/huawei-ml2-evs-mechanism-driver - low, approved, no review, no jenkins, relationship to huawei-ml2-mechanism-driver?
  - https://blueprints.launchpad.net/neutron/+spec/netron-ml2-mechnism-driver-for-cisco-dfa-support - deferred to juno
ML2 Exceptions to UserSpace
- https://bugs.launchpad.net/neutron/+bug/1273730
BPs
- Provider network partial specs: zzelle@irc
- https://wiki.openstack.org/wiki/Provider-network-partial-specs
- https://review.openstack.org/71904
Blueprints
- https://blueprints.launchpad.net/neutron?searchtext=ml2
- 3 high, 13 medium, 3 low, 13 undefined priorities
- 19 not yet approved for icehouse
- all MDs should probably be same priority
- Which should we approve for icehouse, drop from icehouse or adjust priority?
Bugs
- https://bugs.launchpad.net/neutron/+bugs?field.tag=ml2
ovs-firewall security group update: asadoughi
Tests
- Add the capabality to test core MD (l2-pop, ovs, lb) by the gate

Previous Agenda

ML2 Sessions Etherpads from Icehouse summit:
- Extensibility of ML2 (Wednesday 4:40PM): https://etherpad.openstack.org/p/ML2_Extensibility
- ML2 QoS and Deprecated Plugin Migration (Wednesday 5:30PM): https://etherpad.openstack.org/p/neutron-qos-hk and https://etherpad.openstack.org/p/ML2_Deprecated_Plugin_Migration
- ML2 Multiple Backend Support (Friday 9:50AM)
- ML2 Mechanism Drivers and Modular Agents (Friday 1:30PM)
devstack ML2 patches
- Default devstack to ML2: https://bugs.launchpad.net/devstack/+bug/1220745
- Correctly set L3 service plugin for ML2: https://review.openstack.org/#/c/48522/
Documentation
- Need to focus on this now
- Documentation BP link: https://blueprints.launchpad.net/openstack-manuals/+spec/neutron-ml2-docs
- ML2 documentation bugs link: https://bugs.launchpad.net/openstack-manuals/+bugs?field.tag=ml2
- Need to document ML3 with ML2
Testing and bug fixes
- Testing of ML2 by everyone would be appreciated
- We need to find and fix any bugs in the next couple of weeks

Icehouse Design Summit Topic ideas
- RPC handling in ML2 for Type and Mechanism Drivers
- More extensible TypeDrivers for controller-based MechanismDrivers
- Monolithic Plugin vs. ML2 MechanismDriver - current and future plugins
- Future Directions for ML2 (orchestration, deployment, management, ...)
- Migration from deprecated Plugins (openvswitch, linuxbridge) to ML2 - automation?
- Add support for VXLAN multicast to the OVS agent with newer versions of OVS and Linux kernel.
- Multi-segment provider network implementations in MechanismDrivers, specifically OVS.
- Additional provider network attributes for controller-based plugins such as ODL for networks created outside of Neutron.

Icehouse ML2 work for reference

ML2 TypeDriver Extra Port Info
- https://blueprints.launchpad.net/neutron/+spec/ml2-typedriver-extra-port-info
Campus Network Blueprint and ML2 integration
- https://blueprints.launchpad.net/neutron/+spec/campus-network
OpenDaylight Driver
- https://blueprints.launchpad.net/quantum/+spec/ml2-opendaylight-mechanism-driver

Questions?

Previous Meeting Logs

Available here: ML2 Meeting Logs

General ML2 Information

Here are the current ml2-specific blueprints for havana:

These tunnel-related blueprints effect the openvswitch and/or linuxbridge agents, and the features should also (or exclusively) be supported (using both agents simultaneously where applicable) by the ml2 plugin:

These blueprints involve the ml2 MechanismDriver API:

@@ Line 15: / Line 15: @@
 *** partial fix (remove validate_port_binding() and unbind_port() ops) - https://review.openstack.org/#/c/79511/ - merged
 *** remainder (move binding outside transaction) - https://review.openstack.org/#/c/82945/ - WIP
+**** Required minor fixes to mechanism drivers and tests - see REVISIT comments
+**** Update later today with cleanups, record locking, etc.
+**** Not clear if it will make icehouse
 ** Error handling for update ops - https://bugs.launchpad.net/neutron/+bug/1227336
 *** https://review.openstack.org/#/c/69792/ - in review
 *** Needs ML2 and core reviews
 *** Consensus that undoing operation makes sense if postcommit() ops fail?
+*** Do only drivers already called before failure see undo?
+*** Defer to juno?
+** Binding failed - https://bugs.launchpad.net/neutron/+bug/1244255 - closed
 ** VIF security - https://bugs.launchpad.net/nova/+bug/1112912 - in review
-** Binding failed - https://bugs.launchpad.net/neutron/+bug/1244255 - unassigned?
+*** Resistance to merging nova side - just control via nova config?
-*** Will https://review.openstack.org/#/c/80829/ help?
+*** Email thread and IRC discussion
+*** Nachi suggested discussion on #openstack-neutron after this meeting
 === Meeting March 19, 2014 ===