Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move from Buildkite to GitHub Actions #1115

Closed
wants to merge 60 commits into from
+166 −10
Closed

Move from Buildkite to GitHub Actions #1115

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
60 commits
Select commit Hold shift + click to select a range
469707c
First attempt at a GHA pipeline
Aug 11, 2021
74bef07
Provide dummy "run"; parameterize homeserver impl
Aug 11, 2021
4c86c1a
Try to force GHA to trigger in my fork
Aug 11, 2021
f68bbe4
correct yaml nesting; fix image name
Aug 11, 2021
6f8197f
Flesh out test cases and setup env
Aug 11, 2021
c4afc4f
GHA contexts require single quotes around strings
Aug 11, 2021
40e9979
Names are redundant: don't use `yes`
Aug 11, 2021
ed0d9d7
Maybe setting working dir will help?
Aug 11, 2021
3142295
Mount . as /sytest
Aug 11, 2021
f93a663
`.` was too short. Workspace as full path?
Aug 11, 2021
78814e0
Guess to try and fix the logs path
Aug 11, 2021
47ae955
single quotes :(
Aug 11, 2021
f9ac100
bash -xe for debug info
Aug 11, 2021
d87a25e
more -xe
Aug 11, 2021
1e98e8d
More exploratory debugging
Aug 11, 2021
4a66f6d
Try an explicit cat?
Aug 11, 2021
d87f6a8
Comment out to make the pipeline simpler
Aug 11, 2021
7436223
Where are we doing the tarring in?
Aug 11, 2021
1e8f5ad
synapse_sytest: use SYNAPSE_SOURCE throughout
Aug 11, 2021
04329ae
Can we get sytest running?
Aug 11, 2021
06f684b
stupid debug
Aug 11, 2021
bc453ba
desperation printf debugging
Aug 11, 2021
0567858
echo with quotes!?!?!
Aug 11, 2021
1171129
Remove bootstrap debug for now; mount /src
Aug 11, 2021
b8a7175
I think it'll be easier to concentrate on synapse
Aug 11, 2021
8ff4b2d
Again
Aug 11, 2021
5ed4665
Revert a bunch of incorrect changes; mkdir -p
Aug 11, 2021
2302479
Tweak pipeline
Aug 11, 2021
5aab2fe
Tweak checkout
Aug 11, 2021
ec135d6
Explicit paths
Aug 11, 2021
2f0c6d2
Fix blacklist path
Aug 11, 2021
cfb8729
Try to get sytest logs uploaded as artefacts
Aug 11, 2021
b78fad2
update TODOs; try more synapse testing
Aug 11, 2021
0862c20
Try to checkout the corresponding synapse branch
Aug 12, 2021
45cbb1a
Use rest API to avoid checkouts action retrying
Aug 12, 2021
ccd4e89
Need API token to make an API request hurrrr
Aug 12, 2021
7eafc11
Log annotation already handled by tap_to_gha.pl
Aug 12, 2021
66f1ac5
ci -> .ci
Aug 12, 2021
d2e27df
Remove backticks
Aug 12, 2021
58aabb9
dendrite
Aug 12, 2021
c74afbd
Fix docker volume cfg to mount dendrite
Aug 12, 2021
13f9000
I got lost in the yaml
Aug 12, 2021
bc2b9ce
Try to reduce annotation spam
Aug 12, 2021
aec7fa6
Okay, that didn't work because no curl
Aug 12, 2021
1a6a46c
Missing quote
Aug 12, 2021
cfda89b
Fix branches and apply to dendrite
Aug 12, 2021
0dc6a18
Oh, workers was an additional blacklist, not a replacement
Aug 12, 2021
03a59bf
Don't special case the github-actions branch
Aug 12, 2021
fd51fc4
Fix "fetch dendrite branch" step name
DMRobertson Aug 13, 2021
2984f8f
Add explanatory comments
Aug 13, 2021
9e4a8f3
Try to get friendly job names
Aug 13, 2021
b5350ec
TEMP: hack in github-actions branch again
Aug 13, 2021
dd7b896
Fiddle with labels; include, `&& 1` for dendrite
Aug 13, 2021
06e00d5
Mark annotate.md as bk-specific
Aug 13, 2021
8589838
Shorten synapse labels
Aug 13, 2021
ea25d7a
More label faffing
Aug 13, 2021
7a9ca4b
Use BUILDKITE_LABEL for annotations again
Aug 13, 2021
0994d0c
Fix and improve handling of env vars
Aug 18, 2021
ce29fb2
More descriptive blacklist step name
Aug 18, 2021
5692410
We do actually want CI to pass a branch
Aug 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions scripts/dendrite_sytest.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ echo >&2 "--- Copying assets"
# Copy out the logs
rsync -r --ignore-missing-args --min-size=1B -av /work/server-0 /work/server-1 /logs --include "*/" --include="*.log.*" --include="*.log" --exclude="*"

# Generate annotate.md This is Buildkite-specific.
DMRobertson marked this conversation as resolved.
Show resolved Hide resolved
if [ $TEST_STATUS -ne 0 ]; then
# Build the annotation
perl /sytest/scripts/format_tap.pl /logs/results.tap "$GITHUB_SHA" >/logs/annotate.md
DMRobertson marked this conversation as resolved.
Show resolved Hide resolved
Expand Down
1 change: 1 addition & 0 deletions scripts/synapse_sytest.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,6 +201,7 @@ rsync --ignore-missing-args --min-size=1B -av /work/server-0 /work/server-1 /log
#export TOP=/src
#/venv/bin/coverage combine

# Generate annotate.md This is Buildkite-specific.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
# Generate annotate.md This is Buildkite-specific.
# Generate annotate.md. This is Buildkite-specific.

if [ $TEST_STATUS -ne 0 ]; then
# Build the annotation
perl /sytest/scripts/format_tap.pl /logs/results.tap "$GITHUB_SHA" >/logs/annotate.md
DMRobertson marked this conversation as resolved.
Show resolved Hide resolved
Expand Down