Skip to content

[#] JDK 25 support (multiple JIRA) #1533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

[#] JDK 25 support (multiple JIRA) #1533

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
cf8d91b
[#] NO-JIRA: Update Jenkinsfile to add JDK 25
mattrpav Nov 13, 2025
8fdde71
[AMQ-9563] Add a SubjectShim to support JAAS API JDK 24+
mattrpav Nov 14, 2025
4dfc1fd
[AMQ-9366] Add DataFileFactroy to KahaDBPersistenceAdapter and update…
mattrpav Nov 14, 2025
6de4d1f
[#] Temporarily @Ignore JmsSchedulerTest that are failing with JDK 25
mattrpav Nov 16, 2025
29bef28
[#] Temporarily @Ignore org.apache.activemq.perf.InactiveDurableTopic…
mattrpav Nov 18, 2025
1b30f42
Fix NetworkedSyncTest configuration
mattrpav Nov 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions Jenkinsfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ pipeline {

parameters {
choice(name: 'nodeLabel', choices: ['ubuntu', 's390x', 'arm', 'Windows'])
choice(name: 'jdkVersion', choices: ['jdk_17_latest', 'jdk_21_latest', 'jdk_24_latest', 'jdk_17_latest_windows', 'jdk_21_latest_windows', 'jdk_24_latest_windows'])
choice(name: 'jdkVersion', choices: ['jdk_17_latest', 'jdk_21_latest', 'jdk_25_latest', 'jdk_17_latest_windows', 'jdk_21_latest_windows', 'jdk_25_latest_windows'])
Copy link
Member

@jbonofre jbonofre Nov 13, 2025

Choose a reason for hiding this comment

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

I guess we should remove jdk_17 there already.

Copy link
Contributor Author

@mattrpav mattrpav Nov 13, 2025 •
edited
Loading

Choose a reason for hiding this comment

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

JDK 17 should stay. The new change is during release builds we need to ensure JDK 21 is used to support the multi-release jar working. The other PR has the enforce plugin set to check for JDK 21 being used only during release.

Copy link
Member

@gemmellr gemmellr Nov 17, 2025

Choose a reason for hiding this comment

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

It does, but at least at present this PR should also be updating it to require at least 24 (I just went with requiring 25 for Artemis, sets the 25 baseline now and saves changing it later, plus who really needs to use 24 for releasing now) since thats the compilation level being used.

booleanParam(name: 'deployEnabled', defaultValue: false)
booleanParam(name: 'sonarEnabled', defaultValue: false)
booleanParam(name: 'testsEnabled', defaultValue: true)
Expand Down Expand Up @@ -72,12 +72,12 @@ pipeline {
}
}

stage('Build JDK 24') {
stage('Build JDK 25') {
tools {
jdk "jdk_24_latest"
jdk "jdk_25_latest"
}
steps {
echo 'Building JDK 24'
echo 'Building JDK 25'
sh 'java -version'
sh 'mvn -version'
sh 'mvn -U -B -e clean install -DskipTests'
Expand Down