Ubuntu
neutron package

[OVN] port creation with --enable-uplink-status-propagation does not work with OVN mechanism driver

  1. Lunar (23.04)
  2. Bug #2032770
Bug #2032770 reported by Mustafa Kemal Gilor
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Cloud Archive
New
Undecided
Unassigned
Antelope
New
Undecided
Unassigned
Ussuri
In Progress
Undecided
Unassigned
Wallaby
New
Undecided
Unassigned
Xena
New
Undecided
Unassigned
Yoga
New
Undecided
Unassigned
Zed
New
Undecided
Unassigned
neutron
Fix Released
Low
Rodolfo Alonso
neutron (Ubuntu)
New
Undecided
Mustafa Kemal Gilor
Focal
New
Undecided
Unassigned
Jammy
New
Undecided
Unassigned
Lunar
New
Undecided
Unassigned

Bug Description

The port "uplink_status_propagation" feature does not work when OVN is used as the mechanism driver. The reproducer below is working fine with openvswitch as the mechanism driver, but not with the OVN:

openstack port create --binding-profile trusted=true --enable-uplink-status-propagation --net private --vnic-type direct test-sriov-bond-enable-uplink-status-propagation-vm-1-port-1

The command fails with the following error when OVN is the mech driver:

BadRequestException: 400: Client Error for url: https://10.5.3.81:9696/v2.0/ports, Unrecognized attribute(s) 'propagate_uplink_status'

With ML2/OVS, the port creation command above succeeds without any errors.

As for the ml2_conf, "uplink_status_propagation" is listed in the extension drivers:

[ml2]
extension_drivers=port_security,dns_domain_ports,uplink_status_propagation
type_drivers = geneve,gre,vlan,flat,local
tenant_network_types = geneve,gre,vlan,flat,local
mechanism_drivers = ovn,sriovnicswitch
/*...*/

I also found the following document which shows the feature gap between ML2/OVS and OVN, but the uplink_status_propagation is not listed: https://docs.openstack.org/neutron/latest/ovn/gaps.html#id9 , maybe this page can be updated as well.

Revision history for this message
Dongwon Cho (dongwoncho) wrote :

I think it would be much helpful to avoid any confusion if we mention in the doc or man page that the feature is only available on which mechanism driver.

Mamatisa Nurmatov (isabek)
Changed in neutron:
status: New → Confirmed
importance: Undecided → Medium
importance: Medium → Low
tags: added: doc
Revision history for this message
Rodolfo Alonso (rodolfo-alonso-hernandez) wrote :

Hello:

The extensions in the ML2/OVN mechanism driver are, for now, hardcoded in the ``neutron.common.ovn.extensions`` file. I'll push an update.

Regards.

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/+/892895

Changed in neutron:
status: Confirmed → In Progress
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/+/892896

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

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

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

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

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

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

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

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

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

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

commit 0545f40a0401629e68561a54e4a576a9fb15f7fc
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892898
Committed: https://opendev.org/openstack/neutron/commit/62ed750a1596c875bb38e0094a553f11a33d29ec
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 62ed750a1596c875bb38e0094a553f11a33d29ec
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Conflicts:
        neutron/common/ovn/extensions.py

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)

tags: added: in-stable-yoga
tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892899
Committed: https://opendev.org/openstack/neutron/commit/5e8a6b1add51f06e9a360f186c5921c460070624
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 5e8a6b1add51f06e9a360f186c5921c460070624
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Conflicts:
        neutron/common/ovn/extensions.py

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)
    (cherry picked from commit 62ed750a1596c875bb38e0094a553f11a33d29ec)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892900
Committed: https://opendev.org/openstack/neutron/commit/f6e486ce29350c4b7321650435d9abe9fb0204e7
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit f6e486ce29350c4b7321650435d9abe9fb0204e7
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Conflicts:
        neutron/common/ovn/extensions.py

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)
    (cherry picked from commit 62ed750a1596c875bb38e0094a553f11a33d29ec)

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892896
Committed: https://opendev.org/openstack/neutron/commit/4f707101d965e41f4211ccf19a3bbdfcd04c9fe7
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 4f707101d965e41f4211ccf19a3bbdfcd04c9fe7
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/892897
Committed: https://opendev.org/openstack/neutron/commit/e369dbf32ee70e18b0dc7c5e8a4257c0652da407
Submitter: "Zuul (22348)"
Branch: stable/zed

commit e369dbf32ee70e18b0dc7c5e8a4257c0652da407
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 23.0.0.0rc1

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

Mustafa Kemal Gilor (mustafakemalgilor)
Changed in neutron (Ubuntu):
assignee: nobody → Mustafa Kemal Gilor (mustafakemalgilor)
Mustafa Kemal Gilor (mustafakemalgilor)
no longer affects: neutron (Ubuntu Mantic)
Mustafa Kemal Gilor (mustafakemalgilor)
no longer affects: neutron (Ubuntu Focal)
Mustafa Kemal Gilor (mustafakemalgilor)
no longer affects: cloud-archive/victoria
Mustafa Kemal Gilor (mustafakemalgilor)
no longer affects: cloud-archive/bobcat
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/ussuri)

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

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

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

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/898287
Committed: https://opendev.org/openstack/neutron/commit/2cdb322501e97657029987528b0946306626eddc
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 2cdb322501e97657029987528b0946306626eddc
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Mon Aug 28 09:55:05 2023 +0000

    [OVN] Add the 'uplink-status-propagation' extension to ML2/OVN

    Conflicts:
        neutron/common/ovn/extensions.py

    Closes-Bug: #2032770
    Change-Id: I5f11b568bab4518617409b7c3a02c7222df80a92
    (cherry picked from commit 0545f40a0401629e68561a54e4a576a9fb15f7fc)
    (cherry picked from commit 62ed750a1596c875bb38e0094a553f11a33d29ec)
    (cherry picked from commit f6e486ce29350c4b7321650435d9abe9fb0204e7)

tags: added: in-stable-victoria
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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