[ovn-octavia-provider] Deleted members remain in ERROR status

Bug #1964817 reported by Fernando Royo
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Fernando Royo

Bug Description

An OVN provider Load Balancer allows the creation of members without specifying the subnet-id. The internal logic allows OVN provider to create the member by associating it with the subnet-id of the pool to which it belongs, but the member is stored in Octavia DB without a reference to the subnet_id associated with the pool.

Subsequently, when an attempt is made to delete a member that was created without specifying the subnet_id to which it belongs, it is left in an ERROR state, while a member that did include that parameter at creation time is deleted correctly.

Steps to Reproduce:
1. Create a load balancer with OVN as provider
2. Create a listener and a pool associated to LB create in 1.
3. Create a member without parameter subnet-id
4. Try to delete the member created in 3.

Expected result:
Member is deleted correctly

Actual result:
Member keeps in ERROR state.

Changed in neutron:
assignee: nobody → Fernando Royo (froyoredhat)
Akihiro Motoki (amotoki)
tags: added: ovn-octavia-provider
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ovn-octavia-provider (master)
Changed in neutron:
status: New → In Progress
Miguel Lavalle (minsel)
Changed in neutron:
importance: Undecided → Medium
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/+/833799
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/82eabfda244de2de54c8417f201c67129b875199
Submitter: "Zuul (22348)"
Branch: master

commit 82eabfda244de2de54c8417f201c67129b875199
Author: Fernando Royo <email address hidden>
Date: Tue Mar 15 15:36:42 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Closes-Bug: #1964817
    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f

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/yoga)
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/+/835196

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ovn-octavia-provider (stable/ussuri)

Change abandoned by "Fernando Royo <email address hidden>" on branch: stable/ussuri
Review: https://review.opendev.org/c/openstack/ovn-octavia-provider/+/835199
Reason: duplicate Change Id by error

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

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

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

commit caf9f2aed3e9e127f49f1b83ec5d5252af5c1d0d
Author: Fernando Royo <email address hidden>
Date: Tue Mar 15 15:36:42 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Closes-Bug: #1964817
    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f
    (cherry picked from commit 82eabfda244de2de54c8417f201c67129b875199)

tags: added: in-stable-yoga
tags: added: in-stable-xena
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/+/835195
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/3975bb76aeece21021ef951f8069b72d18d7a58e
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 3975bb76aeece21021ef951f8069b72d18d7a58e
Author: Fernando Royo <email address hidden>
Date: Tue Mar 15 15:36:42 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Closes-Bug: #1964817
    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f
    (cherry picked from commit 82eabfda244de2de54c8417f201c67129b875199)

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

commit 5c7118213fad5dcf3b2fe7727e6fdb0e5d6f7f1f
Author: Fernando Royo <email address hidden>
Date: Tue Mar 15 15:36:42 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Closes-Bug: #1964817
    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f
    (cherry picked from commit 82eabfda244de2de54c8417f201c67129b875199)

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

commit f32ece64faeaba5f7842cf90f93963d26357f6dd
Author: Fernando Royo <email address hidden>
Date: Tue Mar 15 15:36:42 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Conflicts:
        ovn_octavia_provider/driver.py

    Closes-Bug: #1964817
    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f
    (cherry picked from commit 82eabfda244de2de54c8417f201c67129b875199)

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

commit f208e7e9de843904982ce2c7b36278760f53859d
Author: Fernando Royo <email address hidden>
Date: Fri Mar 25 11:57:57 2022 +0100

    Fix deletion of members without subnet_id

    OVN octavia provider allows to create member without param subnet_id,
    in that case the driver search it according to the pool_id, but it is
    not propagated to Octavia DB.

    In further operations, as delete or update, Octavia call to the driver
    without subnet_id, getting as result the member in ERROR state.

    Closes-Bug: #1964817
    (manually cherry picked from commit 82eabfda244de2de54c8417f201c67129b875199)

    Change-Id: I08b5ee11dbffe672dfd762c944c2f2c18623826f

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/ovn-octavia-provider 3.0.0.0rc1

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

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 2.1.0

This issue was fixed in the openstack/ovn-octavia-provider 2.1.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.