ovsdb managers shouldn't be erased/overwritten

Bug #1614766 reported by Isaku Yamahata
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Sridhar Gaddam

Bug Description

neutron(ovs or dhcp) agent with native ovslib sets to its own ovsdb managers.

In some use cases, cloud admin sets ovsdb managers. e.g. monitoring/debug purses, sdn controller.
neutron agents shouldn't erase those settings.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

Fix proposed to branch: master
Review: https://review.openstack.org/357528

Changed in neutron:
assignee: nobody → Isaku Yamahata (yamahata)
status: New → In Progress
summary: - ovsdb managers shouldn't bbe erased/overwritten
+ ovsdb managers shouldn't be erased/overwritten
tags: added: ovs
Changed in neutron:
assignee: Isaku Yamahata (yamahata) → Sridhar Gaddam (sridhargaddam)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/357528
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=7d42176853c80545df46c3fa593d7816f12e57cf
Submitter: Jenkins
Branch: master

commit 7d42176853c80545df46c3fa593d7816f12e57cf
Author: Isaku Yamahata <email address hidden>
Date: Thu Aug 18 15:32:12 2016 -0700

    ovsdb: don't erase existing ovsdb managers

    The current existing agent erases already set ovsdb managers
    entries. In some use cases, cloud admin sets ovsdb managers.
    eg, for SDN controllers or monitoring purpose. Neutron agent
    shouldn't unconditionally erase the existing ovsdb managers.

    This patch implements a new api add_manager (along with
    get_manager and remove_manager) to the ovsdb api which will
    allow us to configure a manager on a switch without overriding
    the existing managers.

    Closes-Bug: #1614766
    Change-Id: Ibf9bd02fac3070d166546cac478ef984e4e43f28
    Co-Authored-By: sridhargaddam <email address hidden>
    Co-Authored-By: Terry Wilson <email address hidden>

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/403672

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/newton)

Reviewed: https://review.openstack.org/403672
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=e41cf43fc19458e672906c777d6656f08fdcd3f4
Submitter: Jenkins
Branch: stable/newton

commit e41cf43fc19458e672906c777d6656f08fdcd3f4
Author: Isaku Yamahata <email address hidden>
Date: Thu Aug 18 15:32:12 2016 -0700

    ovsdb: don't erase existing ovsdb managers

    The current existing agent erases already set ovsdb managers
    entries. In some use cases, cloud admin sets ovsdb managers.
    eg, for SDN controllers or monitoring purpose. Neutron agent
    shouldn't unconditionally erase the existing ovsdb managers.

    This patch implements a new api add_manager (along with
    get_manager and remove_manager) to the ovsdb api which will
    allow us to configure a manager on a switch without overriding
    the existing managers.

    Closes-Bug: #1614766
    Change-Id: Ibf9bd02fac3070d166546cac478ef984e4e43f28
    Co-Authored-By: sridhargaddam <email address hidden>
    Co-Authored-By: Terry Wilson <email address hidden>
    (cherry picked from commit 7d42176853c80545df46c3fa593d7816f12e57cf)

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 10.0.0.0b2

This issue was fixed in the openstack/neutron 10.0.0.0b2 development milestone.

tags: added: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 9.2.0

This issue was fixed in the openstack/neutron 9.2.0 release.

tags: removed: neutron-proactive-backport-potential
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.