[OVN] ML2/OVN mech driver does not set the OVS bridge name in the port VIF details

Bug #2045889 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 ML2/OVN mech driver does not set the OVS bridge name nor the datapath type in the Neutron DB port VIF details. Example of an ML2/OVS port:
"""
binding_vif_details: bound_drivers.0='openvswitch', bridge_name='br-int', connectivity='l2', datapath_type='system', ovs_hybrid_plug='False', port_filter='True'
"""

Missing parameters:
* bridge_name
* datapath_type

This information is needed by Nova (os-vif library)

Tags: ovn
Changed in neutron:
importance: Undecided → Medium
tags: added: ovn
Changed in neutron:
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/+/903494

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

Fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/neutron/+/904252

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

Fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/904254

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

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

commit baaf240ce3f7802fe1431cc13913b9d93fc7f742
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sat Aug 19 05:11:55 2023 +0000

    [OVN] Add the bridge name and datapath type to the port VIF details

    Same as in ML2/OVS, the ML2/OVN mechanism driver adds to the port
    VIF details dictionary the OVS bridge the port is connected to
    and the integration bridge datapath type.

    Closes-Bug: #2045889
    Change-Id: Ifda46c42b9506449a58fbaf312cc71c72d9cf2df

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (stable/2023.1)

Change abandoned by "Rodolfo Alonso <email address hidden>" on branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/904254
Reason: We can't backport this patch up to 2023.1. Older OVN versions don't have "Chassis_Private" table, as seen in https://ff513c95ce631b00d7e7-2ea4ab35fcc993920ff2033895dde0a5.ssl.cf2.rackcdn.com/904254/6/check/neutron-ovn-tempest-ovs-release-ubuntu-old/373d6f8/controller/logs/screen-q-ovn-metadata-agent.txt

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/904252
Committed: https://opendev.org/openstack/neutron/commit/f61b6f81afdd3fcbc2ff6c33a984bf1ccdd0dc24
Submitter: "Zuul (22348)"
Branch: stable/2023.2

commit f61b6f81afdd3fcbc2ff6c33a984bf1ccdd0dc24
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sat Aug 19 05:11:55 2023 +0000

    [OVN] Add the bridge name and datapath type to the port VIF details

    Same as in ML2/OVS, the ML2/OVN mechanism driver adds to the port
    VIF details dictionary the OVS bridge the port is connected to
    and the integration bridge datapath type.

    Closes-Bug: #2045889
    Change-Id: Ifda46c42b9506449a58fbaf312cc71c72d9cf2df
    (cherry picked from commit baaf240ce3f7802fe1431cc13913b9d93fc7f742)

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.