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

Test Java 21 on CI #36284

Merged
merged 10 commits into from
Oct 24, 2023
Merged

Test Java 21 on CI #36284

merged 10 commits into from
Oct 24, 2023

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Oct 4, 2023

@snazy trying this crazy stuff in my fork for now :).

@gsmet gsmet changed the title Use GRADLE_JAVA_HOME in CI Use GRADLE_JAVA_HOME in CI and test JDK 21 Oct 4, 2023
@quarkus-bot quarkus-bot bot added the area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure label Oct 4, 2023
@gsmet
Copy link
Member Author

gsmet commented Oct 5, 2023

FYI, I'm making progress on this and testing in my fork for now as I iterate slowly.

@gsmet gsmet force-pushed the jdk-21 branch 2 times, most recently from 5f5bd50 to c7b1b47 Compare October 9, 2023 12:20
@gsmet
Copy link
Member Author

gsmet commented Oct 9, 2023

I'm not exactly sure what's wrong with the JDK 21 build but I end up with:

Unhandled exception. System.IO.IOException: No space left on device : '/home/runner/runners/2.309.0/_diag/Worker_20231006-101639-utc.log'
   at System.IO.RandomAccess.WriteAtOffset(SafeFileHandle handle, ReadOnlySpan`1 buffer, Int64 fileOffset)
   at System.IO.Strategies.BufferedFileStreamStrategy.FlushWrite()
   at System.IO.StreamWriter.Flush(Boolean flushStream, Boolean flushEncoder)
   at System.Diagnostics.TextWriterTraceListener.Flush()
   at System.Diagnostics.TraceSource.Flush()
   at GitHub.Runner.Common.TraceManager.Dispose(Boolean disposing)
   at GitHub.Runner.Common.TraceManager.Dispose()
   at GitHub.Runner.Common.HostContext.Dispose(Boolean disposing)
   at GitHub.Runner.Common.HostContext.Dispose()
   at GitHub.Runner.Worker.Program.Main(String[] args)

I'm investigating what's going on but I'm not entirely sure if I will be able to get the info, especially if the problem is with the build generating too many files...

@github-actions
Copy link

🎊 PR Preview b2179b8 has been successfully built and deployed to https://quarkus-pr-main-36284-preview.surge.sh/version/main/guides/

@gsmet gsmet force-pushed the jdk-21 branch 3 times, most recently from bac4c63 to bad5f4f Compare October 20, 2023 09:14
@gsmet gsmet changed the title Use GRADLE_JAVA_HOME in CI and test JDK 21 Test Java 21 on CI Oct 23, 2023
@gsmet gsmet marked this pull request as ready for review October 23, 2023 12:28
@gsmet gsmet requested a review from yrodiere October 23, 2023 12:31
@gsmet
Copy link
Member Author

gsmet commented Oct 23, 2023

This should now be ready. The tests are passing and I fixed/disabled the failing tests. Let's see what CI has to say.

@quarkus-bot

This comment has been minimized.

This test is not compatible with Java 21.

See jakartaee/cdi-tck#485
@quarkus-bot quarkus-bot bot added the area/arc Issue related to ARC (dependency injection) label Oct 24, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Oct 24, 2023

Failing Jobs - Building 1632b31

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 11 Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 17
JVM Tests - JDK 17 Windows Build Failures Logs Raw logs
✔️ JVM Tests - JDK 21
✔️ Maven Tests - JDK 11
Maven Tests - JDK 11 Windows Build Failures Logs Raw logs
Native Tests - Messaging1 Build Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 17 Windows #

- Failing: extensions/vertx-http/deployment 
! Skipped: devtools/cli extensions/agroal/deployment extensions/amazon-lambda-http/deployment and 364 more

📦 extensions/vertx-http/deployment

io.quarkus.vertx.http.proxy.TrustedForwarderProxyFailureTest.testHeadersAreIgnored line 29 - More details - Source on GitHub

java.net.SocketTimeoutException: Read timed out
	at java.base/sun.nio.ch.NioSocketImpl.timedRead(NioSocketImpl.java:288)
	at java.base/sun.nio.ch.NioSocketImpl.implRead(NioSocketImpl.java:314)

⚙️ Maven Tests - JDK 11 Windows #

- Failing: integration-tests/maven 

📦 integration-tests/maven

io.quarkus.maven.it.DevMojoIT.testExternalReloadableArtifacts line 1448 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.maven.it.DevMojoIT was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

io.quarkus.maven.it.DevMojoIT.testExternalReloadableArtifacts line 1448 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.maven.it.DevMojoIT was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

io.quarkus.maven.it.JarRunnerIT.testNonAsciiDir line 70 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

io.quarkus.maven.it.JarRunnerIT.testNonAsciiDir line 70 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual:

⚙️ Native Tests - Messaging1 #

- Failing: integration-tests/kafka-oauth-keycloak 

📦 integration-tests/kafka-oauth-keycloak

Failed to execute goal io.quarkus:quarkus-maven-plugin:999-SNAPSHOT:build (default) on project quarkus-integration-test-kafka-oauth-keycloak: Failed to build quarkus application

@gsmet gsmet merged commit ee1d5a3 into quarkusio:main Oct 24, 2023
48 of 52 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.6 - main milestone Oct 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/arc Issue related to ARC (dependency injection) area/documentation area/infra-automation anything related to CI, bots, etc. that are used to automated our infrastructure area/scala
Projects
Development

Successfully merging this pull request may close these issues.

3 participants