Your gradle.properties looks good, it shouldn't hit the same issue as the ASF Jenkins. The gradle.properties of the ASF Jenkins looked like this before I updated it:
``` # Disable Gradle daemon for all builds. org.gradle.daemon=false # Lucene specific settings for lucene2 build nodes systemProp.file.encoding=UTF-8 org.gradle.jvmargs=-Xmx3g org.gradle.workers.max=4 tests.jvms=4 tests.multiplier=2 ``` And now: ``` # Disable Gradle daemon for all builds. org.gradle.daemon=false # Lucene specific settings for lucene2 build nodes systemProp.file.encoding=UTF-8 org.gradle.jvmargs=-Xmx3g \ --add-exports jdk.compiler/com.sun.tools.javac.api=ALL-UNNAMED \ --add-exports jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED \ --add-exports jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED \ --add-exports jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED \ --add-exports jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED org.gradle.workers.max=4 tests.jvms=4 tests.multiplier=2 ``` I was seeing a few errors in precommit in early versions of the linked PR, so I wanted to make sure I was using the most up-to-date versions of dependencies before attempting any fix. But to my knowledge this new version doesn't address the failure that we saw on the ASF Jenkins. On Thu, Jan 6, 2022 at 11:33 AM Uwe Schindler <[email protected]> wrote: > > But Jenkins removes its gradle.properties file from the build directory > before each build as it starts with a clean checkout. > > So it is not understandable to me. It recreated the file: > https://jenkins.thetaphi.de/job/Lucene-main-Linux/ws/gradle.properties/*view*/ > See also timestamp: https://jenkins.thetaphi.de/job/Lucene-main-Linux/ws/ > > I have seen that you updated the format in the PR. > > Uwe > > ----- > Uwe Schindler > Achterdiek 19, D-28357 Bremen > https://www.thetaphi.de > eMail: [email protected] > > > -----Original Message----- > > From: Adrien Grand <[email protected]> > > Sent: Thursday, January 6, 2022 11:28 AM > > To: Lucene Dev <[email protected]> > > Subject: Re: Heads up: main branch will soon require Java 17 > > > > To clarify, merging the PR wouldn't address the test failures. > > > > On Thu, Jan 6, 2022 at 11:27 AM Adrien Grand <[email protected]> wrote: > > > > > > This seems to be due to the fact that it's using an old > > > gradle.properties file that doesn't have the module exports that Dawid > > > added as part of LUCENE-10066. > > > https://github.com/apache/lucene/pull/259/files#diff- > > fbdfae634c6898f4a415bc2a278ff771c38436d70b0e64250ab068b4b3e58426R > > 48-R52 > > > > > > Let me look into whether I can make the ASF Jenkins recreate its > > > gradle.properties. > > > > > > > > -- > > Adrien > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [email protected] > > For additional commands, e-mail: [email protected] > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > -- Adrien --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
