Merge pull request #30444 from ClickHouse/trying_actions

Remove statuses from actions

.github/workflows/main.yml

@@ -17,11 +17,6 @@ jobs:
         uses: actions/checkout@v2
       - name: Labels check
         run: cd $GITHUB_WORKSPACE/tests/ci && python3 run_check.py
-        env:
-          ROBOT_TOKEN_0: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN }}
-          ROBOT_TOKEN_1: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_1 }}
-          ROBOT_TOKEN_2: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_2 }}
-          ROBOT_TOKEN_3: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_3 }}
   DockerHubPush:
     needs: CheckLabels
     runs-on: [self-hosted]
@@ -33,10 +28,6 @@ jobs:
         env:
           YANDEX_S3_ACCESS_KEY_ID: ${{ secrets.YANDEX_S3_ACCESS_KEY_ID }}
           YANDEX_S3_ACCESS_SECRET_KEY: ${{ secrets.YANDEX_S3_ACCESS_SECRET_KEY }}
-          ROBOT_TOKEN_0: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN }}
-          ROBOT_TOKEN_1: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_1 }}
-          ROBOT_TOKEN_2: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_2 }}
-          ROBOT_TOKEN_3: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_3 }}
           DOCKER_ROBOT_PASSWORD: ${{ secrets.DOCKER_ROBOT_PASSWORD }}
       - name: Upload images files to artifacts
         uses: actions/upload-artifact@v2
@@ -58,21 +49,4 @@ jobs:
         env:
           YANDEX_S3_ACCESS_KEY_ID: ${{ secrets.YANDEX_S3_ACCESS_KEY_ID }}
           YANDEX_S3_ACCESS_SECRET_KEY: ${{ secrets.YANDEX_S3_ACCESS_SECRET_KEY }}
-          ROBOT_TOKEN_0: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN }}
-          ROBOT_TOKEN_1: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_1 }}
-          ROBOT_TOKEN_2: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_2 }}
-          ROBOT_TOKEN_3: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_3 }}
         run: cd $GITHUB_WORKSPACE/tests/ci && python3 style_check.py
-  FinishCheck:
-    needs: [StyleCheck, DockerHubPush, CheckLabels]
-    runs-on: [self-hosted]
-    steps:
-      - name: Check out repository code
-        uses: actions/checkout@v2
-      - name: Finish label
-        run: cd $GITHUB_WORKSPACE/tests/ci && python3 finish_check.py
-        env:
-          ROBOT_TOKEN_0: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN }}
-          ROBOT_TOKEN_1: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_1 }}
-          ROBOT_TOKEN_2: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_2 }}
-          ROBOT_TOKEN_3: ${{ secrets.ROBOT_CLICKHOUSE_PERSONAL_ACCESS_TOKEN_3 }}

tests/ci/docker_images_check.py

@@ -8,7 +8,6 @@ import os
 from pr_info import PRInfo
 from github import Github
 import shutil
-from get_robot_token import get_best_robot_token
 
 NAME = "Push to Dockerhub (actions)"
 
@@ -168,11 +167,6 @@ def upload_results(s3_client, pr_number, commit_sha, test_results):
     logging.info("Search result in url %s", url)
     return url
 
-def get_commit(gh, commit_sha):
-    repo = gh.get_repo(os.getenv("GITHUB_REPOSITORY", "ClickHouse/ClickHouse"))
-    commit = repo.get_commit(commit_sha)
-    return commit
-
 if __name__ == "__main__":
     logging.basicConfig(level=logging.INFO)
     repo_path = os.getenv("GITHUB_WORKSPACE", os.path.abspath("../../"))
@@ -223,10 +217,6 @@ if __name__ == "__main__":
 
     url = upload_results(s3_helper, pr_info.number, pr_info.sha, test_results)
 
-    gh = Github(get_best_robot_token())
-    commit = get_commit(gh, pr_info.sha)
-    commit.create_status(context=NAME, description=description, state=status, target_url=url)
-
     with open(os.path.join(temp_path, 'changed_images.json'), 'w') as images_file:
         json.dump(result_images, images_file)
 

tests/ci/pvs_check.py

@@ -9,7 +9,6 @@ from s3_helper import S3Helper
 from pr_info import PRInfo
 import shutil
 import sys
-from get_robot_token import get_best_robot_token
 
 NAME = 'PVS Studio (actions)'
 LICENCE_NAME = 'Free license: ClickHouse, Yandex'
