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

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers