From 888593ad223df6bac996369b06c625c8cc70c7e0 Mon Sep 17 00:00:00 2001 From: Alexandre Aubin Date: Tue, 21 Feb 2023 14:57:10 +0100 Subject: [PATCH] appsv2: fix resource provisioning scripts picking up already-closed operation logger, resulting in confusing debugging output --- src/utils/resources.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/utils/resources.py b/src/utils/resources.py index 0b9cb9968..72475fae4 100644 --- a/src/utils/resources.py +++ b/src/utils/resources.py @@ -225,9 +225,10 @@ ynh_abort_if_errors from yunohost.log import OperationLogger - if OperationLogger._instances: - # FIXME ? : this is an ugly hack :( - operation_logger = OperationLogger._instances[-1] + # FIXME ? : this is an ugly hack :( + active_operation_loggers = [o for o in OperationLogger._instances if o.ended_at is None] + if active_operation_loggers: + operation_logger = active_operation_loggers[-1] else: operation_logger = OperationLogger( "resource_snippet", [("app", self.app)], env=env_