Skip to content

HADOOP-19887. Improve GHA website workflow#8482

Merged
pan3793 merged 1 commit into
apache:trunkfrom
pan3793:HADOOP-19887
May 8, 2026
Merged

HADOOP-19887. Improve GHA website workflow#8482
pan3793 merged 1 commit into
apache:trunkfrom
pan3793:HADOOP-19887

Conversation

@pan3793
Copy link
Copy Markdown
Member

@pan3793 pan3793 commented May 8, 2026

Description of PR

  • Upgrade actions to the latest version
  • Manage Maven cache via actions/setup-java
  • Switch mvn to mvnw, so it always picks the requested version of maven, instead of using mvn shipped in OS or installed by actions/setup-java.

How was this patch tested?

I manually run those commands locally to ensure they work as expected.

For code changes:

  • Does the title or this PR starts with the corresponding JIRA issue id (HADOOP-19887)?
  • 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

No AI usage.

@pan3793
Copy link
Copy Markdown
Member Author

pan3793 commented May 8, 2026

@slfan1989 could you please take a quick look? we need this to recover the website deployed, currently it picks an obsolete version of mvn installed by actions/setup-java, which violates the org.apache.maven.enforcer.rules.version.RequireMavenVersion
https://github.com/apache/hadoop/actions/runs/25535467127/job/74950312789

@slfan1989
Copy link
Copy Markdown
Contributor

@pan3793 Thanks for the contribution! LGTM.

@pan3793 pan3793 merged commit 4bbf9ee into apache:trunk May 8, 2026
4 of 6 checks passed
@pan3793
Copy link
Copy Markdown
Member Author

pan3793 commented May 8, 2026

thanks, merged to trunk (lint check pass, no need to wait for building pipeline as it does not touch the code)

@hadoop-yetus
Copy link
Copy Markdown

💔 -1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 18m 54s Docker mode activated.
-1 ❌ patch 0m 5s #8482 does not apply to trunk. Rebase required? Wrong Branch? See https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute for help.
Subsystem Report/Notes
Docker ClientAPI=1.54 ServerAPI=1.54 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8482/1/artifact/out/Dockerfile
GITHUB PR #8482
Console output https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-8482/1/console
versions git=2.43.0
Powered by Apache Yetus 0.14.1 https://yetus.apache.org

This message was automatically generated.

check-latest: false
- name: Build Hadoop maven plugins
run: cd hadoop-maven-plugins && mvn --batch-mode install
run: ./mvnw --batch-mode -f hadoop-maven-plugins/pom.xml install
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

this unfortunately does not work, and it was not caught in my local test as it has cache in ~/.m2, I pushed a hotfix to restore it b3cc7d0

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.

3 participants