Allow modification of max retries in OVSNeutronAgent

Bug #1952898 reported by Szymon Wróblewski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Undecided
Szymon Wróblewski

Bug Description

Modifying max device retries in classes inheriting from OVSNeutronAgent is quite hard because it uses module constant.
Since our internal agent providing vrack functionality depends heavily on OVSNeutronAgent it would be nice to be able to change that value without having to overwrite all functions using MAX_DEVICE_RETRIES constant.

Tags: ovs
Changed in neutron:
assignee: nobody → Szymon Wróblewski (bluex)
Changed in neutron:
status: New → In Progress
Revision history for this message
Lajos Katona (lajos-katona) wrote :

Hi, I suppose you plan to use config option?

tags: added: ovs
Revision history for this message
Lajos Katona (lajos-katona) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/819816
Committed: https://opendev.org/openstack/neutron/commit/0152b43ee112acebbe2f6773fce9561fc25116f6
Submitter: "Zuul (22348)"
Branch: master

commit 0152b43ee112acebbe2f6773fce9561fc25116f6
Author: Szymon Wroblewski <email address hidden>
Date: Tue Nov 30 13:42:39 2021 +0100

    Allow modification of max retries in OVSNeutronAgent

    Use class variable instead of module constant to
    allow modification of max_device_retries in classes inheriting
    from OVSNeutronAgent.

    Closes-Bug: #1952898
    Change-Id: I952491ca89466540d39d78f963b581302c0d8a17

Changed in neutron:
status: In Progress → Fix Released
tags: added: neutron-proactive-backport-potential
tags: removed: neutron-proactive-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 20.0.0.0rc1

This issue was fixed in the openstack/neutron 20.0.0.0rc1 release candidate.

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.