-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
Remove no-jdk distributions #76896
Remove no-jdk distributions #76896
Conversation
Pinging @elastic/es-delivery (Team:Delivery) |
@elasticmachine run elasticsearch-ci/packaging-tests-windows |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice cleanup! lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
} | ||
} | ||
currentDistros.add( | ||
createDistro(distributions, architecture, type, null, bundledJdk, VersionProperties.getElasticsearch()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can probably remove the bundleJdk parameter from the createDistro
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I purposefully didn't to make backports easier. That might be wishful thinking though.
The no-jdk distributions exist in 7.x and before. They were removed with 8.0. This commit removes the remaining deprecation messages for using the no-jdk distribution. Note that when talking with an older node, we drop the bundledJdk attribute. This is ok because it is only possible for this to not be true when talking with a 7.17 node, during an upgrade, and the usingBundledJdk is retained, which is the important thing if debugging a problem. relates elastic#76896 relates elastic#85758
The no-jdk distributions exist in 7.x and before. They were removed with 8.0. This commit removes the remaining deprecation messages for using the no-jdk distribution. Note that when talking with an older node, we drop the bundledJdk attribute. This is ok because it is only possible for this to not be true when talking with a 7.17 node, during an upgrade, and the usingBundledJdk is retained, which is the important thing if debugging a problem. relates #76896 relates #85758
Starting with Elasticsearch 8.0 we will no longer build separate "no-jdk" distributions. All distributions now include by default a bundled JDK and users can configure an alternative JDK distribution via
ES_JAVA_HOME
.I've left some references to no-jdk distribution handling code to make future backports easier but all the build scripts under
distribution/
have had conditional jdk logic stripped out. Additionally, there was a lot of lingering code related to OSS distributions which no longer exist which has been removed as well and will be separately backported to7.x
.Closes #65109