thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-26 09:32:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix minor typo: replace validate_bugix_check with validate_bugfix_check

Browse Source 
Signed-off-by: Pradeep Chhetri <pradeepchhetri4444@gmail.com>
This commit is contained in:
Pradeep Chhetri 2022-12-28 20:53:23 +08:00
parent 2892d447d4
commit d36f2d7ad9
2 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
tests/ci/functional_test_check.py
View File

@ -207,16 +207,16 @@ if __name__ == "__main__":
args = parse_args()
check_name = args.check_name
kill_timeout = args.kill_timeout
validate_bugix_check = args.validate_bugfix
validate_bugfix_check = args.validate_bugfix
flaky_check = "flaky" in check_name.lower()
run_changed_tests = flaky_check or validate_bugix_check
run_changed_tests = flaky_check or validate_bugfix_check
gh = Github(get_best_robot_token(), per_page=100)
# For validate_bugix_check we need up to date information about labels, so pr_event_from_api is used
# For validate_bugfix_check we need up to date information about labels, so pr_event_from_api is used
pr_info = PRInfo(
need_changed_files=run_changed_tests, pr_event_from_api=validate_bugix_check
need_changed_files=run_changed_tests, pr_event_from_api=validate_bugfix_check
)
atexit.register(update_mergeable_check, gh, pr_info, check_name)
@ -224,7 +224,7 @@ if __name__ == "__main__":
if not os.path.exists(temp_path):
os.makedirs(temp_path)
if validate_bugix_check and "pr-bugfix" not in pr_info.labels:
if validate_bugfix_check and "pr-bugfix" not in pr_info.labels:
if args.post_commit_status == "file":
post_commit_status_to_file(
os.path.join(temp_path, "post_commit_status.tsv"),
@ -256,7 +256,7 @@ if __name__ == "__main__":
tests_to_run = get_tests_to_run(pr_info)
if not tests_to_run:
commit = get_commit(gh, pr_info.sha)
state = override_status("success", check_name, validate_bugix_check)
state = override_status("success", check_name, validate_bugfix_check)
if args.post_commit_status == "commit_status":
commit.create_status(
context=check_name_with_group,
@ -279,7 +279,7 @@ if __name__ == "__main__":
if not os.path.exists(packages_path):
os.makedirs(packages_path)
if validate_bugix_check:
if validate_bugfix_check:
download_last_release(packages_path)
else:
download_all_deb_packages(check_name, reports_path, packages_path)
@ -297,7 +297,7 @@ if __name__ == "__main__":
additional_envs = get_additional_envs(
check_name, run_by_hash_num, run_by_hash_total
)
if validate_bugix_check:
if validate_bugfix_check:
additional_envs.append("GLOBAL_TAGS=no-random-settings")
run_command = get_run_command(
@ -327,7 +327,7 @@ if __name__ == "__main__":
state, description, test_results, additional_logs = process_results(
result_path, server_log_path
)
state = override_status(state, check_name, invert=validate_bugix_check)
state = override_status(state, check_name, invert=validate_bugfix_check)
ch_helper = ClickHouseHelper()
mark_flaky_tests(ch_helper, check_name, test_results)

14
tests/ci/integration_test_check.py
View File

@ -153,7 +153,7 @@ if __name__ == "__main__":
args = parse_args()
check_name = args.check_name
validate_bugix_check = args.validate_bugfix
validate_bugfix_check = args.validate_bugfix
if "RUN_BY_HASH_NUM" in os.environ:
run_by_hash_num = int(os.getenv("RUN_BY_HASH_NUM", "0"))
@ -171,13 +171,13 @@ if __name__ == "__main__":
is_flaky_check = "flaky" in check_name
# For validate_bugix_check we need up to date information about labels, so pr_event_from_api is used
# For validate_bugfix_check we need up to date information about labels, so pr_event_from_api is used
pr_info = PRInfo(
need_changed_files=is_flaky_check or validate_bugix_check,
pr_event_from_api=validate_bugix_check,
need_changed_files=is_flaky_check or validate_bugfix_check,
pr_event_from_api=validate_bugfix_check,
)
if validate_bugix_check and "pr-bugfix" not in pr_info.labels:
if validate_bugfix_check and "pr-bugfix" not in pr_info.labels:
if args.post_commit_status == "file":
post_commit_status_to_file(
os.path.join(temp_path, "post_commit_status.tsv"),
@ -209,7 +209,7 @@ if __name__ == "__main__":
if not os.path.exists(build_path):
os.makedirs(build_path)
if validate_bugix_check:
if validate_bugfix_check:
download_last_release(build_path)
else:
download_all_deb_packages(check_name, reports_path, build_path)
@ -252,7 +252,7 @@ if __name__ == "__main__":
subprocess.check_call(f"sudo chown -R ubuntu:ubuntu {temp_path}", shell=True)
state, description, test_results, additional_logs = process_results(result_path)
state = override_status(state, check_name, invert=validate_bugix_check)
state = override_status(state, check_name, invert=validate_bugfix_check)
ch_helper = ClickHouseHelper()
mark_flaky_tests(ch_helper, check_name, test_results)