mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-11-22 15:42:02 +00:00
Merge pull request #32051 from ClickHouse/cancel_checks_more_agressively
Cancel redundant checks with lambda
This commit is contained in:
commit
1977ab6a8e
3
.github/workflows/cancel.yml
vendored
3
.github/workflows/cancel.yml
vendored
@ -1,7 +1,7 @@
|
||||
name: Cancel
|
||||
on: # yamllint disable-line rule:truthy
|
||||
workflow_run:
|
||||
workflows: ["CIGithubActions", "ReleaseCI"]
|
||||
workflows: ["CIGithubActions", "ReleaseCI", "DocsCheck", "BackportPR"]
|
||||
types:
|
||||
- requested
|
||||
jobs:
|
||||
@ -10,4 +10,5 @@ jobs:
|
||||
steps:
|
||||
- uses: styfle/cancel-workflow-action@0.9.1
|
||||
with:
|
||||
all_but_latest: true
|
||||
workflow_id: ${{ github.event.workflow.id }}
|
||||
|
13
tests/ci/cancel_workflow_lambda/Dockerfile
Normal file
13
tests/ci/cancel_workflow_lambda/Dockerfile
Normal file
@ -0,0 +1,13 @@
|
||||
FROM public.ecr.aws/lambda/python:3.9
|
||||
|
||||
# Copy function code
|
||||
COPY app.py ${LAMBDA_TASK_ROOT}
|
||||
|
||||
# Install the function's dependencies using file requirements.txt
|
||||
# from your project folder.
|
||||
|
||||
COPY requirements.txt .
|
||||
RUN pip3 install -r requirements.txt --target "${LAMBDA_TASK_ROOT}"
|
||||
|
||||
# Set the CMD to your handler (could also be done as a parameter override outside of the Dockerfile)
|
||||
CMD [ "app.handler" ]
|
127
tests/ci/cancel_workflow_lambda/app.py
Normal file
127
tests/ci/cancel_workflow_lambda/app.py
Normal file
@ -0,0 +1,127 @@
|
||||
#!/usr/bin/env python3
|
||||
|
||||
import json
|
||||
import time
|
||||
import jwt
|
||||
|
||||
import requests
|
||||
import boto3
|
||||
|
||||
# https://docs.github.com/en/rest/reference/actions#cancel-a-workflow-run
|
||||
#
|
||||
API_URL = 'https://api.github.com/repos/ClickHouse/ClickHouse'
|
||||
|
||||
MAX_RETRY = 5
|
||||
|
||||
def get_installation_id(jwt_token):
|
||||
headers = {
|
||||
"Authorization": f"Bearer {jwt_token}",
|
||||
"Accept": "application/vnd.github.v3+json",
|
||||
}
|
||||
response = requests.get("https://api.github.com/app/installations", headers=headers)
|
||||
response.raise_for_status()
|
||||
data = response.json()
|
||||
return data[0]['id']
|
||||
|
||||
def get_access_token(jwt_token, installation_id):
|
||||
headers = {
|
||||
"Authorization": f"Bearer {jwt_token}",
|
||||
"Accept": "application/vnd.github.v3+json",
|
||||
}
|
||||
response = requests.post(f"https://api.github.com/app/installations/{installation_id}/access_tokens", headers=headers)
|
||||
response.raise_for_status()
|
||||
data = response.json()
|
||||
return data['token']
|
||||
|
||||
def get_key_and_app_from_aws():
|
||||
secret_name = "clickhouse_github_secret_key"
|
||||
session = boto3.session.Session()
|
||||
client = session.client(
|
||||
service_name='secretsmanager',
|
||||
)
|
||||
get_secret_value_response = client.get_secret_value(
|
||||
SecretId=secret_name
|
||||
)
|
||||
data = json.loads(get_secret_value_response['SecretString'])
|
||||
return data['clickhouse-app-key'], int(data['clickhouse-app-id'])
|
||||
|
||||
def get_token_from_aws():
|
||||
private_key, app_id = get_key_and_app_from_aws()
|
||||
payload = {
|
||||
"iat": int(time.time()) - 60,
|
||||
"exp": int(time.time()) + (10 * 60),
|
||||
"iss": app_id,
|
||||
}
|
||||
|
||||
encoded_jwt = jwt.encode(payload, private_key, algorithm="RS256")
|
||||
installation_id = get_installation_id(encoded_jwt)
|
||||
return get_access_token(encoded_jwt, installation_id)
|
||||
|
||||
def _exec_get_with_retry(url):
|
||||
for i in range(MAX_RETRY):
|
||||
try:
|
||||
response = requests.get(url)
|
||||
response.raise_for_status()
|
||||
return response.json()
|
||||
except Exception as ex:
|
||||
print("Got exception executing request", ex)
|
||||
time.sleep(i + 1)
|
||||
|
||||
raise Exception("Cannot execute GET request with retries")
|
||||
|
||||
|
||||
def get_workflows_cancel_urls_for_pull_request(pull_request_event):
|
||||
head_branch = pull_request_event['head']['ref']
|
||||
print("PR", pull_request_event['number'], "has head ref", head_branch)
|
||||
workflows = _exec_get_with_retry(API_URL + f"/actions/runs?branch={head_branch}")
|
||||
workflows_urls_to_cancel = set([])
|
||||
for workflow in workflows['workflow_runs']:
|
||||
if workflow['status'] != 'completed':
|
||||
print("Workflow", workflow['url'], "not finished, going to be cancelled")
|
||||
workflows_urls_to_cancel.add(workflow['cancel_url'])
|
||||
else:
|
||||
print("Workflow", workflow['url'], "already finished, will not try to cancel")
|
||||
|
||||
return workflows_urls_to_cancel
|
||||
|
||||
def _exec_post_with_retry(url, token):
|
||||
headers = {
|
||||
"Authorization": f"token {token}"
|
||||
}
|
||||
for i in range(MAX_RETRY):
|
||||
try:
|
||||
response = requests.post(url, headers=headers)
|
||||
response.raise_for_status()
|
||||
return response.json()
|
||||
except Exception as ex:
|
||||
print("Got exception executing request", ex)
|
||||
time.sleep(i + 1)
|
||||
|
||||
raise Exception("Cannot execute POST request with retry")
|
||||
|
||||
def cancel_workflows(urls_to_cancel, token):
|
||||
for url in urls_to_cancel:
|
||||
print("Cancelling workflow using url", url)
|
||||
_exec_post_with_retry(url, token)
|
||||
print("Workflow cancelled")
|
||||
|
||||
def main(event):
|
||||
token = get_token_from_aws()
|
||||
event_data = json.loads(event['body'])
|
||||
|
||||
print("Got event for PR", event_data['number'])
|
||||
action = event_data['action']
|
||||
print("Got action", event_data['action'])
|
||||
pull_request = event_data['pull_request']
|
||||
labels = { l['name'] for l in pull_request['labels'] }
|
||||
print("PR has labels", labels)
|
||||
if action == 'closed' or 'do not test' in labels:
|
||||
print("PR merged/closed or manually labeled 'do not test' will kill workflows")
|
||||
workflows_to_cancel = get_workflows_cancel_urls_for_pull_request(pull_request)
|
||||
print(f"Found {len(workflows_to_cancel)} workflows to cancel")
|
||||
cancel_workflows(workflows_to_cancel, token)
|
||||
else:
|
||||
print("Nothing to do")
|
||||
|
||||
def handler(event, _):
|
||||
main(event)
|
3
tests/ci/cancel_workflow_lambda/requirements.txt
Normal file
3
tests/ci/cancel_workflow_lambda/requirements.txt
Normal file
@ -0,0 +1,3 @@
|
||||
requests
|
||||
PyJWT
|
||||
cryptography
|
Loading…
Reference in New Issue
Block a user