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

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.

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.

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/892895

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

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

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

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

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

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

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)

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)

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)

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)

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)

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

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

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

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)

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

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

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.]

This will be fixed in stable point releases for yoga and above in ubuntu. https://review.opendev.org/c/openstack/releases/+/901058

For earlier releases we will pick up the individual patches.

This issue was fixed in the openstack/neutron 22.1.0 release.

This issue was fixed in the openstack/neutron 20.5.0 release.

This issue was fixed in the openstack/neutron 21.2.0 release.

I'm going to unsubscribe ubuntu-sponsors as the patches are still UNRELEASED; also we can't sponsor to whatever weird openstack pockets there are outside the main archive, my suggestion would be to seek internal sponsoring instead.

Hello Julian,

I'm unsure what you mean by "as the patches *are* *still* UNRELEASED;", I've re-checked all the patches and only one of them (jammy) mistakenly had the "UNRELEASED;" as the series name -- which I've corrected and re-submitted. Please let me know if I'm doing something incorrectly, or if anything else needs correction for that matter.

Regarding the OpenStack pockets, Corey is already handling that side (btw, thanks Corey!).

A new package version with this fix has been uploaded to the focal unapproved queue and victoria/wallaby/xena staging PPAs.

Hello Mustafa, or anyone else affected,

Accepted neutron into focal-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/neutron/2:16.4.2-0ubuntu6.4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, what testing has been performed on the package and change the tag from verification-needed-focal to verification-done-focal. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-focal. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

This issue was fixed in the openstack/neutron ussuri-eol release.

I have a juju-deployed cloud running focal/ussuri that I was using for other verifications, which I upgraded to neutron 2:16.4.2-0ubuntu6.4 .

On the neutron-api/0 unit I can see this extension is now present in the file:

tail -3 common/ovn/extensions.py
    'quota_details',
    uplink_status_propagation.ALIAS,
]

Then I was enable to add the extension to ml2_conf.ini:

extension_drivers=port_security,dns_domain_ports,uplink_status_propagation

And after re-starting neutron-api I could create a port with --enable-uplink-status-propagation

I just do not have a card (like mlx5) that supports sriov to test part.

Thanks, Brian! In my Focal/Ussuri deployment, I also got the same results here.

The verification of the Stable Release Update for neutron has completed successfully and the package is now being released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package neutron - 2:16.4.2-0ubuntu6.4

---------------
neutron (2:16.4.2-0ubuntu6.4) focal; urgency=medium

  [ Corey Bryant ]
  * d/p/ovn-db-sync-continue-on-duplicate-normalise.patch: Cherry-picked
    from upstream to allow ovn_db_sync to continue on duplicate normalised
    CIDR (LP: #1961112).
  * d/p/ovn-db-sync-check-for-router-port-differences.patch:
    Cherry-picked from upstream to ensure router ports are marked
    for needing updates only if they have changed (LP: #2030773).
  * d/p/ovn-specify-port-type-if-router-port-when-updating.patch:
    Specify port type if it's a router port when updating to avoid
    port flapping (LP: #1955578).
  * d/p/fix-acl-sync-when-default-sg-group-created.patch:
    Cherry-picked form upstream to fix ACL sync when default security
    group is created (LP: #2008943).

  [ Mustafa Kemal GILOR ]
  * d/p/add_uplink_status_propagation.patch: Add the
    'uplink-status-propagation' extension to ML2/OVN (LP: #2032770).

 -- Corey Bryant <email address hidden> Wed, 08 Nov 2023 11:41:21 -0500

Hello Mustafa, or anyone else affected,

Accepted neutron into ussuri-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:ussuri-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-ussuri-needed to verification-ussuri-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-ussuri-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

This issue was fixed in the openstack/neutron victoria-eom release.

This issue was fixed in the openstack/neutron wallaby-eom release.

This issue was fixed in the openstack/neutron xena-eom release.

I have tested neutron version 2:16.4.2-0ubuntu6.4~cloud0 from the cloud-archive:ussuri-proposed repository and can verify the code has this change, and the failure does not occur. I followed the steps from the bug description:

openstack port create --network <network-id> --binding-profile trusted=true --enable-uplink-status-propagation --vnic-type direct aaa

- It did not fail with "BadRequestException: 400: Client Error for url: https://mycloud.example.com:9696/v2.0/ports, Unrecognized attribute(s) 'propagate_uplink_status'" as previously

The verification of the Stable Release Update for neutron has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package neutron - 2:16.4.2-0ubuntu6.4~cloud0
---------------

 neutron (2:16.4.2-0ubuntu6.4~cloud0) bionic-ussuri; urgency=medium
 .
   * New update for the Ubuntu Cloud Archive.
 .
 neutron (2:16.4.2-0ubuntu6.4) focal; urgency=medium
 .
   [ Corey Bryant ]
   * d/p/ovn-db-sync-continue-on-duplicate-normalise.patch: Cherry-picked
     from upstream to allow ovn_db_sync to continue on duplicate normalised
     CIDR (LP: #1961112).
   * d/p/ovn-db-sync-check-for-router-port-differences.patch:
     Cherry-picked from upstream to ensure router ports are marked
     for needing updates only if they have changed (LP: #2030773).
   * d/p/ovn-specify-port-type-if-router-port-when-updating.patch:
     Specify port type if it's a router port when updating to avoid
     port flapping (LP: #1955578).
   * d/p/fix-acl-sync-when-default-sg-group-created.patch:
     Cherry-picked form upstream to fix ACL sync when default security
     group is created (LP: #2008943).
 .
   [ Mustafa Kemal GILOR ]
   * d/p/add_uplink_status_propagation.patch: Add the
     'uplink-status-propagation' extension to ML2/OVN (LP: #2032770).

Hello Mustafa, or anyone else affected,

Accepted neutron into wallaby-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:wallaby-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-wallaby-needed to verification-wallaby-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-wallaby-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

The verification of the Stable Release Update for neutron has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

This bug was fixed in the package neutron - 2:18.6.0-0ubuntu1~cloud3
---------------

 neutron (2:18.6.0-0ubuntu1~cloud3) focal-wallaby; urgency=medium
 .
   [ Corey Bryant ]
   * d/p/ovn-db-sync-check-for-router-port-differences.patch:
     Cherry-picked from upstream to ensure router ports are marked
     for needing updates only if they have changed (LP: #2030773).
   * d/p/fix-acl-sync-when-default-sg-group-created.patch:
     Cherry-picked form upstream to fix ACL sync when default security
     group is created (LP: #2008943).
 .
   [ Mustafa Kemal GILOR ]
   * d/p/add_uplink_status_propagation.patch: Add the
     'uplink-status-propagation' extension to ML2/OVN (LP: #2032770).