From ac26fec9c6c210f08b356609661dc1ec20e04d9e Mon Sep 17 00:00:00 2001 From: Karol Latecki Date: Tue, 28 Jan 2020 10:08:58 +0100 Subject: [PATCH] autorun_post: skip confirming executed tests Allow to skip confirmPerPatchTests if needed. Change-Id: I8741d80de5cac9954e3429b951a71dc065c40bb5 Signed-off-by: Karol Latecki Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/483016 Tested-by: SPDK CI Jenkins Reviewed-by: Seth Howell Reviewed-by: Ben Walker Reviewed-by: Shuhei Matsumoto Community-CI: SPDK CI Jenkins --- autorun_post.py | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/autorun_post.py b/autorun_post.py index 05787d106..b808648f2 100755 --- a/autorun_post.py +++ b/autorun_post.py @@ -144,7 +144,7 @@ def confirmPerPatchTests(test_list, skiplist): exit(1) -def aggregateCompletedTests(output_dir, repo_dir): +def aggregateCompletedTests(output_dir, repo_dir, skip_confirm=False): test_list = {} test_completion_table = [] @@ -172,14 +172,15 @@ def aggregateCompletedTests(output_dir, repo_dir): printListInformation("Tests", test_list) generateTestCompletionTables(output_dir, test_completion_table) skipped_tests = getSkippedTests(repo_dir) - confirmPerPatchTests(test_list, skipped_tests) + if not skip_confirm: + confirmPerPatchTests(test_list, skipped_tests) -def main(output_dir, repo_dir): +def main(output_dir, repo_dir, skip_confirm=False): generateCoverageReport(output_dir, repo_dir) collectOne(output_dir, 'doc') collectOne(output_dir, 'ut_coverage') - aggregateCompletedTests(output_dir, repo_dir) + aggregateCompletedTests(output_dir, repo_dir, skip_confirm) if __name__ == "__main__": @@ -188,5 +189,7 @@ if __name__ == "__main__": help="The location of your build's output directory") parser.add_argument("-r", "--repo_directory", type=str, required=True, help="The location of your spdk repository") + parser.add_argument("-s", "--skip_confirm", required=False, action="store_true", + help="Do not check if all autotest.sh tests were executed.") args = parser.parse_args() - main(args.directory_location, args.repo_directory) + main(args.directory_location, args.repo_directory, args.skip_confirm)