Skip to content

HADOOP-19814: Update GitHub actions to latest versions allowed by ASF infra.#8434

Closed
cnauroth wants to merge 1 commit intoapache:docker-hadoop-runner-jdk17-u2404from
cnauroth:HADOOP-19814-docker-hadoop-runner-jdk17-u2404
Closed

HADOOP-19814: Update GitHub actions to latest versions allowed by ASF infra.#8434
cnauroth wants to merge 1 commit intoapache:docker-hadoop-runner-jdk17-u2404from
cnauroth:HADOOP-19814-docker-hadoop-runner-jdk17-u2404

Conversation

@cnauroth
Copy link
Copy Markdown
Contributor

Description of PR

On branch docker-hadoop-runner-jdk17-u2404, update GitHub action commit hashes to latest allowed by ASF infra:

https://github.com/apache/infrastructure-actions/blob/main/actions.yml

How was this patch tested?

Awaiting GitHub workflow run.

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
  • Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
  • If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
  • If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

AI Tooling

If an AI tool was used:

Copy link
Copy Markdown
Member

@ayushtkn ayushtkn left a comment

Choose a reason for hiding this comment

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

LGTM

- name: Generate image ID
id: meta
uses: docker/metadata-action@8e5442c4ef9f78752691e2d8f8d19755c6f78e81
uses: docker/metadata-action@030e881283bb7a6894de51c315a6bfe6a94e05cf
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

mind adding a comment to mention the tag (human-readable)? e.g.,

Suggested change
uses: docker/metadata-action@030e881283bb7a6894de51c315a6bfe6a94e05cf
uses: docker/metadata-action@030e881283bb7a6894de51c315a6bfe6a94e05cf # v6.0.0

Copy link
Copy Markdown
Member

@pan3793 pan3793 left a comment

Choose a reason for hiding this comment

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

just a suggestion, no blocker

cnauroth added a commit that referenced this pull request Apr 17, 2026
… infra.

Closes #8434

Signed-off-by: Shilun Fan <slfan1989@apache.org>
Signed-off-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Ayush Saxena <ayushsaxena@apache.org>
@cnauroth
Copy link
Copy Markdown
Contributor Author

Thanks for the reviews everyone. I committed this, with Cheng's suggestion to add comments for the tags.

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.

4 participants