Bitmessage/buildbot_multibuild
6
0
Fork 5
Code Issues 4 Pull Requests Projects Releases Wiki Activity

add webhook password #10

Merged
PeterSurda merged 1 commits from cis-muzahid/buildbot_multibuild:github_hook into master 2022-03-16 08:16:44 +01:00
Conversation 0 Commits 1 Files Changed 1 +13 -5
1 changed files with 13 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 83444728f3 - Show all commits

18
lib/worker_multibuild.py
View File

@ -1,4 +1,5 @@
from os import listdir, walk import os
from os import listdir
from os.path import exists, isfile, join, islink, isdir from os.path import exists, isfile, join, islink, isdir
import requests import requests
import re import re
@ -8,10 +9,6 @@ request_data = {
"project": "testproject", "project": "testproject",
"comments": "testcomment", "comments": "testcomment",
} }
request_headers = {
"Content-Type": "application/x-www-form-urlencoded",
"Accept": "text/plain",
}
ty = "/change_hook/base" ty = "/change_hook/base"
path =".buildbot" path =".buildbot"
@ -54,6 +51,11 @@ ENTRYPOINT /usr/local/bin/buildbot_entrypoint.sh "$BUILDMASTER" "$WORKERNAME" "$
""" """
def get_secret():
with open(path.join(os.environ['HOME'], "multibuild_parent_key.key"),'r') as f:
PeterSurda marked this conversation as resolved Outdated
PeterSurda commented 2022-03-15 14:50:15 +01:00
Outdated
Review
Copy Link

filename doesn't match the filename used in StringDownload

filename doesn't match the filename used in `StringDownload`
data = f.read()
return data
def list_jobs(directory=".buildbot"): def list_jobs(directory=".buildbot"):
""" """
list jobs found in a directory list jobs found in a directory
@ -101,6 +103,11 @@ def trigger_child_hooks(buildbotUrl: str, repository, branch, directory=".buildb
request_url = buildbotUrl + ty request_url = buildbotUrl + ty
PeterSurda marked this conversation as resolved Outdated
PeterSurda commented 2022-03-14 08:05:40 +01:00
Outdated
Review
Copy Link

'X-Multibuild-Trigger'

`'X-Multibuild-Trigger'`
PeterSurda commented 2022-03-15 07:52:52 +01:00
Outdated
Review
Copy Link

would be better if " and ' were used consistently.

would be better if `"` and `'` were used consistently.
👍 1
# List all jobs in the directory # List all jobs in the directory
jobs = list_jobs(directory) jobs = list_jobs(directory)
request_headers = {
"Content-Type": "application/x-www-form-urlencoded",
"X-Multibuild-Trigger": get_secret(),
"Accept": "text/plain",
}
# Check if build.sh or test.sh exists in each of the jobs # Check if build.sh or test.sh exists in each of the jobs
for job in jobs: for job in jobs:
@ -122,4 +129,5 @@ def trigger_child_hooks(buildbotUrl: str, repository, branch, directory=".buildb
"branch": branch, "branch": branch,
"jobname": job, "jobname": job,
} }
requests.post(request_url, headers=request_headers, data=request_data) requests.post(request_url, headers=request_headers, data=request_data)