Jump to: navigation, search

Difference between revisions of "Tracking ML2 Subgroup Reviews"

(→‎Specs Under Review: Added Snabb NFV mech driver)
(→‎Specs proposed to Kilo Release)
 
(139 intermediate revisions by 17 users not shown)
Line 1: Line 1:
 
__NOTOC__
 
__NOTOC__
  
−
Here are tables for tracking the status of the specs related to ML2 in Juno: --[[User:Banix|Banix]] ([[User talk:Banix|talk]]) 01:37, 10 June 2014 (UTC)
+
Here are tables for tracking the status of the specs related to ML2 in current release.
  
 
== The guideline for updating this wiki is at the [[#Guide|end of this page]] ==  
 
== The guideline for updating this wiki is at the [[#Guide|end of this page]] ==  
  
−
 
+
=== Specs proposed to Kilo Release ===  
−
=== Specs Under Review ===  
 
  
 
For categories, please see the description at the bottom.
 
For categories, please see the description at the bottom.
Line 13: Line 12:
 
|-
 
|-
 
!!  scope="col"  class="unsortable"  | Title  
 
!!  scope="col"  class="unsortable"  | Title  
−
!!  scope="col"  class="unsortable"  | Spec/BP
 
−
!!  scope="col" | Type
 
−
!!  scope="col" | Priority
 
 
!!  scope="col" | Owner  
 
!!  scope="col" | Owner  
 +
!! scope="col"| Code
 
!!  scope="col" | Status  
 
!!  scope="col" | Status  
−
!!  scope="col" | R1
+
!!  scope="col" | Comments
−
!!  scope="col" | R2
 
−
!!  scope="col" | R3
 
−
!!  scope="col" | R4
 
−
!!  scope="col" | C1
 
−
!!  scope="col" | C2
 
−
!!  scope="col" | C3
 
−
|-
 
−
| ML2 Type drivers refactor to allow extensiblity || [https://review.openstack.org/#/c/91811/  spec] || Common || 2 (Med)  || asomya || Review || Sukhdev <br /> +1 || yamamoto || shivharis || banix  ||  rkukura || - ||  -
 
−
|-
 
−
| GW API: L2 bridging API - Piece 1: Basic use cases || [https://review.openstack.org/#/c/93613/ spec]  || Common ||  3 (Low) || racha-ben-ali  ||Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| Neutron QoS API Extension || [https://review.openstack.org/#/c/88599/ spec]  || Common || 3 (Low) || sc68cal  || Review || irenab || - || - || - ||  - || - ||  -
 
−
|-
 
−
| physical-network-topology extension spec || [https://review.openstack.org/#/c/91275/  spec]  || Common || 3 (Low) || yamahata  || Review || Sukhdev || yamamoto || asomya || banix  ||  - || - ||  -
 
−
|-
 
−
| VLAN aware VMs || [https://review.openstack.org/#/c/94612/ spec] || Common || 3 (Low)  || Erik Moe || Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| Support for extensions in ML2 Mechanism Drivers || [https://review.openstack.org/#/c/89208/ spec]  [https://blueprints.launchpad.net/neutron/+spec/neutron-ml2-mechanismdriver-extensions BP] || Common || 3 (Low) || nlahouti  || Addressed the review comments and also documented details in[https://etherpad.openstack.org/p/ML2_MD_extensions] and waiting for reply from rkukura|| banix || shivharis || matrohon || yamamato || rkukura <br />  -1  || mestery <br /> <br/> '''''nlahouti: removed -2. Please refer to status column''''' ||  -
 
−
|-
 
−
| Neutron External Ports || [https://review.openstack.org/#/c/87825/ spec] [https://blueprints.launchpad.net/neutron/+spec/neutron-external-ports BP] || Common || 2 (Med) || kevinbenton  || Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| API Extension for l2-gateway || [https://review.openstack.org/#/c/100278/ spec] || Common || 3 (Low)  || yamahata || Review || - || - || - || - || - || - || -
 
−
|-
 
−
| ML2 OVS mechanism driver support portsecurity extension  || [https://review.openstack.org/#/c/99873/ spec] || OVS || 3 (Low) || yamahata  || Review || - || - || - || - || - || - || -
 
−
|-
 
−
| VDP support in OVS Neutron Agent || [https://review.openstack.org/#/c/89728/  spec] || OVS || 3 (Low) || nlahouti  ||  Got 4+ approvals. Provided the clarifications for Akihiro Motoki. Waiting for BP [https://blueprints.launchpad.net/neutron/+spec/vdp-network-overlay] approval. || yamamoto <br /> +1 || Sukhdev <br /> +1 || - || - || mestery  <br /> +2 || - ||  -
 
−
|-
 
−
| Open vSwitch-based Security Groups: OVS FirewallDriver || [https://review.openstack.org/#/c/89712/ spec] || OVS|| 3 (Low)  || asadoughi || Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| Add spec for ML2 mechanism driver for SDN-VE || [https://review.openstack.org/#/c/88101/  spec]  || Driver || 3 (Low) || banix || Review || yamamoto  || irenab || shivharis || - ||  - || - ||  -
 
−
|-
 
−
| ML2 mechanism driver for Cisco UCS Manager || [https://review.openstack.org/#/c/95236/ spec] [https://blueprints.launchpad.net/neutron/+spec/ml2-ucs-manager-mechanism-driver BP] ||  Driver || 3 (Low) ||  sadasu  || Review || Sukhdev <br /> +1 || irenab  || - || -  ||  rkukura || SumitN ||  -
 
−
|-
 
−
| ofagent l2pop support || [https://review.openstack.org/#/c/89463/ spec] ||  Driver || 3 (Low)  || yamamoto || Review || - || - || - || - ||  -  || - ||  -
 
−
|-
 
−
| OFAgent: Merge br-int and br-tun || [https://review.openstack.org/#/c/98702/ spec] ||  Driver  || 3 (Low) || yamamoto || Review || - || - || - || - || - || - ||  -
 
−
|-
 
−
| Linuxbridge QoS Support || [https://review.openstack.org/#/c/96331/ spec] ||  Driver || 3 (Low)  || oda-g || Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| Provider Segment Support for Cisco Nexus Switches || [https://review.openstack.org/#/c/95834 spec]  ||  Driver ||  3 (Low)  || rcurran ||Review || - || - || - || - ||  - || - ||  -
 
−
|-
 
−
| ML2 Cisco Nexus Mechanism Driver VxLAN Gateway Support || [https://review.openstack.org/#/c/98177/ spec]  ||  Driver || 3 (Low)  || rcurran || Review || slogan <br /> +1 || Sukhdev <br /> -1 || - || - ||  - || - ||  -
 
−
|-
 
−
| ML2 mechanism driver for Huawei SDN controller || [https://review.openstack.org/#/c/97490/ spec] ||  Driver || 3 (Low)  || yangxurong || Review || Sukhdev || - || - || - || - || - || -
 
 
|-  
 
|-  
−
| Arista L3 Router Service Plugin || [https://review.openstack.org/#/c/97125/ spec] || not || 3 (Low)  || sukhdev || Review ||  banix  <br /> +1 || - || - || - ||  mastery  <br /> +2 || - || -
+
| [https://review.openstack.org/#/c/99873/ portsecurity extension support] || Isaku Yamahata || [https://review.openstack.org/#/c/129178/ passing db entry extend dict] [https://review.openstack.org/#/c/152759/ passing plugin context to ext driver] [https://review.openstack.org/#/c/152759/ extension_driver][https://review.openstack.org/126552 port-sec-ext][https://review.openstack.org/150835 WIP] || ||  
 
|-  
 
|-  
−
| Add spec for ML2 mechanism driver for Cisco N1KV || [https://review.openstack.org/#/c/103276/  spec]  || Driver || 3 (Low) || abhraut || Review || - || - || - || - ||  - || - ||  -
+
| [https://review.openstack.org/#/c/106185/  ML2 hierarchical port binding]  ||rkukura ||  [https://review.openstack.org/#/c/115110/ api] [https://review.openstack.org/#/c/116122/ db] [https://review.openstack.org/#/c/116209/ logic] || merged || need additional tests
−
|-
 
−
| Layer 3 Service Plugin for Cisco Nexus Switches || [https://review.openstack.org/#/c/95910/ spec] ||  not || 3 (Low)  || rcurran || Review || Sukhdev <br /> -1 || - || - || - ||  - || - ||  -
 
−
|-
 
−
| Add spec for ML2 mechanism driver for Snabb NFV || [https://review.openstack.org/#/c/95711/ spec] || Driver || 3 (Low) || lukego || Review || -  || - || - || - ||  - || - || -
 
 
|}
 
|}
  
−
=== Merged Specs with Code Under Review ===  
+
=== Planned ===  
  
 
{| class="wikitable"
 
{| class="wikitable"
 
|-
 
|-
−
! Title !! Spec/BP !! Owner  
+
! Title/BP !! Owner !! Comment
−
!! style="background: blue; color: white" | Code
 
−
!!  scope="col" | R1
 
−
!!  scope="col" | R2
 
−
!!  scope="col" | R3
 
−
!!  scope="col" | R4
 
−
!!  scope="col" | C1
 
−
!!  scope="col" | C2
 
−
!! scope="col" | C3
 
−
!!  scope="col" | Code Status
 
−
|-
 
−
| Allow partial specification of Provider Network attributes || [https://review.openstack.org/#/c/91540/ spec] || zzelle ||  [https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:bp/provider-network-partial-specs,n,z code] || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| ML2 Mechanism Driver for Cisco DFA || [https://review.openstack.org/#/c/89740/  spec] || nlahouti  || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| ML2 Mechanism Driver for the Cisco APIC || [https://review.openstack.org/#/c/89305/ spec] ||  HenryG  || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| ML2 Mechanism Driver for SR-IOV NIC switching || [https://review.openstack.org/#/c/89763/ spec] || irenab  || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| Neutron OVS based Distributed Virtual Router || [https://review.openstack.org/#/c/89934/ spec] ||  swami  || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| L3 router Service plugin for the Cisco APIC || [https://review.openstack.org/#/c/89608/  spec] ||  asomya  || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|-
 
−
| Freescale SDN Mechanism Driver || [https://review.openstack.org/#/c/88190 spec] || trinaths  || [https://review.openstack.org/#/c/78092 code] || yamamoto || - || - || - || mestery || amotoki  ||  - ||  Merged
 
−
|-
 
−
| ofagent: port monitoring w/o ovsdb accesses || [https://review.openstack.org/#/c/91742/ spec] ||  yamamoto || - || - || - || - || - ||  - || - ||  - ||  -
 
−
|}
 
−
 
 
−
=== Planned ===
 
−
 
 
−
{| class="wikitable"
 
 
|-
 
|-
−
! Title !! Spec/BP !! Owner !! Comment
+
| ML2 Async driver invocation || manishg || [https://review.openstack.org/#/c/154333 poc-code]
−
|-
 
−
| ML2 Hierarchical Port Binding ||   || rkukura  || Under development
 
 
|-  
 
|-  
−
| Modular L2 Agent ||   || banix  || Under development
+
| Modular L2 Agent || banix  || On hold
 
|-  
 
|-  
−
| ofagent: sub driver || [https://blueprints.launchpad.net/neutron/+spec/ofagent-sub-driver BP] ||  yamamoto  || probably some overlap with modular l2 agent  
+
| [https://blueprints.launchpad.net/neutron/+spec/ofagent-sub-driver ofagent: sub driver] ||  yamamoto  || probably some overlap with modular l2 agent  
 
|}
 
|}
  
 
= Guide =
 
= Guide =
  
−
* The owner of a given spec is responsible for keeping the corresponding row in this table up to date
+
* Mechanism Drivers do not need a blueprint starting Kilo Release (?)
−
* A reviewer can add her name to the table when she reviews a spec but ultimately the spec owner needs to make sure the information is up to date
+
* The owner of a given spec is responsible for keeping the corresponding row in this table up to date
−
* Priority
+
 
−
** Use the following strings for priority: "1 (High)", "2 (Med)", "3 (Low)" so we can easily sort the table based on the priority if need be
 
−
** If not sure, insert the spec with priority set to 3 (Low)
 
−
** The priority of specs were decided to be defined as follows ([http://eavesdrop.openstack.org/meetings/networking_ml2/2014/networking_ml2.2014-06-11-16.03.log.html IRC Log]):
 
−
*** So mestery suggests vendor-specific drivers should be at low priority, consistent with vendor plugins and vendor service drivers
 
−
*** The ML2 team can identify 2 or 3 BPs to treat as high priority
 
−
*** These need to be of general community interest, and really important to complete for Juno
 
−
*** And we can identify several more BPs to treat as medium priority, which also should be of general community interest
 
−
* Types
 
−
** Common:  ML2 common features : features which affects ML2 plugin core or are required by multiple drivers
 
−
** OVS:  Open vSwitch mechanism driver / agent : features specific to OVS mech driver or OVS agent. Some blueprints are proposed only for OVS but they are sometimes common to ML2 plugin. I prepared a separate category for OVS (amotoki).
 
−
** Driver:    ML2 mechanism driver specific : features specific to a driver
 
  
 
=== Other Considerations ===
 
=== Other Considerations ===
 
* Should we do the same for regular (not specs) neutron reviews? Will that be too much? We can wait and see if the current table will be used and if it will make the review process any better and then decide.
 
* Should we do the same for regular (not specs) neutron reviews? Will that be too much? We can wait and see if the current table will be used and if it will make the review process any better and then decide.
 
* Any changes we can make to Launchpad and/or Gerrit review that will make the tracking of the specs easier and that will make using a table like this unnecessary?
 
* Any changes we can make to Launchpad and/or Gerrit review that will make the tracking of the specs easier and that will make using a table like this unnecessary?

Latest revision as of 13:41, 18 February 2015


Here are tables for tracking the status of the specs related to ML2 in current release.

The guideline for updating this wiki is at the end of this page

Specs proposed to Kilo Release

For categories, please see the description at the bottom.

Title Owner Code Status Comments
portsecurity extension support Isaku Yamahata passing db entry extend dict passing plugin context to ext driver extension_driverport-sec-extWIP
ML2 hierarchical port binding rkukura api db logic merged need additional tests

Planned

Title/BP Owner Comment
ML2 Async driver invocation manishg poc-code
Modular L2 Agent banix On hold
ofagent: sub driver yamamoto probably some overlap with modular l2 agent

Guide

  • Mechanism Drivers do not need a blueprint starting Kilo Release (?)
  • The owner of a given spec is responsible for keeping the corresponding row in this table up to date


Other Considerations

  • Should we do the same for regular (not specs) neutron reviews? Will that be too much? We can wait and see if the current table will be used and if it will make the review process any better and then decide.
  • Any changes we can make to Launchpad and/or Gerrit review that will make the tracking of the specs easier and that will make using a table like this unnecessary?
Retrieved from "https://wiki.openstack.org/w/index.php?title=Tracking_ML2_Subgroup_Reviews&oldid=73908"