[OVN Octavia Provider] OVN provider fails creating member without subnet ID

Bug #1896677 reported by Brian Haley
22
This bug affects 5 people
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Brian Haley

Bug Description

The OVN Octavia Provider driver requires a subnet ID be present in the API call to create a member. Since this is an optional parameter according to the API, it should be able to work without one. The octavia-lib DriverLibrary class provides methods to look it up given other information we have in the API call, so we should support it.

summary: - [OVN Octavia Provider] OVN provider fails creating pool without subnet
+ [OVN Octavia Provider] OVN provider fails creating member without subnet
ID
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (master)
Changed in neutron:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovn-octavia-provider (master)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/795896
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/675ea9c35ef73765828e3db4b636f13465a76596
Submitter: "Zuul (22348)"
Branch: master

commit 675ea9c35ef73765828e3db4b636f13465a76596
Author: Brian Haley <email address hidden>
Date: Thu Jun 10 16:42:36 2021 -0400

    Support creating members without a subnet ID

    Since subnet ID is an optional API argument, if not
    available the provider driver will now attempt to look
    it up via the pool ID that is required.

    Closes-bug: #1896677
    Change-Id: Iec17b36ce38be89f83a45ea3ef4c652e837c69c5

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (stable/xena)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (stable/wallaby)

Fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827011

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

Fix proposed to branch: stable/victoria
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827040

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (stable/ussuri)

Fix proposed to branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827045

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovn-octavia-provider (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827010
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/0896d5f4ec77febc939c3e8b33d94f934eddf805
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 0896d5f4ec77febc939c3e8b33d94f934eddf805
Author: Brian Haley <email address hidden>
Date: Thu Jun 10 16:42:36 2021 -0400

    Support creating members without a subnet ID

    Since subnet ID is an optional API argument, if not
    available the provider driver will now attempt to look
    it up via the pool ID that is required.

    Closes-bug: #1896677
    Change-Id: Iec17b36ce38be89f83a45ea3ef4c652e837c69c5
    (cherry picked from commit 675ea9c35ef73765828e3db4b636f13465a76596)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovn-octavia-provider (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827011
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/08430b535eb56e3758e63f2a97c7e3a383769bee
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 08430b535eb56e3758e63f2a97c7e3a383769bee
Author: Brian Haley <email address hidden>
Date: Thu Jun 10 16:42:36 2021 -0400

    Support creating members without a subnet ID

    Since subnet ID is an optional API argument, if not
    available the provider driver will now attempt to look
    it up via the pool ID that is required.

    Closes-bug: #1896677
    Change-Id: Iec17b36ce38be89f83a45ea3ef4c652e837c69c5
    (cherry picked from commit 675ea9c35ef73765828e3db4b636f13465a76596)

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovn-octavia-provider (stable/ussuri)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827045
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/bf86846205718f652457aec1943e4c53334dd2a6
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit bf86846205718f652457aec1943e4c53334dd2a6
Author: Luis Tomas Bolivar <email address hidden>
Date: Mon Jan 31 11:41:15 2022 +0100

    Support creating members without a subnet ID

    Since subnet ID is an optional API argument, if not
    available the provider driver will now attempt to look
    it up via the pool ID that is required.

    NOTE: Manually cherry-picked as there was a code reshape,
    and in the backport all the code goes to driver.py

    Closes-bug: #1896677
    (manually cherry picked from commit 675ea9c35ef73765828e3db4b636f13465a76596)
    Change-Id: I69a052aadf299907bf8ba1634376504461e2d779

tags: added: in-stable-ussuri
tags: added: in-stable-victoria
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ovn-octavia-provider (stable/victoria)

Reviewed: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/827040
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/3860ca0159f50bd530c2198a55c50724db02cc0c
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 3860ca0159f50bd530c2198a55c50724db02cc0c
Author: Brian Haley <email address hidden>
Date: Thu Jun 10 16:42:36 2021 -0400

    Support creating members without a subnet ID

    Since subnet ID is an optional API argument, if not
    available the provider driver will now attempt to look
    it up via the pool ID that is required.

    Conflicts:
        ovn_octavia_provider/tests/functional/test_driver.py

    Closes-bug: #1896677
    Change-Id: Iec17b36ce38be89f83a45ea3ef4c652e837c69c5
    (cherry picked from commit 675ea9c35ef73765828e3db4b636f13465a76596)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ovn-octavia-provider 2.0.0.0rc1

This issue was fixed in the openstack/ovn-octavia-provider 2.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ovn-octavia-provider 1.0.1

This issue was fixed in the openstack/ovn-octavia-provider 1.0.1 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/networking-ovn train-eol

This issue was fixed in the openstack/networking-ovn train-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ovn-octavia-provider 1.2.0

This issue was fixed in the openstack/ovn-octavia-provider 1.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ovn-octavia-provider ussuri-eol

This issue was fixed in the openstack/ovn-octavia-provider ussuri-eol release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/ovn-octavia-provider victoria-eom

This issue was fixed in the openstack/ovn-octavia-provider victoria-eom release.

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.