neutron

[FT] "TestMaintenance.test_port_forwarding" randomly failing in call assertion

Bug #2033387 reported by Rodolfo Alonso
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Low
Rodolfo Alonso

Bug Description

Test "neutron.tests.functional.plugins.ml2.drivers.ovn.mech_driver.ovsdb.test_maintenance.TestMaintenance.test_port_forwarding" is randomly failing during the assertion of the "neutron_lib.callbacks.registry.publish" calls after calling "delete_floatingip_port_forwarding". The method is reporting two calls:
* call('port', 'provisioning_complete', 'L2', payload=<neutron_lib.callbacks.events.DBEventPayload object at 0x7fe9b9912f80>),
* call('port_forwarding', 'after_delete', <neutron.services.portforwarding.pf_plugin.PortForwardingPlugin object at 0x7fe9b3b15ab0>, payload=<neutron_lib.callbacks.events.DBEventPayload object at 0x7fe9b39a7400>)

The first one could be a delayed call from the same test. The proposal is to specifically test the call we are expecting, that is ('port_forwarding', 'after_delete').

Logs: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_8a2/periodic/opendev.org/openstack/neutron/master/neutron-functional-with-pyroute2-master/8a279fa/testr_results.html

Snippet: https://paste.opendev.org/show/bNlr2vMbWn8i7JCuiF8a/

Rodolfo Alonso (rodolfo-alonso-hernandez)
Changed in neutron:
importance: Undecided → Low
assignee: nobody → Rodolfo Alonso (rodolfo-alonso-hernandez)
status: New → Triaged
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (master)

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

Changed in neutron:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893028
Committed: https://opendev.org/openstack/neutron/commit/d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4
Submitter: "Zuul (22348)"
Branch: master

commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/zed)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/yoga)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/xena)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893168
Committed: https://opendev.org/openstack/neutron/commit/234a098d7b10261037c97d1b1e04f45dc122f8b2
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit 234a098d7b10261037c97d1b1e04f45dc122f8b2
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0
    (cherry picked from commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4)

tags: added: in-stable-zed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893169
Committed: https://opendev.org/openstack/neutron/commit/d08194bbb756b41cd3d7c78f5ac363c2ca1ede0e
Submitter: "Zuul (22348)"
Branch: stable/zed

commit d08194bbb756b41cd3d7c78f5ac363c2ca1ede0e
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0
    (cherry picked from commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4)

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893170
Committed: https://opendev.org/openstack/neutron/commit/22c3951a93f2834010cf9bd8a30a41b4d570b8b0
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 22c3951a93f2834010cf9bd8a30a41b4d570b8b0
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0
    (cherry picked from commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4)

tags: added: in-stable-yoga
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893171
Committed: https://opendev.org/openstack/neutron/commit/4acbf7a15cf13257b5f5916024aca45d70963ae7
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 4acbf7a15cf13257b5f5916024aca45d70963ae7
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0
    (cherry picked from commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4)

tags: added: in-stable-xena
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 23.0.0.0b3

This issue was fixed in the openstack/neutron 23.0.0.0b3 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to neutron (stable/wallaby)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/893374
Committed: https://opendev.org/openstack/neutron/commit/03f3b8ded732932182b5b03e9602baad8e2e57b7
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 03f3b8ded732932182b5b03e9602baad8e2e57b7
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Tue Aug 29 10:04:52 2023 +0000

    [FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "delete_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    Closes-Bug: #2033387
    Change-Id: I95074a0fe071e3ed8bf58c9210d63a2f30ee68a0
    (cherry picked from commit d9eb04478e0d9e8a7de3e1eee1138be05e64c9a4)
    (cherry picked from commit 4acbf7a15cf13257b5f5916024aca45d70963ae7)
    Conflicts:
            neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

tags: added: in-stable-wallaby
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 22.1.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 20.5.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron 21.2.0

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/neutron/+/903579

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903579
Committed: https://opendev.org/openstack/neutron/commit/2f637b2cb92dc37ef080eaf433fe580279136f32
Submitter: "Zuul (22348)"
Branch: master

commit 2f637b2cb92dc37ef080eaf433fe580279136f32
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/2023.2)

