-
Notifications
You must be signed in to change notification settings - Fork 24.8k
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
Upgrade to Gradle 4.8 and enable JDK 11 in CI #31230
Comments
Pinging @elastic/es-core-infra |
Relates gradle/gradle#5692 |
Relates gradle/gradle#5696 |
After working around the Gradle issues, there are still problems when running the tests JDK 11, or more precisely javadoc, seems to bring it's own baggage. It detects many javadoc issues like invalid |
Fun!
These seem like things we could fix up before merging this PR. |
I will push a fix for the tests. For JDK 11 I have those fixed as well, but was planing a separate PR. |
org.elasticsearch.ingest.attachment.TikaImpl in the ingest-attachemnt plugin has an assert for java version < 11 due to a bug in the JDK. I'm updating it to add the required permission only if running on <= 10, then we can remove this logic when we only support 11. This is a note so we don't forget about it, not sure what the best place to note this is, perhaps an issue to move to java 11. One Tika test is failing. According to @dadoonet the behavior of Tika changes depending on the JVM being used. Muted the test and submitted #31305 |
Getting several of these in various ITs when using JDK 11:
|
The linked issue seems to be a different problem, no? |
@s1monw indeed, thanks for noticing. I edited the comments to have the link in the right place. There's no issue for |
@atorok I am very curious what could cause this. Do you have more information or only this error? I am not sure when I will have time to dig into it but I will eventually. |
@s1monw I was planing to look at it more Today, there's much I still need to understand about that what that test does, my first taught is to investigate around the lines of |
It seems to fail only with async. The test randomizes that. |
The problem with |
Builds are finally passing on JDK 10 with Gradle 4.8. |
JDK 11 builds are also passing in CI except for when the runtime java is set to 11 |
++ thanks for clarifying |
I can't build es-5.6, it report errors like follows, how can i do some thing:
|
With rampdown phase one of JDK 11 approaching, we want to enable JDK 11 in CI. However, Gradle 4.7 is not compatible with JDK 11 (gradle/gradle#4860). The upgrade to Gradle 4.8 is non-trivial due to Gradle reporting dependency conflicts.
Since the non-trivial nature keeps escalating feature/31230_gradle48_jdk11 will track this work.
CI for the branch is at (elastic/infra#5603, elastic/infra#5601): https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+31230_gradle48_jdk11+matrix-java-feature-branch/
TODO:
.ci
These changes should be done for 6.x and master.
Note: Gradle doesn't fully support JDK 11 as of 4.8 and full support is planned for 4.9 gradle/gradle#5120 .
The text was updated successfully, but these errors were encountered: