Skip to content

HDDS-14833. Bump GitHub action versions#23

Merged
adoroszlai merged 7 commits intoapache:mainfrom
adoroszlai:HDDS-14833
Apr 2, 2026
Merged

HDDS-14833. Bump GitHub action versions#23
adoroszlai merged 7 commits intoapache:mainfrom
adoroszlai:HDDS-14833

Conversation

@adoroszlai
Copy link
Copy Markdown
Contributor

@adoroszlai adoroszlai commented Mar 25, 2026

What changes were proposed in this pull request?

Node.js 20 is deprecated in GitHub runners. Bump GitHub actions versions to the latest ones for Node.js 24 support (where available).

Latest chart-releaser-action is more than a year old, so it may still have references to other actions using Node.js 20.

EDIT (to reflect current state of the PR): Previously 3rd-party actions were used via git submodules due to problems with Apache Infra's whitelisting. Since the whitelist was fixed/updated, we can use the actions directly.

https://issues.apache.org/jira/browse/HDDS-14833

How was this patch tested?

CI (for test workflow):
https://github.com/apache/ozone-helm-charts/actions/runs/23684819360

@adoroszlai adoroszlai self-assigned this Mar 25, 2026
@adoroszlai
Copy link
Copy Markdown
Contributor Author

@dnskr please take a look

@adoroszlai adoroszlai marked this pull request as draft March 25, 2026 08:53
@adoroszlai
Copy link
Copy Markdown
Contributor Author

@adoroszlai adoroszlai marked this pull request as ready for review March 28, 2026 12:06
@adoroszlai adoroszlai requested a review from spacemonkd March 30, 2026 13:39
Copy link
Copy Markdown

@spacemonkd spacemonkd left a comment

Choose a reason for hiding this comment

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

Just one question, apart from that it looks good to me.
Thanks @adoroszlai for working on this

@adoroszlai adoroszlai requested a review from spacemonkd April 1, 2026 09:10
Copy link
Copy Markdown

@spacemonkd spacemonkd left a comment

Choose a reason for hiding this comment

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

Thanks @adoroszlai for clearing the confusion.
This looks good to me, +1

@adoroszlai adoroszlai merged commit 745322d into apache:main Apr 2, 2026
1 check passed
@adoroszlai adoroszlai deleted the HDDS-14833 branch April 2, 2026 17:07
@adoroszlai
Copy link
Copy Markdown
Contributor Author

Thanks @dnskr, @spacemonkd for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants