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

Support JDK-21 builds on Jenkins #4153

Merged
merged 1 commit into from
Oct 19, 2023

Conversation

reta
Copy link
Contributor

@reta reta commented Oct 19, 2023

Description

Support JDK-21 builds on Jenkins

Issues Resolved

Part of #4071

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@codecov
Copy link

codecov bot commented Oct 19, 2023

Codecov Report

Merging #4153 (708e661) into main (559e00d) will not change coverage.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #4153   +/-   ##
=======================================
  Coverage   93.27%   93.27%           
=======================================
  Files         187      187           
  Lines        5755     5755           
=======================================
  Hits         5368     5368           
  Misses        387      387           

@peterzhuamazon
Copy link
Member

I will rebuild the images with both #4154 and this PR.

@peterzhuamazon peterzhuamazon merged commit b583bc1 into opensearch-project:main Oct 19, 2023
13 checks passed
@peterzhuamazon
Copy link
Member

peterzhuamazon commented Oct 19, 2023

Hi @reta need one more for the windows images here:

https://github.com/opensearch-project/opensearch-build/blob/main/docker/ci/config/windows-servercore-setup.ps1#L87

Note that temurin hasnt have 21 on scoop, so openjdk21 it is for now.

Thanks.

@reta
Copy link
Contributor Author

reta commented Oct 20, 2023

Tried a few images, all is set:

[opensearch@fb75291982da ~]$ export | grep JAVA
declare -x JAVA11_HOME="/opt/java/openjdk-11"
declare -x JAVA17_HOME="/opt/java/openjdk-17"
declare -x JAVA19_HOME="/opt/java/openjdk-19"
declare -x JAVA20_HOME="/opt/java/openjdk-20"
declare -x JAVA21_HOME="/opt/java/openjdk-21"
declare -x JAVA8_HOME="/opt/java/openjdk-8"
declare -x JAVA_HOME="/opt/java/openjdk-21"
[opensearch@fb75291982da ~]$ $JAVA21_HOME/bin/java --version
openjdk 21 2023-09-19 LTS
OpenJDK Runtime Environment Temurin-21+35 (build 21+35-LTS)
OpenJDK 64-Bit Server VM Temurin-21+35 (build 21+35-LTS, mixed mode, sharing)
[opensearch@fb75291982da ~]$ 

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.

None yet

2 participants