Related fix proposed to branch: stable/2023.2
Review: https://review.opendev.org/c/openstack/neutron/+/903829

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/2023.1)

Related fix proposed to branch: stable/2023.1
Review: https://review.opendev.org/c/openstack/neutron/+/903830

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/zed)

Related fix proposed to branch: stable/zed
Review: https://review.opendev.org/c/openstack/neutron/+/903831

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/yoga)

Related fix proposed to branch: stable/yoga
Review: https://review.opendev.org/c/openstack/neutron/+/903832

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/xena)

Related fix proposed to branch: stable/xena
Review: https://review.opendev.org/c/openstack/neutron/+/903833

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (stable/wallaby)

Related fix proposed to branch: stable/wallaby
Review: https://review.opendev.org/c/openstack/neutron/+/903926

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/2023.2)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903829
Committed: https://opendev.org/openstack/neutron/commit/76a58448e50a7159cf5722145904ae8b2ed3b722
Submitter: "Zuul (22348)"
Branch: stable/2023.2

commit 76a58448e50a7159cf5722145904ae8b2ed3b722
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/2023.1)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903830
Committed: https://opendev.org/openstack/neutron/commit/c4b322597bcd9a1ba8aab77fd0f2ce000d39c57e
Submitter: "Zuul (22348)"
Branch: stable/2023.1

commit c4b322597bcd9a1ba8aab77fd0f2ce000d39c57e
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/zed)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903831
Committed: https://opendev.org/openstack/neutron/commit/602bb1a325cf61eca910648a6425a02fad908bb1
Submitter: "Zuul (22348)"
Branch: stable/zed

commit 602bb1a325cf61eca910648a6425a02fad908bb1
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/yoga)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903832
Committed: https://opendev.org/openstack/neutron/commit/6b86538e7dc59af9dace030d1dabeb3f312fadde
Submitter: "Zuul (22348)"
Branch: stable/yoga

commit 6b86538e7dc59af9dace030d1dabeb3f312fadde
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/xena)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903833
Committed: https://opendev.org/openstack/neutron/commit/01786719557131d81b540e843cfbba2fac415d8b
Submitter: "Zuul (22348)"
Branch: stable/xena

commit 01786719557131d81b540e843cfbba2fac415d8b
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (stable/wallaby)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/903926
Committed: https://opendev.org/openstack/neutron/commit/9030e266de40334740ee28fd1310e28bae2d3971
Submitter: "Zuul (22348)"
Branch: stable/wallaby

commit 9030e266de40334740ee28fd1310e28bae2d3971
Author: Rodolfo Alonso Hernandez <email address hidden>
Date: Sun Aug 20 01:59:35 2023 +0000

    [OVN][FT] Make explicit the "publish" call check in "test_port_forwarding"

    The functional test "TestMaintenance.test_port_forwarding" is checking
    the "registry.publish" call after calling
    "create_floatingip_port_forwarding" and
    "update_floatingip_port_forwarding". This patch makes this check
    explicit for the expected call.

    This patch is similar to [1].

    [1]https://review.opendev.org/c/openstack/neutron/+/893028

    Related-Bug: #2033387
    Change-Id: I595dbd9b3abf413408fddf60735c0e8bd54e1d5c
    (cherry picked from commit 2f637b2cb92dc37ef080eaf433fe580279136f32)
    (cherry picked from commit 01786719557131d81b540e843cfbba2fac415d8b)
    Conflicts:
            neutron/tests/functional/plugins/ml2/drivers/ovn/mech_driver/ovsdb/test_maintenance.py

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron wallaby-eom

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron xena-eom

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

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

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.