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

[META] Ensure JDK-21 (baseline) ecosystem readiness #10334

Closed
95 tasks done
reta opened this issue Oct 3, 2023 · 17 comments
Closed
95 tasks done

[META] Ensure JDK-21 (baseline) ecosystem readiness #10334

reta opened this issue Oct 3, 2023 · 17 comments
Assignees
Labels
enhancement Enhancement or improvement to existing feature or request v2.12.0 Issues and PRs related to version 2.12.0 v3.0.0 Issues and PRs related to version 3.0.0

Comments

@reta
Copy link
Collaborator

reta commented Oct 3, 2023

Is your feature request related to a problem? Please describe.
The JDK-21 is already out and becomes the next LTS release.

Describe the solution you'd like
We need to make sure the ecosystem is ready to be using JDK-21 at runtime as well as build time:

Plugins / clients / tools:

There are some foreseeing delays unfortunately with Eclipse Temurin JDK-21 distribution:

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context

@reta reta added the enhancement Enhancement or improvement to existing feature or request label Oct 3, 2023
@reta reta self-assigned this Oct 3, 2023
@heemin32
Copy link
Contributor

Can't wait to see this to be completed.

@reta
Copy link
Collaborator Author

reta commented Oct 11, 2023

Can't wait to see this to be completed.

Thanks @heemin32 , moving forward as things become available :-)

@jmazanec15
Copy link
Member

jmazanec15 commented Nov 2, 2023

Hey @reta, for some plugins, I see current jdk matrix is 11/17/20 or 11/17 - I assume we can drop testing of 20 for 21, but keep 11 and 17, correct?

Ref:

  1. https://github.com/opensearch-project/anomaly-detection/blob/4c6ba48bf9fb9234ad8bc0b9193f3c68409acfb9/.github/workflows/test_security.yml#L14

@reta
Copy link
Collaborator Author

reta commented Nov 2, 2023

Hey @reta, for some plugins, I see current jdk matrix is 11/17/20 or 11/17 - I assume we can drop testing of 20 for 21, but keep 11 and 17, correct?

Thanks @jmazanec15 , that is correct, I will be going through all plugins to get them ready for JDK-21

jmazanec15 added a commit to jmazanec15/anomaly-detection-1 that referenced this issue Nov 2, 2023
Coming from
opensearch-project/OpenSearch#10334, this
updates the CI system to use jdk-21, which is latest LTS supported
version.

Signed-off-by: John Mazanec <jmazane@amazon.com>
jmazanec15 added a commit to jmazanec15/k-NN-1 that referenced this issue Nov 2, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
@reta reta removed the untriaged label Nov 2, 2023
jmazanec15 added a commit to jmazanec15/k-NN-1 that referenced this issue Nov 6, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
jmazanec15 added a commit to opensearch-project/k-NN that referenced this issue Nov 7, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
opensearch-trigger-bot bot pushed a commit to opensearch-project/k-NN that referenced this issue Nov 7, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
(cherry picked from commit c72f096)
jmazanec15 added a commit to opensearch-project/k-NN that referenced this issue Nov 7, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
(cherry picked from commit c72f096)
jmazanec15 pushed a commit to opensearch-project/k-NN that referenced this issue Nov 7, 2023
This updates the CI system to use jdk-21, which is latest LTS supported version.
Coming from opensearch-project/OpenSearch#10334

Signed-off-by: John Mazanec <jmazane@amazon.com>
@jmazanec15
Copy link
Member

Hey @reta knn CI upgrade has been completed for main/2.x. I cant check off the box in the issue, but wanted to update.

@reta
Copy link
Collaborator Author

reta commented Nov 7, 2023

Hey @reta knn CI upgrade has been completed for main/2.x. I cant check off the box in the issue, but wanted to update.

Thanks a lot @jmazanec15 , I've added both pull requests, thank you!

@Pallavi-AWS
Copy link
Member

@reta, @bbarani @prudhvigodithi - Are we impacted with the JDK bug in https://bugs.openjdk.org/browse/JDK-8323659?

@reta
Copy link
Collaborator Author

reta commented Jan 19, 2024

@reta, @bbarani @prudhvigodithi - Are we impacted with the JDK bug in https://bugs.openjdk.org/browse/JDK-8323659?

@Pallavi-AWS not at the moment, the bug was introduced in 21.0.2 / 22 (which we haven't updated yet, see please #11906), we'll watch for any regressions once 21.0.2 bits become available (Eclipse Temurin), thank you for heads up

@prudhvigodithi prudhvigodithi changed the title [META] Ensure JDK-21 ecosystem readiness [META] Ensure JDK-21 (21.0.1) ecosystem readiness Jan 19, 2024
@prudhvigodithi
Copy link
Member

Thanks @reta I have just updated the title to reflect 21.0.1 and apart from core the build/assemble I see its using 21.0.1 as well opensearch-project/opensearch-build#4071.
@Pallavi-AWS @bbarani @peterzhuamazon

@joshuali925
Copy link
Member

this meta issue is missing two plugins
https://github.com/opensearch-project/observability
https://github.com/opensearch-project/reporting

I created opensearch-project/observability#1778 for observability, will do for reporting, but currently CI on windows is failing

@reta
Copy link
Collaborator Author

reta commented Jan 19, 2024

@Pallavi-AWS this issues has no reference to the JDK version, this is just for tracking JDK-21 baseline

@reta reta changed the title [META] Ensure JDK-21 (21.0.1) ecosystem readiness [META] Ensure JDK-21 (baseline) ecosystem readiness Jan 19, 2024
@reta
Copy link
Collaborator Author

reta commented Jan 19, 2024

I created opensearch-project/observability#1778 for observability, will do for reporting, but currently CI on windows is failing

Thanks @joshuali925 , updated the description

@joshuali925
Copy link
Member

any other plugin running into javax.management.InstanceNotFoundException: org.jacoco:type=Runtime (link) on windows after upgrading gradle to 8.5?

i'm having trouble with debugger on windows, but i pulled notifications main on windows and got the same error using ./gradlew build

@reta
Copy link
Collaborator Author

reta commented Jan 20, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement or improvement to existing feature or request v2.12.0 Issues and PRs related to version 2.12.0 v3.0.0 Issues and PRs related to version 3.0.0
Projects
None yet
Development

No branches or pull requests

6 participants