[stable only] Logging service plugin expects wrong arguments in the callback function

Bug #1955546 reported by Slawek Kaplonski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
High
Slawek Kaplonski

Bug Description

To fix bug https://bugs.launchpad.net/neutron/+bug/1939558 we proposed 2 patches https://review.opendev.org/c/openstack/neutron/+/810870 and https://review.opendev.org/c/openstack/neutron/+/815298/ (those are wallaby backports).
But by mistake we used new style of the payload arguments in the callback functions in neutron/services/logapi/logging_plugin.py in _clean_logs_by_resource_id method thus now it's failing when that callback is called.
It affects only wallaby and older releases as Xena and master are already moved to the new callbacks so it works fine there.

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

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

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

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

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

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

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

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

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

commit adbcc77a3f7ebbf9a2302248dcdfbeece1a0528f
Author: Slawek Kaplonski <email address hidden>
Date: Wed Dec 22 08:37:45 2021 +0100

    [Stable only] Fix callback function arguments

    Fixes [1] and [2] for bug [3] are using new callabacks mechanism and
    expects payload as one of the arguments passed to the callback functions.
    This works fine in Xena+ releases but not in Wallaby and olders where
    old callbacks mechanism is still used, at least for the security group
    AFTER_DELETE events.
    So this patch fixes that by changing to the old callbacks mechanism in
    the logging service plugin in stable/wallaby and older branches only.

    Closes-Bug: #1955546

    [1] https://review.opendev.org/c/openstack/neutron/+/804237
    [2] https://review.opendev.org/c/openstack/neutron/+/812459
    [3] https://bugs.launchpad.net/neutron/+bug/1939558

    Change-Id: I0003bd566dc769436ad1342351ad058394bd52da

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/822610
Committed: https://opendev.org/openstack/neutron/commit/90156cef695b26d6f8bcf5f7f0329effafc4e322
Submitter: "Zuul (22348)"
Branch: stable/victoria

commit 90156cef695b26d6f8bcf5f7f0329effafc4e322
Author: Slawek Kaplonski <email address hidden>
Date: Wed Dec 22 08:37:45 2021 +0100

    [Stable only] Fix callback function arguments

    Fixes [1] and [2] for bug [3] are using new callabacks mechanism and
    expects payload as one of the arguments passed to the callback functions.
    This works fine in Xena+ releases but not in Wallaby and olders where
    old callbacks mechanism is still used, at least for the security group
    AFTER_DELETE events.
    So this patch fixes that by changing to the old callbacks mechanism in
    the logging service plugin in stable/wallaby and older branches only.

    Closes-Bug: #1955546

    [1] https://review.opendev.org/c/openstack/neutron/+/804237
    [2] https://review.opendev.org/c/openstack/neutron/+/812459
    [3] https://bugs.launchpad.net/neutron/+bug/1939558

    Change-Id: I0003bd566dc769436ad1342351ad058394bd52da

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

Reviewed: https://review.opendev.org/c/openstack/neutron/+/822611
Committed: https://opendev.org/openstack/neutron/commit/eb1190d2067f24b6accc3463e601e5b41c0e0be4
Submitter: "Zuul (22348)"
Branch: stable/ussuri

commit eb1190d2067f24b6accc3463e601e5b41c0e0be4
Author: Slawek Kaplonski <email address hidden>
Date: Wed Dec 22 08:37:45 2021 +0100

    [Stable only] Fix callback function arguments

    Fixes [1] and [2] for bug [3] are using new callabacks mechanism and
    expects payload as one of the arguments passed to the callback functions.
    This works fine in Xena+ releases but not in Wallaby and olders where
    old callbacks mechanism is still used, at least for the security group
    AFTER_DELETE events.
    So this patch fixes that by changing to the old callbacks mechanism in
    the logging service plugin in stable/wallaby and older branches only.

    Closes-Bug: #1955546

    [1] https://review.opendev.org/c/openstack/neutron/+/804237
    [2] https://review.opendev.org/c/openstack/neutron/+/812459
    [3] https://bugs.launchpad.net/neutron/+bug/1939558

    Change-Id: I0003bd566dc769436ad1342351ad058394bd52da

tags: added: in-stable-ussuri
yatin (yatinkarel)
Changed in neutron:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (stable/train)

Reviewed: https://review.opendev.org/c/openstack/neutron/+/822612
Committed: https://opendev.org/openstack/neutron/commit/65310eb566f82258847c08d8f48725a47152b053
Submitter: "Zuul (22348)"
Branch: stable/train

commit 65310eb566f82258847c08d8f48725a47152b053
Author: Slawek Kaplonski <email address hidden>
Date: Wed Dec 22 08:37:45 2021 +0100

    [Stable only] Fix callback function arguments

    Fixes [1] and [2] for bug [3] are using new callabacks mechanism and
    expects payload as one of the arguments passed to the callback functions.
    This works fine in Xena+ releases but not in Wallaby and olders where
    old callbacks mechanism is still used, at least for the security group
    AFTER_DELETE events.
    So this patch fixes that by changing to the old callbacks mechanism in
    the logging service plugin in stable/wallaby and older branches only.

    Closes-Bug: #1955546

    [1] https://review.opendev.org/c/openstack/neutron/+/804237
    [2] https://review.opendev.org/c/openstack/neutron/+/812459
    [3] https://bugs.launchpad.net/neutron/+bug/1939558

    Change-Id: I0003bd566dc769436ad1342351ad058394bd52da

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

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

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

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

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron train-eol

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

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

This issue was fixed in the openstack/neutron ussuri-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.