2021-09-10 11:52:21 +00:00
|
|
|
#!/usr/bin/env python3
|
2022-07-19 13:17:05 +00:00
|
|
|
import argparse
|
2022-07-19 11:31:43 +00:00
|
|
|
import csv
|
2021-09-10 11:52:21 +00:00
|
|
|
import logging
|
|
|
|
import os
|
2022-07-19 11:31:43 +00:00
|
|
|
import subprocess
|
2021-12-01 14:23:51 +00:00
|
|
|
import sys
|
2022-07-30 05:07:22 +00:00
|
|
|
import atexit
|
2021-12-01 14:23:51 +00:00
|
|
|
|
2021-11-26 14:00:09 +00:00
|
|
|
|
2022-01-13 19:52:02 +00:00
|
|
|
from clickhouse_helper import (
|
|
|
|
ClickHouseHelper,
|
|
|
|
mark_flaky_tests,
|
|
|
|
prepare_tests_results_for_clickhouse,
|
|
|
|
)
|
2022-07-30 05:07:22 +00:00
|
|
|
from commit_status_helper import post_commit_status, update_mergeable_check
|
2022-07-19 11:31:43 +00:00
|
|
|
from docker_pull_helper import get_image_with_version
|
2022-08-11 13:01:32 +00:00
|
|
|
from env_helper import GITHUB_WORKSPACE, RUNNER_TEMP
|
2022-07-19 11:31:43 +00:00
|
|
|
from get_robot_token import get_best_robot_token
|
|
|
|
from github_helper import GitHub
|
2022-07-19 13:17:05 +00:00
|
|
|
from git_helper import git_runner
|
2022-07-19 11:31:43 +00:00
|
|
|
from pr_info import PRInfo
|
2021-12-01 14:23:51 +00:00
|
|
|
from rerun_helper import RerunHelper
|
2022-07-19 11:31:43 +00:00
|
|
|
from s3_helper import S3Helper
|
2022-07-19 13:17:05 +00:00
|
|
|
from ssh import SSHKey
|
2022-07-19 11:31:43 +00:00
|
|
|
from stopwatch import Stopwatch
|
|
|
|
from upload_result_helper import upload_results
|
2021-09-10 14:27:03 +00:00
|
|
|
|
2022-07-21 11:10:22 +00:00
|
|
|
NAME = "Style Check"
|
2021-09-15 09:02:38 +00:00
|
|
|
|
2021-09-10 14:27:03 +00:00
|
|
|
|
2021-09-10 11:52:21 +00:00
|
|
|
def process_result(result_folder):
|
|
|
|
test_results = []
|
|
|
|
additional_files = []
|
|
|
|
# Just upload all files from result_folder.
|
2022-01-13 19:52:02 +00:00
|
|
|
# If task provides processed results, then it's responsible
|
|
|
|
# for content of result_folder.
|
2021-09-10 11:52:21 +00:00
|
|
|
if os.path.exists(result_folder):
|
2022-01-13 19:52:02 +00:00
|
|
|
test_files = [
|
|
|
|
f
|
|
|
|
for f in os.listdir(result_folder)
|
|
|
|
if os.path.isfile(os.path.join(result_folder, f))
|
|
|
|
]
|
2021-09-10 11:52:21 +00:00
|
|
|
additional_files = [os.path.join(result_folder, f) for f in test_files]
|
|
|
|
|
2021-12-12 12:09:44 +00:00
|
|
|
status = []
|
2021-09-10 11:52:21 +00:00
|
|
|
status_path = os.path.join(result_folder, "check_status.tsv")
|
2021-12-12 12:09:44 +00:00
|
|
|
if os.path.exists(status_path):
|
|
|
|
logging.info("Found test_results.tsv")
|
2022-01-13 19:52:02 +00:00
|
|
|
with open(status_path, "r", encoding="utf-8") as status_file:
|
|
|
|
status = list(csv.reader(status_file, delimiter="\t"))
|
2021-09-10 11:52:21 +00:00
|
|
|
if len(status) != 1 or len(status[0]) != 2:
|
2021-12-12 12:20:25 +00:00
|
|
|
logging.info("Files in result folder %s", os.listdir(result_folder))
|
2021-09-10 11:52:21 +00:00
|
|
|
return "error", "Invalid check_status.tsv", test_results, additional_files
|
|
|
|
state, description = status[0][0], status[0][1]
|
|
|
|
|
|
|
|
try:
|
|
|
|
results_path = os.path.join(result_folder, "test_results.tsv")
|
2022-07-19 11:31:43 +00:00
|
|
|
with open(results_path, "r", encoding="utf-8") as fd:
|
|
|
|
test_results = list(csv.reader(fd, delimiter="\t"))
|
2021-09-10 11:52:21 +00:00
|
|
|
if len(test_results) == 0:
|
|
|
|
raise Exception("Empty results")
|
|
|
|
|
|
|
|
return state, description, test_results, additional_files
|
|
|
|
except Exception:
|
|
|
|
if state == "success":
|
|
|
|
state, description = "error", "Failed to read test_results.tsv"
|
|
|
|
return state, description, test_results, additional_files
|
|
|
|
|
2021-11-29 14:17:17 +00:00
|
|
|
|
2022-07-19 13:17:05 +00:00
|
|
|
def parse_args():
|
|
|
|
parser = argparse.ArgumentParser("Check and report style issues in the repository")
|
|
|
|
parser.add_argument("--push", default=True, help=argparse.SUPPRESS)
|
|
|
|
parser.add_argument(
|
|
|
|
"--no-push",
|
|
|
|
action="store_false",
|
|
|
|
dest="push",
|
|
|
|
help="do not commit and push automatic fixes",
|
|
|
|
default=argparse.SUPPRESS,
|
|
|
|
)
|
|
|
|
return parser.parse_args()
|
|
|
|
|
|
|
|
|
|
|
|
def checkout_head(pr_info: PRInfo):
|
|
|
|
# It works ONLY for PRs, and only over ssh, so either
|
|
|
|
# ROBOT_CLICKHOUSE_SSH_KEY should be set or ssh-agent should work
|
|
|
|
assert pr_info.number
|
|
|
|
if not pr_info.head_name == pr_info.base_name:
|
|
|
|
# We can't push to forks, sorry folks
|
|
|
|
return
|
|
|
|
remote_url = pr_info.event["pull_request"]["base"]["repo"]["ssh_url"]
|
|
|
|
git_prefix = ( # All commits to remote are done as robot-clickhouse
|
|
|
|
"git -c user.email=robot-clickhouse@clickhouse.com "
|
2022-07-21 16:39:14 +00:00
|
|
|
"-c user.name=robot-clickhouse -c commit.gpgsign=false "
|
|
|
|
"-c core.sshCommand="
|
|
|
|
"'ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no'"
|
2022-07-19 13:17:05 +00:00
|
|
|
)
|
|
|
|
fetch_cmd = (
|
|
|
|
f"{git_prefix} fetch --depth=1 "
|
|
|
|
f"{remote_url} {pr_info.head_ref}:head-{pr_info.head_ref}"
|
|
|
|
)
|
|
|
|
if os.getenv("ROBOT_CLICKHOUSE_SSH_KEY", ""):
|
|
|
|
with SSHKey("ROBOT_CLICKHOUSE_SSH_KEY"):
|
|
|
|
git_runner(fetch_cmd)
|
|
|
|
else:
|
|
|
|
git_runner(fetch_cmd)
|
|
|
|
git_runner(f"git checkout -f head-{pr_info.head_ref}")
|
|
|
|
|
|
|
|
|
|
|
|
def commit_push_staged(pr_info: PRInfo):
|
|
|
|
# It works ONLY for PRs, and only over ssh, so either
|
|
|
|
# ROBOT_CLICKHOUSE_SSH_KEY should be set or ssh-agent should work
|
|
|
|
assert pr_info.number
|
|
|
|
if not pr_info.head_name == pr_info.base_name:
|
|
|
|
# We can't push to forks, sorry folks
|
|
|
|
return
|
|
|
|
git_staged = git_runner("git diff --cached --name-only")
|
|
|
|
if not git_staged:
|
|
|
|
return
|
|
|
|
remote_url = pr_info.event["pull_request"]["base"]["repo"]["ssh_url"]
|
|
|
|
git_prefix = ( # All commits to remote are done as robot-clickhouse
|
|
|
|
"git -c user.email=robot-clickhouse@clickhouse.com "
|
|
|
|
"-c user.name=robot-clickhouse -c commit.gpgsign=false "
|
|
|
|
"-c core.sshCommand="
|
|
|
|
"'ssh -o UserKnownHostsFile=/dev/null -o StrictHostKeyChecking=no'"
|
|
|
|
)
|
|
|
|
git_runner(f"{git_prefix} commit -m 'Automatic style fix'")
|
|
|
|
push_cmd = (
|
|
|
|
f"{git_prefix} push {remote_url} head-{pr_info.head_ref}:{pr_info.head_ref}"
|
|
|
|
)
|
|
|
|
if os.getenv("ROBOT_CLICKHOUSE_SSH_KEY", ""):
|
|
|
|
with SSHKey("ROBOT_CLICKHOUSE_SSH_KEY"):
|
|
|
|
git_runner(push_cmd)
|
|
|
|
else:
|
|
|
|
git_runner(push_cmd)
|
|
|
|
|
|
|
|
|
2021-09-10 11:52:21 +00:00
|
|
|
if __name__ == "__main__":
|
2021-09-10 14:27:03 +00:00
|
|
|
logging.basicConfig(level=logging.INFO)
|
2022-07-19 13:17:05 +00:00
|
|
|
logging.getLogger("git_helper").setLevel(logging.DEBUG)
|
|
|
|
args = parse_args()
|
2021-11-19 14:47:04 +00:00
|
|
|
|
|
|
|
stopwatch = Stopwatch()
|
|
|
|
|
2021-11-26 14:00:09 +00:00
|
|
|
repo_path = GITHUB_WORKSPACE
|
2022-01-13 19:52:02 +00:00
|
|
|
temp_path = os.path.join(RUNNER_TEMP, "style_check")
|
2021-09-15 12:59:39 +00:00
|
|
|
|
2021-11-26 14:00:09 +00:00
|
|
|
pr_info = PRInfo()
|
2022-07-19 13:17:05 +00:00
|
|
|
if args.push:
|
|
|
|
checkout_head(pr_info)
|
2021-09-15 12:59:39 +00:00
|
|
|
|
2022-07-19 11:31:43 +00:00
|
|
|
gh = GitHub(get_best_robot_token())
|
2021-12-01 14:23:51 +00:00
|
|
|
|
2022-07-30 05:07:22 +00:00
|
|
|
atexit.register(update_mergeable_check, gh, pr_info, NAME)
|
|
|
|
|
2021-12-01 14:23:51 +00:00
|
|
|
rerun_helper = RerunHelper(gh, pr_info, NAME)
|
|
|
|
if rerun_helper.is_already_finished_by_status():
|
|
|
|
logging.info("Check is already finished according to github status, exiting")
|
2022-07-21 16:25:22 +00:00
|
|
|
# Finish with the same code as previous
|
|
|
|
state = rerun_helper.get_finished_status().state # type: ignore
|
|
|
|
# state == "success" -> code = 0
|
|
|
|
code = int(state != "success")
|
|
|
|
sys.exit(code)
|
2021-12-01 14:23:51 +00:00
|
|
|
|
2021-09-15 18:52:37 +00:00
|
|
|
if not os.path.exists(temp_path):
|
|
|
|
os.makedirs(temp_path)
|
|
|
|
|
2022-01-13 19:52:02 +00:00
|
|
|
docker_image = get_image_with_version(temp_path, "clickhouse/style-test")
|
2022-08-11 13:01:32 +00:00
|
|
|
s3_helper = S3Helper()
|
2021-09-10 11:52:21 +00:00
|
|
|
|
2022-02-03 13:06:21 +00:00
|
|
|
cmd = (
|
2022-01-13 19:52:02 +00:00
|
|
|
f"docker run -u $(id -u ${{USER}}):$(id -g ${{USER}}) --cap-add=SYS_PTRACE "
|
|
|
|
f"--volume={repo_path}:/ClickHouse --volume={temp_path}:/test_output "
|
2022-02-03 13:06:21 +00:00
|
|
|
f"{docker_image}"
|
|
|
|
)
|
|
|
|
|
|
|
|
logging.info("Is going to run the command: %s", cmd)
|
|
|
|
subprocess.check_call(
|
|
|
|
cmd,
|
2022-01-13 19:52:02 +00:00
|
|
|
shell=True,
|
|
|
|
)
|
2022-02-03 13:06:21 +00:00
|
|
|
|
2022-07-19 13:17:05 +00:00
|
|
|
if args.push:
|
|
|
|
commit_push_staged(pr_info)
|
|
|
|
|
2021-09-15 10:50:30 +00:00
|
|
|
state, description, test_results, additional_files = process_result(temp_path)
|
2021-11-19 14:47:04 +00:00
|
|
|
ch_helper = ClickHouseHelper()
|
|
|
|
mark_flaky_tests(ch_helper, NAME, test_results)
|
|
|
|
|
2022-01-13 19:52:02 +00:00
|
|
|
report_url = upload_results(
|
|
|
|
s3_helper, pr_info.number, pr_info.sha, test_results, additional_files, NAME
|
|
|
|
)
|
2022-07-19 11:31:43 +00:00
|
|
|
print(f"::notice ::Report url: {report_url}")
|
2021-11-12 12:36:25 +00:00
|
|
|
post_commit_status(gh, pr_info.sha, NAME, description, state, report_url)
|
2021-11-19 14:47:04 +00:00
|
|
|
|
2022-01-13 19:52:02 +00:00
|
|
|
prepared_events = prepare_tests_results_for_clickhouse(
|
|
|
|
pr_info,
|
|
|
|
test_results,
|
|
|
|
state,
|
|
|
|
stopwatch.duration_seconds,
|
|
|
|
stopwatch.start_time_str,
|
|
|
|
report_url,
|
|
|
|
NAME,
|
|
|
|
)
|
2022-03-29 19:06:50 +00:00
|
|
|
ch_helper.insert_events_into(db="default", table="checks", events=prepared_events)
|
2022-03-29 12:41:47 +00:00
|
|
|
|
2022-07-18 19:15:21 +00:00
|
|
|
if state in ["error", "failure"]:
|
2022-03-29 12:41:47 +00:00
|
|
|
sys.exit(1)
|