diff options
author | Alexis Lothoré <alexis.lothore@bootlin.com> | 2024-08-12 17:14:03 +0200 |
---|---|---|
committer | Steve Sakoman <steve@sakoman.com> | 2024-09-25 08:04:40 -0700 |
commit | bf88a67b45235236d6655dce604e632eb94a813c (patch) | |
tree | c5a7488fef60d083aa6fd187aaa9bbc435092dbe | |
parent | 88c86347f70bff692d9bfdb5eec2eeb6b2d031a9 (diff) | |
download | poky-bf88a67b45235236d6655dce604e632eb94a813c.tar.gz |
oeqa/postactions: fix exception handling
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: 60a7448abce091eb8c1cb953058fade0beb8b670)
Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit a50e72bb64fb8b0d14c23164eaeeabd9c271ac19)
Signed-off-by: Steve Sakoman <steve@sakoman.com>
-rw-r--r-- | meta/lib/oeqa/utils/postactions.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/meta/lib/oeqa/utils/postactions.py b/meta/lib/oeqa/utils/postactions.py index 2a08129d6c..a0e3b70892 100644 --- a/meta/lib/oeqa/utils/postactions.py +++ b/meta/lib/oeqa/utils/postactions.py | |||
@@ -71,7 +71,7 @@ def retrieve_test_artifacts(target, artifacts_list, target_dir): | |||
71 | raise Exception("Error while fetching compressed artifacts") | 71 | raise Exception("Error while fetching compressed artifacts") |
72 | p = subprocess.run(["tar", "zxf", "-", "-C", local_artifacts_dir], input=output) | 72 | p = subprocess.run(["tar", "zxf", "-", "-C", local_artifacts_dir], input=output) |
73 | except Exception as e: | 73 | except Exception as e: |
74 | bb.warn(f"Can not retrieve {artifact_path} from test target: {e}") | 74 | bb.warn(f"Can not retrieve artifacts from test target: {e}") |
75 | 75 | ||
76 | def list_and_fetch_failed_tests_artifacts(d, tc): | 76 | def list_and_fetch_failed_tests_artifacts(d, tc): |
77 | artifacts_list = get_artifacts_list(tc.target, d.getVar("TESTIMAGE_FAILED_QA_ARTIFACTS")) | 77 | artifacts_list = get_artifacts_list(tc.target, d.getVar("TESTIMAGE_FAILED_QA_ARTIFACTS")) |