Skip to content

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Aug 5, 2024

This PR contains a snapshot of 2023.1 from upstream stable/2023.1.

ralonsoh and others added 7 commits July 25, 2024 02:43
The method ``ProcessMonitor._check_child_processes`` was releasing
the thread executor inside a method that creates a lock for the resource
"_check_child_processes". Despite this resource is not used anywhere
else (at least for this instance), this could lead to a potential
deadlock.

The current implementation of ``lockutils.synchronized`` with the
default value "external=False" and "fair=False" is a
``threading.Lock()`` instance. The goal of this lock is, precisely, to
execute the code inside the locked code without any interruption and
then to be able to release the executor.

Closes-Bug: #2073743
Change-Id: I44c7a4ce81a67b86054832ac050cf5b465727adf
(cherry picked from commit baa57ab)
If northd is very busy, it may happen port is deleted when handling an
LSP down event causing standard attribute being gone when bumping ovn
revision number. This is because the port is set down in SB DB first and
then northd propagates that to NB DB, and then the event is emited.

This patch just makes sure the traceback is not printed in case this
happens.

TrivialFix
Closes-bug: #2069442

Depends-On: https://review.opendev.org/c/openstack/nova/+/925000
Change-Id: I7d21e4adc27fab411346e0458c92191e69ce6b30
Signed-off-by: Jakub Libosvar <libosvar@redhat.com>
(cherry picked from commit 8ab385f)
Almost all of the periodic methods in ml2/OVN's maintenance code
exit early if the IDL lock is not held. This adds a decorator
that wraps the futurist periodic that exits if the lock isn't held.

There are still a few methods that exit raising NeverAgain if
certain features aren't available, and those have been left alone
for now.

Conflicts:
    neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/maintenance.py

Partial-bug: #2074209

Change-Id: I9771bd4f76a9ec073afeeb80a787832102446cd6
(cherry picked from commit 1d7e99b)
(cherry picked from commit 35d0535)
This is follow up patch for the [1] which introduced this new decorator.

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

Change-Id: I2de3b5d7ba5783dd82acacda89ab4b64c2d29149
(cherry picked from commit 2a6bc5d)
(cherry picked from commit e007ea6)
@github-actions github-actions bot requested a review from a team as a code owner August 5, 2024 08:23
@github-actions github-actions bot added automated Automated action performed by GitHub Actions synchronisation labels Aug 5, 2024
@markgoddard markgoddard merged commit 744847f into stackhpc/2023.1 Aug 5, 2024
@markgoddard markgoddard deleted the upstream/2023.1-2024-08-05 branch August 5, 2024 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automated Automated action performed by GitHub Actions synchronisation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants