Jump to: navigation, search

Difference between revisions of "Tracking ML2 Subgroup Reviews"

(Specs proposed to Kilo Release)
(Specs proposed to Kilo Release)
 
(12 intermediate revisions by 4 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]] ==  
Line 12: Line 12:
 
|-
 
|-
 
!!  scope="col"  class="unsortable"  | Title  
 
!!  scope="col"  class="unsortable"  | Title  
!!  scope="col" | Type
 
!!  scope="col" | Priority
 
!!  scope="col" | Code
 
 
!!  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
 
!!  scope="col" | Code Status
 
 
|-  
 
|-  
| [https://review.openstack.org/#/c/99873/ portsecurity extension support] || || || || Isaku Yamahata || || || || || || || || ||
+
| [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] || ||  
|-
 
|  [https://review.openstack.org/139859 neutron port traffic rate support for SR-IOV NIC] || || || || Samer Deeb || || || || || || || || ||
 
|-
 
|  [https://review.openstack.org/138786 Kilo ML2 type driver refactoring] || || || || Arvind Somya || || || || || || || || ||
 
|-
 
|  [https://review.openstack.org/138849 ML2 Mechanism Driver for Mellanox SDN] || || || || Nurit Vilosny || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/#/c/138581/ Blueprint for ML2 Mech Driver for VDX non-AMPP mode] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/140261 Spec for Brocade IronwareOS ML2 mechanism driver]  || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/140257 Spec for Brocade SVI plugin for MLX]  || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/140138 Cisco routing service using vif hotplugging] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/130737 New ML2 mechanism driver for 'Calico'-style VM connectivity] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/139204 ML2 Cisco Nexus VxLAN type driver] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/137990 Specification for ml2-huawei-switch-mech-driver] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/101124 Neutron ML2 mechanism driver for VCenter DVS network creation] || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/132363 Add spec for ML2 host IP support in port bindings] || || || || || || || || || || || || ||
 
|-
 
|| DEFERED From Juno To Kilo  || || || || || || || || || || || || ||
 
|-
 
| [https://review.openstack.org/#/c/93613/ GW API: L2 bridging API - Piece 1: Basic use cases]  || || Common ||  3 (Low) || || racha-ben-ali  ||  || - || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/88599/ Neutron QoS API Extension]  || || Common || 3 (Low) || || sc68cal  || || irenab || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/91275/  physical-network-topology extension spec]  || || Common || 3 (Low) || || yamahata  || || Sukhdev || yamamoto || asomya || banix  ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/94612/ VLAN aware VMs] || || Common || 3 (Low)  || || Erik Moe || || - || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/100278/ API Extension for l2-gateway] || || Common || 3 (Low)  || || yamahata || || - || - || - || - || - || - || -
 
|-
 
| [https://review.openstack.org/#/c/99873/ ML2 OVS mechanism driver support portsecurity extension] || || OVS || 3 (Low) || || yamahata  || || - || - || - || - || - || - || -
 
|-
 
| [https://review.openstack.org/#/c/89712/ Open vSwitch-based Security Groups: OVS FirewallDriver] || || OVS|| 3 (Low)  || || asadoughi ||  || - || - || - || - ||  - || - ||  -
 
|-
 
| [https://review.openstack.org/#/c/96331/ Linuxbridge QoS Support ] || ||  Driver || 3 (Low) || || oda-g ||  || - || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/95834 Provider Segment Support for Cisco Nexus Switches] || ||  Driver ||  3 (Low)  || || rcurran ||  || gessau <br/> +1 || pcm <br/> +1 || - || - ||  - || - ||  -
 
|-
 
| [https://review.openstack.org/#/c/98177/ ML2 Cisco Nexus Mechanism Driver VxLAN Gateway Support]  || ||  Driver || 3 (Low)  || || rcurran ||  || gessau <br /> +1 || pcm <br /> +1 || asomya <br/> +1 || - ||  - || - ||  -
 
|-
 
| [https://review.openstack.org/#/c/95910/ Layer 3 Service Plugin for Cisco Nexus Switches]  || ||  || 3 (Low) || || rcurran ||  || Sukhdev <br /> +1 || pcm <br/> +1  || gessau <br/> +1  || asomya <br/> +1  ||  - || - ||  -
 
|-
 
| [https://review.openstack.org/#/c/106185/  ML2 hierarchical port binding]  || || || 1 (High) || [https://review.openstack.org/#/c/115110/ api] [https://review.openstack.org/#/c/116122/ db] [https://review.openstack.org/#/c/116209/ logic] || rkukura ||  || - || - || - ||  - || - ||  - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/87825/  Neutron External Ports]  || || ||  2 (Med) || [https://review.openstack.org/#/c/97173/ code] [https://review.openstack.org/116187 code2] || kevinbenton ||  || - || - || - ||  - || - ||  - ||  -
 
|-
 
| [https://review.openstack.org/#/c/89728/ VDP Support in OVS Neutron Agent]n || ||  || 3 (Low) ||  [https://review.openstack.org/#/c/105563/ code1] [https://review.openstack.org/#/c/115442/ code2] [https://review.openstack.org/#/c/115452/ code3] || padkrish ||  || Yamamoto || - || - || - ||  amotoki || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/97490/ ML2 mechanism driver for Huawei SDN controller] || || ||  3 (Low) || [https://review.openstack.org/#/c/104103/ code] || yangxurong ||  || - || - || - || - ||  - || - ||  -
 
|-
 
| [https://review.openstack.org/#/c/95236/  ML2 mechanism driver for Cisco UCS Manager] || || ||  3 (Low) || [https://review.openstack.org/#/c/74134/ code] || sadasu ||  || - || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/132226/  Add spec for ML2 mechanism driver for SDN-VE]  || || ||  3 (Low) || N/A || mprabhu ||  || Yamamoto <br/> +1 || banix <br/> +1 || - || - ||  Mark || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/95707/ Add spec for ML2 mechanism driver for Snabb NFV] || || || 3 (Low) ||  [https://review.openstack.org/#/c/95711/ code] || lukego || ||- || - || - || - ||  - || - ||  -
 
|-
 
[https://review.openstack.org/#/c/103276/ Add spec for ML2 mechanism driver for Cisco N1KV] || || ||  3 (Low) || N/A || abhraut ||  || - || - || - || - ||  - || - ||  -
 
|-
 
|  [https://review.openstack.org/#/c/138742/ DPDKOVS - ML2 DPDK OVS Support] || || Driver ||  3 (Low) || N/A || sean-k-mooney || new|| - || - || - || - ||  - || - ||  -
 
|}
 
 
 
=== Merged Specs along with Merged Code in Juno ===
 
 
 
{| class="wikitable sortable"
 
|-
 
!!  scope="col" class="unsortable"  | Title
 
!!  scope="col" class="unsortable"  | Spec/BP
 
!! scope="col" | Owner
 
!!  scope="col" | Priority
 
!! 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
 
 
 
|-
 
| ML2 Type drivers refactor to allow extensiblity  || [https://review.openstack.org/#/c/91811/  spec]  || asomya ||  2 (Med) || [https://review.openstack.org/#/c/110404/ abandoned]  [https://review.openstack.org/#/c/114666/ code1] [https://review.openstack.org/#/c/115025/ code2] [https://review.openstack.org/#/c/115151/ code3] || - || - || - || - ||  - || - ||  - ||  code1 & code2 merged
 
|-
 
| ML2 Mechanism Driver for Cisco DFA || [https://review.openstack.org/#/c/89740/  spec] || nlahouti  ||  3 (Low) || [https://review.openstack.org/#/c/103281/ code1] [https://review.openstack.org/#/c/110065/ code2] [https://review.openstack.org/#/c/111761/ code3] [https://review.openstack.org/#/c/111863/ code4] || banix || yamamoto || rossella_s || - ||  - || - ||  - ||  code1 merged
 
|-
 
| ofagent: port monitoring w/o ovsdb accesses || [https://review.openstack.org/#/c/91742/ spec] ||  yamamoto ||  3 (Low) || [https://review.openstack.org/#/c/108915/ 108915 (devstack)]  || - || - || - || - ||  - || - ||  - || Merged
 
|-  
 
| ofagent l2pop support || [https://review.openstack.org/#/c/89463/ spec] || yamamoto ||  3 (Low) || all merged || - || - || - || - ||  - || - ||  - || Merged
 
|-
 
| OFAgent: Merge br-int and br-tun || [https://review.openstack.org/#/c/98702/ spec] || yamamoto ||  3 (Low) || all merged || - || - || - || - ||  - || - ||  - || Merged
 
|-
 
| ofagent: physical interface mappings || [http://git.openstack.org/cgit/openstack/neutron-specs/tree/specs/juno/ofagent-physical-interface-mappings.rst spec] || yamamoto ||  3 (Low) || all merged || - || - || - || - ||  - || - ||  - || Merged
 
|-
 
| Support for extensions in ML2 || [http://git.openstack.org/cgit/openstack/neutron-specs/tree/specs/juno/neutron-ml2-mechanismdriver-extensions.rst spec] || nlahouti ||  3 (Low) || [https://review.openstack.org/#/c/89211/ code] || - || - || - || - ||  - || - ||  - || Merged
 
|-
 
| Arista L3 Router Service Plugin  || [https://review.openstack.org/#/c/97125/ spec] || sukhdev ||  3 (Low) || [https://review.openstack.org/#/c/114081/ code] || - || - || - || - ||  - || - || - ||  Merged
 
|-
 
| Layer 3 Service Plugin for Brocade VDX Switches  || [https://review.openstack.org/#/c/105250/ spec] || shivharis || 3 (Low) || [https://review.openstack.org/#/c/114801/ code] || - || - || - || - ||  - || - ||  - ||  Merged
 
|-
 
| Allow partial specification of Provider Network attributes || [https://review.openstack.org/#/c/91540/ spec] || zzelle ||  3 (Low) || [https://review.openstack.org/#/c/71904/ code1] [https://review.openstack.org/#/c/74055/ code2] [https://review.openstack.org/#/c/101467/ code3] || - || - || - || - ||  - || - ||  - || Merged
 
|-
 
| ML2 Mechanism Driver for the Cisco APIC || [https://review.openstack.org/#/c/89305/ spec] ||  HenryG  || 3 (Low) ||  [https://review.openstack.org/#/c/73355/ code1] [https://review.openstack.org/#/c/73372/ Code2] || - || - || - || - ||  - || - ||  - ||  Merged
 
|-
 
| ML2 Mechanism Driver for SR-IOV NIC switching || [https://review.openstack.org/#/c/89763/ spec] || irenab  ||  3 (Low) || [https://review.openstack.org/#/c/74464 code1] [https://review.openstack.org/#/c/107651 code2] || - || - || - || - ||  - || - ||  - ||  Merged
 
|-
 
| Neutron OVS based Distributed Virtual Router || [https://review.openstack.org/#/c/89934/ spec] ||  swami  ||  1 (High) || (DVR subteam) || - || - || - || - ||  - || - ||  - ||  ML2 related review is merged
 
|-
 
| Freescale SDN Mechanism Driver || [https://review.openstack.org/#/c/88190 spec] || trinaths  ||  3 (Low) || [https://review.openstack.org/#/c/78092 code] || yamamoto || - || - || - || mestery || amotoki  ||  - || Merged
 
 
|-  
 
|-  
 +
| [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
 
|}
 
|}
  
Line 148: Line 26:
 
{| class="wikitable"
 
{| class="wikitable"
 
|-
 
|-
! Title !! Spec/BP !! Owner !! Comment  
+
! Title/BP !! Owner !! Comment  
 +
|-
 +
| ML2 Async driver invocation || manishg || [https://review.openstack.org/#/c/154333 poc-code]
 
|-  
 
|-  
| 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  
 
|}
 
|}
  
Line 159: Line 39:
 
* Mechanism Drivers do not need a blueprint starting Kilo Release (?)
 
* 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
 
* The owner of a given spec is responsible for keeping the corresponding row in this table up to date
* 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
+
 
* 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?