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

[3.2] Upgrade to Hibernate ORM 6.2.18.Final #38105

Merged
merged 2 commits into from
Jan 14, 2024

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Jan 9, 2024

No description provided.

@quarkus-bot
Copy link

quarkus-bot bot commented Jan 9, 2024

Thanks for your pull request!

The title of your pull request does not follow our editorial rules. Could you have a look?

  • title should preferably start with an uppercase character (if it makes sense!)

This message is automatically generated by a bot.

@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/hibernate-orm Hibernate ORM area/persistence labels Jan 9, 2024
@quarkus-bot
Copy link

quarkus-bot bot commented Jan 9, 2024

/cc @yrodiere (hibernate-orm)

@quarkus-bot
Copy link

quarkus-bot bot commented Jan 9, 2024

Failing Jobs - Building 512e746

Status Name Step Failures Logs Raw logs Build scan
✔️ Gradle Tests - JDK 11 🚧
Gradle Tests - JDK 11 Windows Build Failures Logs Raw logs 🚧
✔️ JVM Tests - JDK 11 🚧
✔️ JVM Tests - JDK 17 🚧
JVM Tests - JDK 17 Windows Build Failures Logs Raw logs 🚧

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

io.quarkus.gradle.CustomManifestArgumentsTest.shouldContainsSpecificManifestProperty line 37 - More details - Source on GitHub

java.lang.AssertionError: 

Expecting actual not to be null
	at io.quarkus.gradle.CustomManifestArgumentsTest.shouldContainsSpecificManifestProperty(CustomManifestArgumentsTest.java:37)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.base/java.lang.reflect.Method.invoke(Method.java:566)

⚙️ JVM Tests - JDK 17 Windows #

- Failing: extensions/resteasy-reactive/rest-client-reactive/runtime 
! Skipped: extensions/keycloak-admin-client-reactive/deployment extensions/keycloak-admin-client-reactive/runtime extensions/oidc-client-reactive-filter/deployment and 30 more

📦 extensions/resteasy-reactive/rest-client-reactive/runtime

io.quarkus.rest.client.reactive.runtime.RestClientCDIDelegateBuilderTest. - More details - Source on GitHub

java.io.IOException: Failed to delete temp directory D:\a\quarkus\quarkus\extensions\resteasy-reactive\rest-client-reactive\runtime\target\junit5956971861130122502. The following paths could not be deleted (see suppressed exceptions for details): <root>, keystore.jks, truststore.jks
	at org.junit.jupiter.engine.extension.TempDirectory$CloseablePath.createIOExceptionWithAttachedFailures(TempDirectory.java:351)
	at org.junit.jupiter.engine.extension.TempDirectory$CloseablePath.close(TempDirectory.java:252)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.jupiter.engine.execution.ExtensionValuesStore.lambda$closeAllStoredCloseableValues$3(ExtensionValuesStore.java:68)
	at java.base/java.util.stream.ForEachOps$ForEachOp$OfRef.accept(ForEachOps.java:183)
	at java.base/java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:197)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

@yrodiere
Copy link
Member

The test failures seem unrelated... ?

Looks like the upgrade doesn't cause any problem in our test suite.

@yrodiere
Copy link
Member

BTW @gsmet 6.2.19.Final was released yesterday, so... let's target that instead?

@yrodiere
Copy link
Member

FWIW this fixes the following issues in Quarkus (didn't find any other): #33740 #37184

@aloubyansky aloubyansky merged commit 2e1aae2 into quarkusio:3.2 Jan 14, 2024
44 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/dependencies Pull requests that update a dependency file area/hibernate-orm Hibernate ORM area/persistence
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants