-
Notifications
You must be signed in to change notification settings - Fork 127
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.
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
feat(.github): run QUIC Interop Runner #1682
Merged
Merged
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
c840abf
feat(.github): run QUIC Interop Runner
mxinden 3392440
Do not always construct comment
mxinden d2377aa
Test for all pull requests to main
mxinden 69a1bca
Don't build arm64 on pull requests
mxinden b490228
Use actions/download-artifact
mxinden b055192
Provide client, server and test argument
mxinden b9191aa
Fix action args
mxinden c10ce08
Copy action into neqo repo
mxinden 6033c63
Merge branch 'main' of https://github.com/mozilla/neqo into qns-ci
mxinden a9d8112
checkout
mxinden 08d91ea
yml
mxinden 36e629c
introduce folder
mxinden a72e3a6
load image
mxinden 33ce044
Only run interop on pull requests
mxinden 626dd04
Merge branch 'main' of https://github.com/mozilla/neqo into qns-ci
mxinden 6a82788
Remove pull_request path filter
mxinden 0be3c49
Trigger CI
mxinden 07fe740
Cache Python deps
mxinden d2e3efa
Trigger CI to test python cache
mxinden File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,56 @@ | ||
# Post test results as pull request comment. | ||
# | ||
# This is done as a separate workflow as it requires write permissions. The | ||
# tests itself might run off of a fork, i.e. an untrusted environment and should | ||
# thus not be granted write permissions. | ||
|
||
name: Comment on the pull request | ||
|
||
on: | ||
workflow_run: | ||
workflows: ["QUIC Network Simulator"] | ||
types: | ||
- completed | ||
larseggert marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
jobs: | ||
upload: | ||
permissions: | ||
pull-requests: write | ||
runs-on: ubuntu-latest | ||
if: > | ||
github.event.workflow_run.event == 'pull_request' | ||
steps: | ||
- name: Download comment-data | ||
uses: dawidd6/[email protected] | ||
larseggert marked this conversation as resolved.
Show resolved
Hide resolved
|
||
with: | ||
github_token: ${{ secrets.GITHUB_TOKEN }} | ||
workflow: main.yml | ||
run_id: ${{ github.event.workflow_run.id }} | ||
name: comment-data | ||
|
||
- name: Format GitHub comment | ||
run: | | ||
pwd | ||
ls -la | ||
echo '[**QUIC Interop Runner**](https://github.com/quic-interop/quic-interop-runner)' >> comment | ||
echo '' >> comment | ||
echo '```' >> comment | ||
cat summary >> comment | ||
echo '```' >> comment | ||
echo '' >> comment | ||
echo 'Download artifacts [here](' >> comment | ||
cat logs-url >> comment | ||
echo ').' >> comment | ||
if: always() | ||
shell: bash | ||
|
||
- name: Read PR Number | ||
id: pr-number | ||
run: echo "::set-output name=number::$(cat pr-number)" | ||
shell: bash | ||
|
||
- name: Comment PR | ||
uses: thollander/actions-comment-pull-request@v2 | ||
with: | ||
filePath: comment | ||
pr_number: ${{ steps.pr-number.outputs.number }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A comment with the test results on each pull requests will only appear once this workflow is in
main
.