``DvrDriver`` and ``OvnDriver`` incorrectly define distributed flag

Bug #2056199 reported by Rodolfo Alonso
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Rodolfo Alonso

Bug Description

The class ``L3ServiceProvider`` defines the distributed support with the class variable "distributed_support" [1]. The classes ``DvrDriver`` and ``OvnDriver`` are using the variable "dvr_support" instead.

The method to ensure a driver "ha" and "distributed" support uses "distributed_support" too [2].

[1]https://github.com/openstack/neutron/blob/c4c14f9589b54cc518564f1e1679898d2729c9e2/neutron/services/l3_router/service_providers/base.py#L57
[2]https://github.com/openstack/neutron/blob/c4c14f9589b54cc518564f1e1679898d2729c9e2/neutron/services/l3_router/service_providers/driver_controller.py#L273

Changed in neutron:
importance: Undecided → Medium
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
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/c/openstack/neutron/+/911151

Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/911151
Committed: https://opendev.org/openstack/neutron/commit/06d48cb980d777514ffe9283eed0b59dcdb14345
Submitter: "Zuul (22348)"
Branch: master

commit 06d48cb980d777514ffe9283eed0b59dcdb14345
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Mar 5 02:34:30 2024 +0000

    ``OvnDriver`` and ``DvrHaDriver`` to use "distributed_support" variable

    ``OvnDriver`` and ``DvrHaDriver`` classes were using an incorrect
    variable name to define the DVR support, that should be
    "distributed_support" instead of "dvr_support".

    Closes-Bug: #2056199
    Change-Id: Id2ee080dde8cd094995e94564f2877a89e9cc5aa

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 24.0.0.0rc1

This issue was fixed in the openstack/neutron 24.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.