Skip to content

Upgrade github action versions - #38202

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

Abacn merged 6 commits into
apache:masterfrom
derrickaw:20260415_upgradeGithubactions

Conversation

@derrickaw

@derrickaw derrickaw commented Apr 15, 2026 •

Copy link
Copy Markdown
Collaborator
  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

Abacn commented Apr 17, 2026

Copy link
Copy Markdown
Contributor

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

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

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).

@Abacn Abacn left a comment

Copy link
Copy Markdown
Contributor

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
aIbrahiim pushed a commit to aIbrahiim/beam that referenced this pull request May 12, 2026
* rebase conflict

* address merge conflicts

* add wget install

* fix conflict

* remove accident rebased file

* remove other accidental rebased files
ash6898 pushed a commit to ash6898/beam that referenced this pull request Jun 29, 2026
* rebase conflict

* address merge conflicts

* add wget install

* fix conflict

* remove accident rebased file

* remove other accidental rebased files
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