Jump to: navigation, search

Difference between revisions of "Neutron Plugins and Drivers"

(Existing Plugin and Drivers)
(Existing Plugin and Drivers)
Line 52: Line 52:
 
| Big Switch || ML2 Driver || Kevin Benton || Y || 8.5 hour delay
 
| Big Switch || ML2 Driver || Kevin Benton || Y || 8.5 hour delay
 
|-
 
|-
| Brocade || Vyatta Plugin || Karthik Natarajan || N || Only found logs for ML2. Needs clarification from Brocade
+
| Brocade || Vyatta Plugin || Karthik Natarajan || Y || None
 
|-
 
|-
 
| Brocade || ML2 Driver || Shiv Haris || Y || None
 
| Brocade || ML2 Driver || Shiv Haris || Y || None

Revision as of 17:00, 19 August 2014

Introduction

Ensuring release quality through proper testing is an important tenant of the OpenStack community and Neutron team wants to do our part. We are introducing changes below provide more visibility into the quality and stability of vendor plugin and driver code. The policies described here are in effect immediately.


Rationale

Code proposals for third party plugins have always presented a review challenge for the Neutron core team. In the early days, code was often proposed by core project contributors and our review process only validated whether the requirements were met for community coding style and unit testing. As Neutron has added new resources via extensions, it has become more difficult for Neutron reviewers to ensure the proposed code is functional. Many of the plugins and/or drivers require proprietary hardware and/or software to conduct such testing.

In addition to testing changes, the Neutron team is revising the requirements for the point of contact for third party code. The changes bring the written expectations for contacts in line with current practice.


Plugin and Driver Processes

Getting a plugin or driver merged upstream into Neutron allows you the benefit of being a part of the simultaneous release, and likely having your plugin or driver packaged with distributions which ship releases based on the simultaneous releases from upstream. However, this is not a "free ride", you should ideally be giving back more than you are taking by getting your code upstream. When you submit your code, you're putting a burden on the existing reviews and infrastructure. The same goes with subsequent bug fixes and backports to stable releases. Thus, you should have someone from your company or team reviewing other code upstream, participating in meetings, etc. The following is a list of requirements for inclusion of code upstream:

  • Code which passes review, has adequate unit tests, and passes pep8 guidelines.
  • A functioning CI system, which has been running successfully against your plugin/driver patches, and other patches as well.
  • Participation in Neutron meetings, IRC channels, and email lists.
  • A member of the plugin/driver team participating in code reviews of other upstream code.


Removal of Upstream Plugins or Drivers

If you fail to meet the criteria above, you risk having your plugin or driver removed from upstream. The core team will continue to evaluate third party CI systems to ensure they are running and correctly testing patches for third party plugins and drivers. If they fail to function, an email will be sent to the openstack-dev mailing list asking for the owner to fix this issue. Ideally, the owner will reply to the email thread, and will work with the Neutron and Infra teams to address the CI issue. If no reply is received or if adequate progress is not being made to address the issue within 2 weeks, a process to remove the plugin or driver from upstream will commence. To get your plugin or driver upstream after it has been removed will take a functioning third party CI system running for a month, correctly voting on changes upstream.


Point of Contact Requirements

Each third party plugin and/or driver shall designate a point of contact for each coordinated release cycle. The contact will serve as a liaison between the Neutron core team and the vendor or community supporting the plugin or driver. The contact shall:

  • Attend weekly Neutron team IRC meetings
  • Be an active reviewer and contributor
  • Be an active participant on openstack-dev mailing list
  • Assist the core team with triaging bugs specific to the plugin and/or driver
  • Ensure OpenStack development deadlines are properly communicated back to their company and/or community


Testing Requirements

https://wiki.openstack.org/wiki/NeutronThirdPartyTesting


Existing Plugin and Drivers

Plugins and drivers currently in the Neutron project repository will be given a grace period until the Icehouse-2 milestone to implement external third party testing. At that time, the Neutron team will release a list of the compatible plugins and drivers (i.e. those that meet the testing requirements). Plugins and drivers that do not have external testing will be deprecated at the Icehouse release and will be candidates for removal when the J-release cycle opens.

Vendor Plugin/Driver Name Contact Name Status Notes
Neutron Team ML2 - OVS/LB Infra Team Y Covered by Infra (Jenkins)
A10 Networks LBaaS Driver Doug Wiegley Y None
Arista Networks ML2 Driver Sukhdev Kapur Y None
Big Switch Plugin Kevin Benton Y 8.5 hour delay
Big Switch ML2 Driver Kevin Benton Y 8.5 hour delay
Brocade Vyatta Plugin Karthik Natarajan Y None
Brocade ML2 Driver Shiv Haris Y None
Cisco Plugin - NXOS Dane Leblanc N It will be removed in Juno
Cisco Plugin - N1Kv Dane Leblanc Y None
Cisco ML2 - APIC Dane Leblanc N It is not running against all Neutron commits, results are fake
Cisco ML2 - DFA Dane Leblanc N Results are fake. Needs clarification from Cisco
Cisco ML2 - NXOS Dane Leblanc Y None
Cisco VPNaaS - Driver Dane Leblanc Y None
Embrane Plugin Ignacio Scopetta N No CI in place. Candidate to be removed from Neutron
Embrane LBaaS - Driver Ignacio Scopetta N No CI in place. Candidate to be removed from Neutron
Freescale ML2 - Driver Trinath Somanchi - Needs to be verified
Freescale FWaaS - Driver Trinath Somanchi - Needs to be verified
IBM Plugin - SDN-VE Mohammad Banikazemi - Results are not clear. Need clarification from IBM
Juniper Plugin - Contrail Rudra Rugge Y Needs to start testing all Neutron commits. Plugins just got merged in Juno-3
Mellanox Plugin Omri Marcovitch N It is deprecated. Will be removed in Kilo
Mellanox ML2 - Driver Omri Marcovitch N Logs are not available. Needs clarification
Mellanox ML2 - Sriovnic switch Omri Marcovitch N Logs are not available. Needs clarification
Metaplugin Plugin - Multiple NTT Team N It is missing some Neutron commits
Midokura Plugin - Midonet Lucas Eznarriaga Y None
NEC Plugin - OpenFlow Akihiro Motoki Y None
Citrix NetScaler LBaaS - Driver Parikshit Manur Y Only testing LBaaS changes
Nuage Networks Plugin - VSP Franck Yelles Y Plugin behaving correctly for +1, manual -1
OFAgent ML2 - OF Yamamoto Takashi Y None
OneCoverage Plugin - NVSD Hemanth Ravi N Logs are not available. It is missing some Neutron commits. Needs clarification
OpenDaylight ML2 - ODL  ? - It is missing some Neutron commits
PLUMgrid Plugin - Director Fawad Khaliq Y None
Ryu Plugin - Ryu Controller Yamamoto Takashi - It is been deprecated and It will be removed in Kilo
Radware LBaaS - Driver Samuel Bercovici N CI does not exist. Should be removed from Neutron
Tail-f NCS ML2 - NCS Luke Gorrie - It is missing some Neutron commits
vArmour FWaaS Gary Duan N CI does not exist. Should be removed from Neutron
VMware Plugin - NSX Aaron Rosen - It is missing some Neutron commits.