Ubuntu Cloud Archive

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

Series zed
Bug #2032770

Bug #2032770 reported by Mustafa Kemal Gilor on 2023-08-23

This bug affects 1 person

	Status	Importance	Assigned to
Ubuntu Cloud Archive	New	Undecided	Unassigned
Antelope	New	Undecided	Unassigned
Ussuri	Fix Committed	Undecided	Unassigned
Victoria	New	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

[Impact]

This SRU is a backport of https://review.opendev.org/c/openstack/neutron/+/892895 to the respective Ubuntu and UCA releases. The patch is merged to all respective upstream branches (master & stable/[u,v,w,x,y,z,2023.1(a)]).

This SRU intends to add the missing 'uplink-status-propagation' extension to ML2/OVN. This extension is already present and working in ML2/OVS, and it is supported by ML2/OVN but the extension is somehow not added to ML2/OVN. The patch simply adds the missing extension to the ML2/OVN too.

The impact of this is visible for the deployments migrating from ML2/OVS to ML2/OVN. The following command fails to work on ML2/OVN:

```
openstack port create --network 8d30fb08-2c6a-42fd-98c4-223d345c8c4f --binding-profile trusted=true --enable-uplink-status-propagation --vnic-type direct aaa

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

```

The fix corrects this behavior by adding the missing extension.

[Test Case]

- Deploy a Focal/Yoga cloud:
    - ./generate-bundle.sh -s focal -r yoga --name test-focal-yoga-stack --run --ovn
    # After the dust settles
    - ./configure
    - source ./novarc
- openstack port create --network <network-id> --binding-profile trusted=true --enable-uplink-status-propagation --vnic-type direct aaa
- It should fail with "BadRequestException: 400: Client Error for url: https://mycloud.example.com:9696/v2.0/ports, Unrecognized attribute(s) 'propagate_uplink_status'"

To confirm the fix, repeat the scenario and observe that the error disappears and port creation succeeds.

[Regression Potential]
The patch is quite trivial and should not affect any deployment negatively. The extension is optional and disabled by default.

See original description

Tags:

Revision history for this message

Dongwon Cho (dongwoncho) wrote on 2023-08-25:

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) on 2023-08-28

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 on 2023-08-28:

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 on 2023-08-28: 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 on 2023-08-28: 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 on 2023-08-28: 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 on 2023-08-28: 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 on 2023-08-28: 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 on 2023-08-28: 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 on 2023-08-31: 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 on 2023-08-31: Fix merged to neutron (stable/yoga)

#10

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 on 2023-08-31: Fix merged to neutron (stable/xena)

#11

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 on 2023-08-31: Fix merged to neutron (stable/wallaby)

#12

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

tags:

added: in-stable-wallaby

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2023-08-31: Fix merged to neutron (stable/2023.1)

#13

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 on 2023-08-31: Fix merged to neutron (stable/zed)

#14

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 on 2023-09-15: Fix included in openstack/neutron 23.0.0.0rc1

#15

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

Mustafa Kemal Gilor (mustafakemalgilor) on 2023-10-09

Changed in neutron (Ubuntu):
assignee:	nobody → Mustafa Kemal Gilor (mustafakemalgilor)

Mustafa Kemal Gilor (mustafakemalgilor) on 2023-10-09

no longer affects:

neutron (Ubuntu Mantic)

Mustafa Kemal Gilor (mustafakemalgilor) on 2023-10-09

no longer affects:

neutron (Ubuntu Focal)

Mustafa Kemal Gilor (mustafakemalgilor) on 2023-10-09

no longer affects:

cloud-archive/victoria

Mustafa Kemal Gilor (mustafakemalgilor) on 2023-10-16

no longer affects:

cloud-archive/bobcat

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2023-10-16: Fix proposed to neutron (stable/ussuri)

#16

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 on 2023-10-16: Fix proposed to neutron (stable/victoria)

#17

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 on 2023-10-19: Fix merged to neutron (stable/victoria)

#18

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

tags:

added: in-stable-victoria

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2023-10-31: Fix merged to neutron (stable/ussuri)

#19

Reviewed: https://review.opendev.org/c/openstack/neutron/+/898285
Committed: https://opendev.org/openstack/neutron/commit/dbc2cf46fb1852ca032a2d86417fd5b0eea646ac
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit dbc2cf46fb1852ca032a2d86417fd5b0eea646ac
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

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#20

focal.debdiff Edit (3.3 KiB, text/plain)

description:	updated
summary:	- [OVN] port creation with --enable-uplink-status-propagation does not - work with OVN mechanism driver + [SRU] [OVN] port creation with --enable-uplink-status-propagation does + not work with OVN mechanism driver

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#21

jammy.debdiff Edit (3.5 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#22

lunar.debdiff Edit (2.8 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#23

uca_bionic_ussuri.debdiff Edit (3.3 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#24

uca_focal_victoria.debdiff Edit (3.4 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#25

uca_focal_wallaby.debdiff Edit (2.9 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#26

uca_jammy_xena.debdiff Edit (2.9 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#27

uca_jammy_yoga.debdiff Edit (2.8 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#28

uca_jammy_zed.debdiff Edit (2.9 KiB, text/plain)

Revision history for this message

Mustafa Kemal Gilor (mustafakemalgilor) wrote on 2023-11-02:

#29

uca_jammy_antelope.debdiff Edit (2.9 KiB, text/plain)

Revision history for this message

Ubuntu Foundations Team Bug Bot (crichton) wrote on 2023-11-02:

#30

The attachment "focal.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags:

added: patch

Report a bug

This report contains Public information

Everyone can see this information.

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

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

Ubuntu Cloud Archive

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

Bug Description

Other bug subscribers

Patches

Remote bug watches