From 3ce0e9b058aaf1ee94dbf956d52d3af77db0034e Mon Sep 17 00:00:00 2001 From: Alexis Lothoré Date: Mon, 12 Aug 2024 17:14:03 +0200 Subject: oeqa/postactions: fix exception handling MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The current exception handler in list_and_fetch_failed_tests_artifacts expects a non-exisiting variable and then fail to display the original exception message since it raises a new one. The issue has been introduced with commit 6e80b2ab660e ("oeqa/utils/postactions: transfer whole archive over ssh instead of doing individual copies"). Now that tests artifacts are now handled individually, there's no point of trying to print individual names in the exception. (From OE-Core rev: a50e72bb64fb8b0d14c23164eaeeabd9c271ac19) Signed-off-by: Alexis Lothoré Signed-off-by: Richard Purdie --- meta/lib/oeqa/utils/postactions.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/meta/lib/oeqa/utils/postactions.py b/meta/lib/oeqa/utils/postactions.py index 2de2d18328..3b537146c5 100644 --- a/meta/lib/oeqa/utils/postactions.py +++ b/meta/lib/oeqa/utils/postactions.py @@ -70,7 +70,7 @@ def list_and_fetch_failed_tests_artifacts(d, tc, artifacts_list, outputdir): raise Exception("Error while fetching compressed artifacts") p = subprocess.run(["tar", "zxf", "-", "-C", outputdir], input=output) except Exception as e: - bb.warn(f"Can not retrieve {artifact_path} from test target: {e}") + bb.warn(f"Can not retrieve artifacts from test target: {e}") ################################################################## -- cgit v1.2.3-54-g00ecf