Skip to content

Commit a5d16ab

Browse files
committed
sanitycheck: add option to store platform reports
Now we create plaform reports for each enabled platform, this is time consuming, especially if you are not using those reports for anything. Make this an option and enable only if the generated reports are going to be used. Individual reports can also be generated from the JSON file if needed. Signed-off-by: Anas Nashif <[email protected]>
1 parent 7822e7b commit a5d16ab

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

scripts/pylib/twister/twisterlib.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2732,10 +2732,11 @@ def summary(self, results, unrecognized_sections):
27322732
logger.info(f"{Fore.GREEN}{run}{Fore.RESET} test configurations executed on platforms, \
27332733
{Fore.RED}{results.total - run - results.skipped_configs}{Fore.RESET} test configurations were only built.")
27342734

2735-
def save_reports(self, name, suffix, report_dir, no_update, release, only_failed):
2735+
def save_reports(self, name, suffix, report_dir, no_update, release, only_failed, platform_reports):
27362736
if not self.instances:
27372737
return
27382738

2739+
logger.info("Saving reports...")
27392740
if name:
27402741
report_name = name
27412742
else:
@@ -2760,7 +2761,8 @@ def save_reports(self, name, suffix, report_dir, no_update, release, only_failed
27602761
self.csv_report(filename + ".csv")
27612762
self.json_report(filename + ".json", append=only_failed, version=self.version)
27622763

2763-
self.target_report(outdir, suffix, append=only_failed)
2764+
if platform_reports:
2765+
self.target_report(outdir, suffix, append=only_failed)
27642766
if self.discards:
27652767
self.discard_report(filename + "_discard.csv")
27662768

scripts/twister

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -338,6 +338,10 @@ Artificially long but functional example:
338338
(twister.csv and twister.xml).
339339
""")
340340

341+
parser.add_argument(
342+
"--platform-reports", action="store_true",
343+
help="""Create individual reports for each platform.
344+
""")
341345
parser.add_argument(
342346
"--report-name",
343347
help="""Create a report with a custom name.
@@ -1248,7 +1252,9 @@ def main():
12481252
options.report_dir,
12491253
options.no_update,
12501254
options.release,
1251-
options.only_failed)
1255+
options.only_failed,
1256+
options.platform_reports,
1257+
)
12521258

12531259
# FIXME: remove later
12541260
#logger.info(f"failed: {results.failed}, cases: {results.cases}, skipped configurations: {results.skipped_configs}, skipped_cases: {results.skipped_cases}, skipped(runtime): {results.skipped_runtime}, passed: {results.passed}, total: {results.total}, done: {results.done}")

0 commit comments

Comments
 (0)