@@ -40,11 +39,6 @@ def _process_txt_report(path):
                 errors.append(':'.join(line.split('\t')[0:2]))
     return warnings, errors
 
-def get_commit(gh, commit_sha):
-    repo = gh.get_repo(os.getenv("GITHUB_REPOSITORY", "ClickHouse/ClickHouse"))
-    commit = repo.get_commit(commit_sha)
-    return commit
-
 def upload_results(s3_client, pr_number, commit_sha, test_results, additional_files):
     s3_path_prefix = str(pr_number) + "/" + commit_sha + "/" + NAME.lower().replace(' ', '_')
     additional_urls = process_logs(s3_client, additional_files, s3_path_prefix)
@@ -84,8 +78,6 @@ if __name__ == "__main__":
     aws_secret_key_id = os.getenv("YANDEX_S3_ACCESS_KEY_ID", "")
     aws_secret_key = os.getenv("YANDEX_S3_ACCESS_SECRET_KEY", "")
 
-    gh = Github(get_best_robot_token())
-
     images_path = os.path.join(temp_path, 'changed_images.json')
     docker_image = 'clickhouse/pvs-test'
     if os.path.exists(images_path):
@@ -138,8 +130,6 @@ if __name__ == "__main__":
         report_url = upload_results(s3_helper, pr_info.number, pr_info.sha, test_results, additional_logs)
 
         print("::notice ::Report url: {}".format(report_url))
-        commit = get_commit(gh, pr_info.sha)
-        commit.create_status(context=NAME, description=description, state=status, target_url=report_url)
     except Exception as ex:
         print("Got an exception", ex)
         sys.exit(1)

tests/ci/run_check.py

@@ -5,8 +5,6 @@ import requests
 from pr_info import PRInfo
 import sys
 import logging
-from github import Github
-from get_robot_token import get_best_robot_token
 
 NAME = 'Run Check (actions)'
 
@@ -114,13 +112,8 @@ if __name__ == "__main__":
 
     pr_info = PRInfo(event, need_orgs=True)
     can_run, description = should_run_checks_for_pr(pr_info)
-    gh = Github(get_best_robot_token())
-    commit = get_commit(gh, pr_info.sha)
-    url = f"https://github.com/ClickHouse/ClickHouse/actions/runs/{os.getenv('GITHUB_RUN_ID')}"
     if not can_run:
         print("::notice ::Cannot run")
-        commit.create_status(context=NAME, description=description, state="failure", target_url=url)
         sys.exit(1)
     else:
         print("::notice ::Can run")
-        commit.create_status(context=NAME, description=description, state="pending", target_url=url)

tests/ci/style_check.py

@@ -10,7 +10,6 @@ from s3_helper import S3Helper
 import time
 import json
 from pr_info import PRInfo
-from get_robot_token import get_best_robot_token
 
 NAME = "Style Check (actions)"
 
@@ -79,12 +78,6 @@ def upload_results(s3_client, pr_number, commit_sha, test_results, additional_fi
     logging.info("Search result in url %s", url)
     return url
 
-
-def get_commit(gh, commit_sha):
-    repo = gh.get_repo(os.getenv("GITHUB_REPOSITORY", "ClickHouse/ClickHouse"))
-    commit = repo.get_commit(commit_sha)
-    return commit
-
 def update_check_with_curl(check_id):
     cmd_template = ("curl -v --request PATCH --url https://api.github.com/repos/ClickHouse/ClickHouse/check-runs/{} "
            "--header 'authorization: Bearer {}' "
@@ -109,8 +102,6 @@ if __name__ == "__main__":
     aws_secret_key_id = os.getenv("YANDEX_S3_ACCESS_KEY_ID", "")
     aws_secret_key = os.getenv("YANDEX_S3_ACCESS_SECRET_KEY", "")
 
-    gh = Github(get_best_robot_token())
-
     images_path = os.path.join(temp_path, 'changed_images.json')
     docker_image = 'clickhouse/style-test'
     if os.path.exists(images_path):
@@ -141,5 +132,3 @@ if __name__ == "__main__":
     state, description, test_results, additional_files = process_result(temp_path)
     report_url = upload_results(s3_helper, pr_info.number, pr_info.sha, test_results, additional_files)
     print("::notice ::Report url: {}".format(report_url))
-    commit = get_commit(gh, pr_info.sha)
-    commit.create_status(context=NAME, description=description, state=state, target_url=report_url)