]> git.ipfire.org Git - thirdparty/openembedded/openembedded-core-contrib.git/commitdiff
resulttool: Improve overlapping ptest result reporting
authorRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 16 Mar 2023 22:51:10 +0000 (22:51 +0000)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 16 Mar 2023 22:52:13 +0000 (22:52 +0000)
With the move to a ptest per image, the multiple ptest log scenario
is much more likely. Tweak the handling to only warn if there are
overlapping files.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
scripts/lib/resulttool/log.py

index 621c6c4bacc7b02907612cf51b878cb2abbb601a..15148ca2881fc197cb6bcd0509cba92a713b4946 100644 (file)
@@ -28,11 +28,6 @@ def show_reproducible(result, reproducible, logger):
 def log(args, logger):
     results = resultutils.load_resultsdata(args.source)
 
-    ptest_count = sum(1 for _, _, _, r in resultutils.test_run_results(results) if 'ptestresult.sections' in r)
-    if ptest_count > 1 and not args.prepend_run:
-        print("%i ptest sections found. '--prepend-run' is required" % ptest_count)
-        return 1
-
     for _, run_name, _, r in resultutils.test_run_results(results):
         if args.list_ptest:
             print('\n'.join(sorted(r['ptestresult.sections'].keys())))
@@ -51,6 +46,9 @@ def log(args, logger):
 
                     os.makedirs(dest_dir, exist_ok=True)
                     dest = os.path.join(dest_dir, '%s.log' % name)
+                    if os.path.exists(dest):
+                        print("Overlapping ptest logs found, skipping %s. The '--prepend-run' option would avoid this" % name)
+                        continue
                     print(dest)
                     with open(dest, 'w') as f:
                         f.write(logdata)