Open vSwitch Agent should use ovs_lib to list bridges not ip_lib

Bug #1331569 reported by sean mooney on 2014-06-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Low
sean mooney

Bug Description

Problem description
===================
In the Open vSwitch neutron agent, ip_lib is currently used to list the bridges.
as userspace only vswiches do not create bridge local ports in the kernel ip_lib will not correctly detect the bridge configuration.

Proposed Change
==============
use of ip_lib blocks reuse of the Open vSwitch agent with userspace only open vSwitchs implementations.
to enable this use case ovs_lib.get_bridges should be used instead.

summary: - Open vSwitch Agent should use ovs_lib to list bridgesnot ip_lib
+ Open vSwitch Agent should use ovs_lib to list bridges not ip_lib
description: updated
Changed in neutron:
status: New → Triaged
importance: Undecided → Low
Changed in neutron:
assignee: nobody → sean mooney (sean-k-mooney)
Changed in neutron:
status: Triaged → In Progress
Kyle Mestery (mestery) on 2014-06-27
Changed in neutron:
milestone: none → juno-2

Reviewed: https://review.openstack.org/101965
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=1b46d36269adc5c64c8cd55a2d9f2c47e42b341f
Submitter: Jenkins
Branch: master

commit 1b46d36269adc5c64c8cd55a2d9f2c47e42b341f
Author: Sean Mooney <email address hidden>
Date: Fri Jun 27 11:18:48 2014 +0100

    changes ovs agent to get bridges via ovs_lib

    ip_lib is currently used to list the bridges
    in the Open vSwitch neutron agent.

    use of ip_lib blocks reuse of the Open vSwitch agent
    with userspace only open vSwitchs implementations.

    this patch replaces calls to ip_lib with ovs_lib.get_bridges

    Closes-Bug: #1331569
    Change-Id: I5935d39b314055063a64266bda0cc4c2d1ac05fc

Changed in neutron:
status: In Progress → Fix Committed
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2014-10-16
Changed in neutron:
milestone: juno-2 → 2014.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers