Skip to content

HBASE-30133 opt-in yetus prerelease testing in GHA precommit workflows#8182

Merged
ndimiduk merged 1 commit intoapache:masterfrom
ndimiduk:30133-yetus-prerelease
May 4, 2026
Merged

HBASE-30133 opt-in yetus prerelease testing in GHA precommit workflows#8182
ndimiduk merged 1 commit intoapache:masterfrom
ndimiduk:30133-yetus-prerelease

Conversation

@ndimiduk
Copy link
Copy Markdown
Member

@ndimiduk ndimiduk commented May 1, 2026

Adds a test-yetus-prerelease PR label that swaps the pinned signed yetus tarball for a freshly-built apache/yetus@HEAD artifact in the three yetus precommit workflows. Lets us validate an in-flight yetus revision against a real HBase PR before the project bumps the pinned YETUS_VERSION; mirrors the dead USE_YETUS_PRERELEASE parameter from the retired dev-support/Jenkinsfile. Resolved yetus SHA is logged to the step summary for reproducibility. Default path is unchanged.

@ndimiduk ndimiduk added the test-yetus-prerelease Test the precommit run against an unreleased apache/yetus revision. See HBASE-30133. label May 1, 2026
Adds a `test-yetus-prerelease` PR label that swaps the pinned signed
yetus tarball for a freshly-built `apache/yetus@HEAD` artifact in the
three yetus precommit workflows. Lets us validate an in-flight yetus
revision against a real HBase PR before the project bumps the pinned
YETUS_VERSION; mirrors the dead `USE_YETUS_PRERELEASE` parameter from
the retired `dev-support/Jenkinsfile`. Resolved yetus SHA is logged to
the step summary for reproducibility. Default path is unchanged.

apache/yetus master enforces RequireJavaVersion [21,) in its top-level
pom, so the prerelease branch installs JDK 21 on the host runner before
the source build. JAVA_HOME stays pointed at JDK 21 afterward, but
jenkins_precommit_github_yetus.sh unsets JAVA_HOME before invoking
yetus and yetus runs everything in docker via SET_JAVA_HOME, so the
host JDK doesn't matter to anything downstream.
@ndimiduk ndimiduk force-pushed the 30133-yetus-prerelease branch from 5997c9f to 31bef91 Compare May 1, 2026 12:51
@ndimiduk ndimiduk marked this pull request as ready for review May 1, 2026 13:54
@ndimiduk ndimiduk requested review from Apache9 and charlesconnell May 1, 2026 13:54
@ndimiduk
Copy link
Copy Markdown
Member Author

ndimiduk commented May 1, 2026

FYI @aw-was-here

@ndimiduk ndimiduk merged commit 9119caf into apache:master May 4, 2026
8 checks passed
ndimiduk added a commit that referenced this pull request May 4, 2026
#8182)

Signed-off-by: Charles Connell <cconnell@apache.org>
(cherry picked from commit 9119caf)
ndimiduk added a commit that referenced this pull request May 4, 2026
#8182)

Signed-off-by: Charles Connell <cconnell@apache.org>
(cherry picked from commit 9119caf)
ndimiduk added a commit that referenced this pull request May 4, 2026
#8182)

Signed-off-by: Charles Connell <cconnell@apache.org>
(cherry picked from commit 9119caf)
ndimiduk added a commit that referenced this pull request May 4, 2026
#8182)

Signed-off-by: Charles Connell <cconnell@apache.org>
(cherry picked from commit 9119caf)
@PDavid
Copy link
Copy Markdown
Contributor

PDavid commented May 5, 2026

Very cool, many thanks! 👍

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

Labels

test-yetus-prerelease Test the precommit run against an unreleased apache/yetus revision. See HBASE-30133.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants