-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Enable remote gradle cache and build scan for GitHub Actions workflows #28539
Conversation
publish gradle build scans publish gradle build scans
Checks are failing. Will not request review until checks are succeeding. If you'd like to override that behavior, comment |
Run RAT PreCommit |
Assigning reviewers. If you would like to opt out of this review, comment R: @damccorm for label build. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
publish gradle build scans
76bf494
to
3f72c1e
Compare
Run RAT PreCommit |
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.
Thanks!
seeing many errors in post step of /.github/actions/setup-action : Index out of range . sample run: https://github.com/apache/beam/actions/runs/6250219560/job/16969848231?pr=28557 actions/runner#2816 looks similar . Is it caused by this change? |
from reading actions/runner#2816 , it might be caused when a cached version of a step has a different # of stages. recommendation is to use the same version for a checkout step. I am seeing we using checkout@v3 in several suites, but v4 in all other suites - could this be an issue?
|
@tvalentyn I just checked and i see failures in workflows with v3 and v4 so i dont think that could be the cause. The issue im looking at is that checkout pulls in the current master. Then the setup action pulls in the merge comment and that causes a difference in number of steps. I have checked the PR and no changes to actions were made so im not sure from where the issue comes from |
fixes #28378
The following changes have been made:
setup-environment-action/action.yml
action to publish build scan for all GitHub Actions workflows by adding the mandatory stepgradle/gradle-build-action@v2
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
addresses #123
), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, commentfixes #<ISSUE NUMBER>
instead.CHANGES.md
with noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.