Skip to content

Allow ATH usage on Java 25#2245

Merged
timja merged 2 commits intojenkinsci:masterfrom
NotMyFault:jdk25
Jan 1, 2026
Merged

Allow ATH usage on Java 25#2245
timja merged 2 commits intojenkinsci:masterfrom
NotMyFault:jdk25

Conversation

@NotMyFault
Copy link
Member

@NotMyFault NotMyFault commented Nov 10, 2025

The change proposed tests on Java 25. The one failure is known and present on the master branch too.

After the merge of jenkinsci/jenkins#26018, I'll remove the Java 17 branch from the axes.

Signed-off-by: Alexander Brandes <mc.cache@web.de>
Signed-off-by: Alexander Brandes <mc.cache@web.de>
@NotMyFault NotMyFault marked this pull request as ready for review December 31, 2025 11:34
@NotMyFault NotMyFault requested a review from a team as a code owner December 31, 2025 11:34
@NotMyFault NotMyFault requested a review from timja December 31, 2025 11:34
Copy link
Member

@timja timja left a comment

Choose a reason for hiding this comment

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

LGTM - although need to investigate the ssh agent issue

@NotMyFault
Copy link
Member Author

LGTM - although need to investigate the ssh agent issue

=> #2568 given the same failure is present on the master branch and shouldn't block this PR.

@timja
Copy link
Member

timja commented Dec 31, 2025

LGTM - although need to investigate the ssh agent issue

=> #2568 given the same failure is present on the master branch and shouldn't block this PR.

CI should pass here before integrating more changes. Top priority is a stable (green) master branch not adding more changes.

it can be temporarily ignored (via @Ignored / @Disabled) although really should have investigation done first.

@timja timja merged commit ef7dc6b into jenkinsci:master Jan 1, 2026
34 checks passed
@NotMyFault NotMyFault deleted the jdk25 branch January 1, 2026 17:03
@NotMyFault NotMyFault changed the title Test on Java 25 Allow ATH usage on Java 25 Jan 1, 2026
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