ovsdb_timeout config option isn't used to set when adding manager

Bug #1868686 reported by Slawek Kaplonski
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Rodolfo Alonso

Bug Description

It seems that method add_manager, which is in [1] is not used at all.
In the past it was used by function "enable_connection_uri" from neutron.agent.ovsdb.native.helpers module but commit [2] switched it to use helper function from ovsdbapp.

[1] https://opendev.org/openstack/neutron/src/branch/master/neutron/agent/common/ovs_lib.py#L122
[2] https://review.opendev.org/#/c/453014/

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

Fix proposed to branch: master
Review: https://review.opendev.org/720711

Changed in neutron:
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/720711
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=de4c10882fc31f91c58f341d1e25cc769edac5af
Submitter: Zuul
Branch: master

commit de4c10882fc31f91c58f341d1e25cc769edac5af
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Fri Apr 17 13:22:00 2020 +0000

    Set OVS manager command timeout and inactiviy probe

    When creating the OVS manager, define the command timeout
    (CONF.OVS.ovsdb_timeout) and inactivity probe time
    (CONF.OVS.of_inactivity_probe)

    NOTE: CONF.OVS.of_inactivity_probe is defined in seconds but the
    parameter should be passed to ovs-vsctl in milliseconds [1].

    [1]http://www.openvswitch.org/support/dist-docs/ovs-vswitchd.conf.db.5.txt

    Depends-On: https://review.opendev.org/#/c/720785

    Change-Id: I8ed1fc85c2f78710bf6589ba3deca518471339b8
    Closes-Bug: #1868686

Changed in neutron:
status: In Progress → Fix Released
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.