-
Notifications
You must be signed in to change notification settings - Fork 183
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
Use $GITHUB_EVENT_PATH variable instead of inline file writing #9134
Conversation
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
fb9ab78
to
692203f
Compare
The following pipelines have been queued for testing: |
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.
…1412) Sync .github/workflows directory with azure-sdk-tools for PR Azure/azure-sdk-tools#9134 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Ben Broderick Phillips <[email protected]>
${{ toJson(github.event) }} | ||
EOF | ||
github-event-processor ${{ github.event_name }} payload.json CloseAddressedIssues | ||
github-event-processor ${{ github.event_name }} ${{ github.event_path }} |
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.
Looks like we all missed the fact you deleted CloseAddressedIssues
which is causing failures like https://github.com/Azure/azure-sdk/actions/runs/11358965282/job/31594328835
Reduce github event processor logging
FYI @JimSuplizio I didn't know until now that you can grab the webhook payload from disk (thanks to @hallipr). This saves us a lot of log storage space, as before I was putting the whole payload inline to the bash command (to work around env var max size issues).