Closed luigidellaquila closed 1 year ago
Pinging @elastic/es-core-infra (Team:Core/Infra)
Looks like there's something different in Java 21, works fine in Java 20 and the failure history is all Java 21.
I had to install Java 21 and update gradle.properties
, but this reproduces.
diff --git a/gradle.properties b/gradle.properties
index ac5f070bebc..5aad65317ba 100644
--- a/gradle.properties
+++ b/gradle.properties
@@ -12,7 +12,7 @@ systemProp.jdk.tls.client.protocols=TLSv1.2
# java homes resolved by environment variables
org.gradle.java.installations.auto-detect=false
-org.gradle.java.installations.fromEnv=JAVA_TOOLCHAIN_HOME,JAVA_HOME,RUNTIME_JAVA_HOME,JAVA20_HOME,JAVA19_HOME,JAVA18_HOME,JAVA17_HOME,JAVA16_HOME,JAVA15_HOME,JAVA14_HOME,JAVA13_HOME,JAVA12_HOME,JAVA11_HOME,JAVA8_HOME
+org.gradle.java.installations.fromEnv=JAVA_TOOLCHAIN_HOME,JAVA_HOME,RUNTIME_JAVA_HOME,JAVA20_HOME,JAVA19_HOME,JAVA18_HOME,JAVA17_HOME,JAVA16_HOME,JAVA15_HOME,JAVA14_HOME,JAVA13_HOME,JAVA12_HOME,JAVA11_HOME,JAVA8_HOME,JAVA21_HOME
# log some dependency verification info to console
org.gradle.dependency.verification.console=verbose
https://github.com/elastic/elasticsearch/pull/96932 added java 21 EA to test, it seems to break several painless tests.
Fixed by #97062
Build scan: https://gradle-enterprise.elastic.co/s/qrcbcmi7orzxa/tests/:modules:lang-painless:yamlRestTestV7CompatTest/org.elasticsearch.painless.LangPainlessClientYamlTestSuiteIT/test%20%7Byaml=painless%2F130_metric_agg%2FScripted%20Metric%20Agg%20Total%7D
Reproduction line:
Applicable branches: main
Reproduces locally?: No
Failure history: https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.painless.LangPainlessClientYamlTestSuiteIT&tests.test=test%20%7Byaml%3Dpainless/130_metric_agg/Scripted%20Metric%20Agg%20Total%7D
Failure excerpt: