Skip to content
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.

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

Test with Java 21 and modernization #105

Merged
merged 6 commits into from
Jan 23, 2024
Merged

Conversation

gounthar
Copy link
Contributor

@gounthar gounthar commented Oct 3, 2023

Test with Java 21

Java 21 was released Sep 19, 2023. We want to announce full support for Java 21 in early October and would like the most used plugins to be compiled and tested with Java 21.

The acceptance test harness and plugin bill of materials tests are already passing with Java 21. This is a further step to improve plugin readiness for use with Java 21 and for development with Java 21.

The change intentionally tests only two Java configurations, Java 17 and Java 21 because we believe that the risk of a regression that only affects Java 11 is shallow. We generate Java 11 byte code with the Java 17 and the Java 21 builds, so we're already testing Java 11 byte code.

Testing done

Confirmed tests pass with Java 21 after upgrading to a recent parent POM and to Jenkins version 2.361.4 as the minimum version.

 General error during conversion: Unsupported class file major version 65
  This version of maven-hpi-plugin requires Jenkins 2.361 or later
  This version of maven-hpi-plugin requires Jenkins 2.361 or later
@gounthar
Copy link
Contributor Author

gounthar commented Oct 3, 2023

We're still using JDK8, and that's why the build is failing.
Could someone please re-run it with our proposed Jenkinsfile?
Thanks.

@basil
Copy link
Member

basil commented Oct 3, 2023

https://ci.jenkins.io/job/Plugins/job/pipeline-stage-step-plugin/job/PR-105/2/

@gounthar
Copy link
Contributor Author

gounthar commented Oct 4, 2023

Thanks, Basil.

pom.xml Outdated Show resolved Hide resolved
Copy link

@MarkEWaite MarkEWaite left a comment

Choose a reason for hiding this comment

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

Needs the correct 2.361.x plugin bom release number.

@gounthar
Copy link
Contributor Author

gounthar commented Oct 9, 2023

Of course, sorry about that.

Co-authored-by: Mark Waite <[email protected]>
@MarkEWaite
Copy link

@car-roll , @rsandell , or @dwnusbaum could you merge this so that the plugin is regularly tested with Java 21 and Java 17?

pom.xml Outdated Show resolved Hide resolved
@car-roll car-roll added the tests label Jan 23, 2024
@car-roll car-roll merged commit 458cac9 into jenkinsci:master Jan 23, 2024
14 checks passed
@MarkEWaite
Copy link

Thanks @car-roll !

@gounthar gounthar deleted the jdk21 branch January 24, 2024 12:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants