autorun_post: skip confirming executed tests
Allow to skip confirmPerPatchTests if needed. Change-Id: I8741d80de5cac9954e3429b951a71dc065c40bb5 Signed-off-by: Karol Latecki <karol.latecki@intel.com> Reviewed-on: https://review.gerrithub.io/c/spdk/spdk/+/483016 Tested-by: SPDK CI Jenkins <sys_sgci@intel.com> Reviewed-by: Seth Howell <seth.howell@intel.com> Reviewed-by: Ben Walker <benjamin.walker@intel.com> Reviewed-by: Shuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com> Community-CI: SPDK CI Jenkins <sys_sgci@intel.com>
This commit is contained in:
parent
f257be1313
commit
ac26fec9c6
@ -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)
|
||||
|
Loading…
Reference in New Issue
Block a user