Skip to content

Upgrade github action versions#38202

Merged
Abacn merged 6 commits intoapache:masterfrom
derrickaw:20260415_upgradeGithubactions
May 4, 2026
Merged

Upgrade github action versions#38202
Abacn merged 6 commits intoapache:masterfrom
derrickaw:20260415_upgradeGithubactions

Conversation

@derrickaw
Copy link
Copy Markdown
Collaborator

@derrickaw derrickaw commented Apr 15, 2026

  1. upgrade upload_artifact version to v7
  2. upgrade download_artifact version to v7
  3. upgrade checkout version to v6
  4. Addresses this warning:
    Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/upload-artifact@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

  • Mention the appropriate issue in your description (for example: 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, comment fixes #<ISSUE NUMBER> instead.
  • Update CHANGES.md with noteworthy changes.
  • If this contribution is large, please file an Apache Individual Contributor License Agreement.

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)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions github-actions Bot added the build label Apr 15, 2026
@derrickaw
Copy link
Copy Markdown
Collaborator Author

Hi @Abacn, do the runner containers still run on node 20? or have they been upgraded to 24? Thanks.

@Abacn
Copy link
Copy Markdown
Contributor

Abacn commented Apr 17, 2026

Thanks for doing this. There are two caveats

  • .github/workflow changes does not get honored in PR run, and one need to trigger these tests on a branch under apache/beam with workflow_dispatch trigger to test them

  • Apache Infra has an allowlist of github actions (though not sure github's official actions is in scope) therefore we still need to test them before merge

I would suggest we merge this after current release done, as the release workflows also using GHA definition at HEAD

@derrickaw derrickaw force-pushed the 20260415_upgradeGithubactions branch from bd47ee0 to fbb1f12 Compare April 29, 2026 16:13
@derrickaw
Copy link
Copy Markdown
Collaborator Author

  • .github/workflow changes does not get honored in PR run, and one need to trigger these tests on a branch under apache/beam with workflow_dispatch trigger to test them

Hi @Abacn, do I need to check every workflow? or just spot check say 10? Thanks

@derrickaw derrickaw force-pushed the 20260415_upgradeGithubactions branch from fbb1f12 to 812e30b Compare May 4, 2026 15:38
Comment thread .github/workflows/beam_PreCommit_SQL_Java8.yml Outdated
@derrickaw derrickaw marked this pull request as ready for review May 4, 2026 16:50
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 4, 2026

Assigning reviewers:

R: @Abacn for label build.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

Copy link
Copy Markdown
Contributor

@Abacn Abacn left a comment

Choose a reason for hiding this comment

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

Thank you!

@Abacn Abacn merged commit 4baf3ca into apache:master May 4, 2026
6 checks passed
@derrickaw derrickaw deleted the 20260415_upgradeGithubactions branch May 4, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants