yahoo-eng-team team mailing list archive

[OpenStack Infra <1622644@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/372272
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=039673c2a412282eba9c26f9ff4ac148748ebfb4
Submitter: Jenkins
Branch:    master

commit 039673c2a412282eba9c26f9ff4ac148748ebfb4
Author: Thomas Morin <thomas.morin@xxxxxxxxxx>
Date:   Mon Sep 19 09:39:57 2016 +0200

    OVS agent: configure both OF10 and OF13
    
    This change avoids issues where a piece of code restricts
    a bridge to OF13 while there is code still needing OF10, and
    vice-versa, by configuring bridge to both versions.
    
    This is aimed to be a less complex and easier to merge fix than
    Id5ac7e6431c97fc70d8404b16f89533b6f270eee.
    
    Change-Id: I4475865c4f83cb9f3e12c709af752bc490692ca3
    Closes-Bug: 1622644


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1622644

Title:
  OVS agent ryu/native implementation breaks non-OF1.3 uses

Status in networking-bgpvpn:
  Confirmed
Status in BaGPipe:
  New
Status in neutron:
  Fix Released

Bug description:
  The ryu-based OVS agent variant forces the bridge Openflow version to 1.3 only [1], which breaks a few things:
  - troubleshooting tools relying on ovs-ofctl, unless they specify "-O Openflow13", will break:
    version negotiation failed (we support version 0x01, peer supports version 0x04)
    ovs-ofctl: br-tun: failed to connect to socket (Broken pipe)

  - calling add_flow on an OVSCookieBridge derived from a bridge that is
  an native.ovs_bridge.OVSAgentBridge, will fail with the same error,
  because add_flow will call ovs-ofctl without specifying "-O
  Openflow13"  (this issue is currently hitting networking-bgpvpn: [2])

  It seems like a possible fix would be to not restrict the set of
  Openflow versions supported by the bridge to Openflow13, but to just
  *add* Openflow13 to the set of supported versions.

  [1] https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/agent/openflow/native/ovs_bridge.py#L78
  [2] https://github.com/openstack/networking-bagpipe/blob/master/networking_bagpipe/agent/bagpipe_bgp_agent.py#L512

To manage notifications about this bug go to:
https://bugs.launchpad.net/bgpvpn/+bug/1622644/+subscriptions