[OVN Octavia Provider] Load Balancer not reachable from some Subnets

Bug #1931639 reported by Flavio Fernandes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Flavio Fernandes

Bug Description

In situations where router port and load balancer are created back to back, there is a potential race condition that would render OVN with a logical switch that is missing a reference to the load balancer.

This issue is also being tracked in Bugzilla, under the link:

     https://bugzilla.redhat.com/show_bug.cgi?id=1937392

Changed in neutron:
assignee: nobody → Flavio Fernandes (ffernand)
importance: Undecided → High
status: New → In Progress
description: updated
tags: added: ovn-octavia-provider
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (master)
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/+/796839

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

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

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/+/796095
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/b2a862fdb3d4eae440a342f6b6bbc0323022f8ba
Submitter: "Zuul (22348)"
Branch: master

commit b2a862fdb3d4eae440a342f6b6bbc0323022f8ba
Author: Flavio Fernandes <email address hidden>
Date: Sat Jun 12 08:09:33 2021 -0400

    Ensure that load balancer is added to logical switch

    Add logic to handle cases where a race between logical router port and
    load balancer creation renders the affected logical switch without the
    proper load balancer in its attribute.

    This change adds _get_subnet() to the helper module, so obtaining the
    subnet is done from a shared location.

    Closes-bug: #1931639
    Change-Id: Ie4a843fc5dd98fc25f8cecf91997e275c36f121d

Changed in neutron:
status: In Progress → Fix Released
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/+/796877
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/62891fa3c29f1e2748250f65108f4383c6cce7fb
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 62891fa3c29f1e2748250f65108f4383c6cce7fb
Author: Flavio Fernandes <email address hidden>
Date: Sat Jun 12 08:09:33 2021 -0400

    Ensure that load balancer is added to logical switch

    Add logic to handle cases where a race between logical router port and
    load balancer creation renders the affected logical switch without the
    proper load balancer in its attribute.

    This change adds _get_subnet() to the helper module, so obtaining the
    subnet is done from a shared location.

    Closes-bug: #1931639
    Change-Id: Ie4a843fc5dd98fc25f8cecf91997e275c36f121d
    (cherry picked from commit b2a862fdb3d4eae440a342f6b6bbc0323022f8ba)

tags: added: in-stable-victoria
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/+/796876
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/4d20d3243fc7376bb31c94834d43550f1200ffcd
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit 4d20d3243fc7376bb31c94834d43550f1200ffcd
Author: Flavio Fernandes <email address hidden>
Date: Sat Jun 12 08:09:33 2021 -0400

    Ensure that load balancer is added to logical switch

    Add logic to handle cases where a race between logical router port and
    load balancer creation renders the affected logical switch without the
    proper load balancer in its attribute.

    Conflicts:
      ovn_octavia_provider/helper.py
      ovn_octavia_provider/tests/unit/test_helper.py

    Closes-bug: #1931639
    Change-Id: Ie4a843fc5dd98fc25f8cecf91997e275c36f121d
    (cherry picked from commit b2a862fdb3d4eae440a342f6b6bbc0323022f8ba)

tags: added: in-stable-ussuri
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/+/796839
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/9c362b3ded873121e9626e5322245a33fd429840
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 9c362b3ded873121e9626e5322245a33fd429840
Author: Flavio Fernandes <email address hidden>
Date: Sat Jun 12 08:09:33 2021 -0400

    Ensure that load balancer is added to logical switch

    Add logic to handle cases where a race between logical router port and
    load balancer creation renders the affected logical switch without the
    proper load balancer in its attribute.

    This change adds _get_subnet() to the helper module, so obtaining the
    subnet is done from a shared location.

    Closes-bug: #1931639
    Change-Id: Ie4a843fc5dd98fc25f8cecf91997e275c36f121d
    (cherry picked from commit b2a862fdb3d4eae440a342f6b6bbc0323022f8ba)

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

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

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

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

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

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

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.

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.