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

Upgrade to version 42.5.0 of the PostgreSQL JDBC driver #27509

Merged
merged 1 commit into from Aug 26, 2022

Conversation

Sanne
Copy link
Member

@Sanne Sanne commented Aug 25, 2022

Needs CI opinion; we also want to check if this works flawlessly:

Rationale: Hibernate ORM 6 already upgraded.

@quarkus-bot quarkus-bot bot added the area/dependencies Pull requests that update a dependency file label Aug 25, 2022
@Sanne
Copy link
Member Author

Sanne commented Aug 25, 2022

Hibernate ORM v 5.6 seems compatible w/o need for changes.

@quarkus-bot
Copy link

quarkus-bot bot commented Aug 25, 2022

Failing Jobs - Building 5253d96

Status Name Step Failures Logs Raw logs
JVM Tests - JDK 11 Build Failures Logs Raw logs
JVM Tests - JDK 11 Windows Build Failures Logs Raw logs
✔️ JVM Tests - JDK 17
JVM Tests - JDK 18 Build Failures Logs Raw logs
MicroProfile TCKs Tests Verify Failures Logs Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ JVM Tests - JDK 11 #

- Failing: extensions/smallrye-reactive-messaging-kafka/deployment 
! Skipped: integration-tests/kafka-oauth-keycloak integration-tests/kafka-sasl-elytron integration-tests/kubernetes/quarkus-standard-way-kafka and 3 more

📦 extensions/smallrye-reactive-messaging-kafka/deployment

io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingTestCase.testContinuousTestingScenario2 line 83 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Failed to wait for test run 4 State{lastRun=3, running=true, inProgress=true, run=1, passed=0, failed=1, skipped=0, isBrokenOnly=false, isTestOutput=false, isInstrumentationBasedReload=false, isLiveReload=true}
	at io.quarkus.test.ContinuousTestingTestUtils.waitForNextCompletion(ContinuousTestingTestUtils.java:44)
	at io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingTestCase.testContinuousTestingScenario2(KafkaDevServicesContinuousTestingTestCase.java:83)

⚙️ JVM Tests - JDK 11 Windows #

- Failing: extensions/smallrye-reactive-messaging-amqp/deployment 
! Skipped: integration-tests/reactive-messaging-amqp 

📦 extensions/smallrye-reactive-messaging-amqp/deployment

io.quarkus.smallrye.reactivemessaging.amqp.devmode.AmqpDevModeTest.testCodeUpdate line 53 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with io.quarkus.smallrye.reactivemessaging.amqp.devmode.AmqpDevModeTest was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

⚙️ JVM Tests - JDK 18 #

- Failing: extensions/smallrye-reactive-messaging-kafka/deployment 
! Skipped: integration-tests/kafka-oauth-keycloak integration-tests/kafka-sasl-elytron integration-tests/kubernetes/quarkus-standard-way-kafka and 3 more

📦 extensions/smallrye-reactive-messaging-kafka/deployment

io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingTestCase.testContinuousTestingScenario2 line 83 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Failed to wait for test run 4 State{lastRun=3, running=true, inProgress=false, run=1, passed=0, failed=1, skipped=0, isBrokenOnly=false, isTestOutput=false, isInstrumentationBasedReload=false, isLiveReload=true}
	at io.quarkus.test.ContinuousTestingTestUtils.waitForNextCompletion(ContinuousTestingTestUtils.java:44)
	at io.quarkus.smallrye.reactivemessaging.kafka.deployment.testing.KafkaDevServicesContinuousTestingTestCase.testContinuousTestingScenario2(KafkaDevServicesContinuousTestingTestCase.java:83)

⚙️ MicroProfile TCKs Tests #

- Failing: tcks/resteasy-reactive 

📦 tcks/resteasy-reactive

Failed to execute goal org.codehaus.mojo:exec-maven-plugin:3.0.0:exec (test) on project quarkus-tck-resteasy-reactive: Command execution failed.

📦 tcks/resteasy-reactive/target/testsuite/tests

com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerClassNotAssignableContractsInMapTest line 924 - More details - Source on GitHub

com.sun.ts.tests.jaxrs.common.JAXRSCommonClient$Fault: unexpected number of registered classes found: 1 on Client configuration
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.checkConfig(JAXRSClient0133.java:1227)
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerClassNotAssignableContractsInMapTest(JAXRSClient0133.java:924)

com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerObjectNotAssignableContractsInMapTest line 1139 - More details - Source on GitHub

com.sun.ts.tests.jaxrs.common.JAXRSCommonClient$Fault: unexpected number of registered classes found: 1 on Client configuration
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.checkConfig(JAXRSClient0133.java:1227)
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerObjectNotAssignableContractsInMapTest(JAXRSClient0133.java:1139)

com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerObjectNotAssignableContractsTest line 669 - More details - Source on GitHub

com.sun.ts.tests.jaxrs.common.JAXRSCommonClient$Fault: unexpected number of registered classes found: 1 on Client configuration
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.checkConfig(JAXRSClient0133.java:1227)
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerObjectNotAssignableContractsTest(JAXRSClient0133.java:669)

com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerClassNotAssignableContractsTest line 294 - More details - Source on GitHub

com.sun.ts.tests.jaxrs.common.JAXRSCommonClient$Fault: unexpected number of registered classes found: 1 on Client configuration
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.checkConfig(JAXRSClient0133.java:1227)
	at com.sun.ts.tests.jaxrs.ee.rs.core.configurable.JAXRSClient0133.registerClassNotAssignableContractsTest(JAXRSClient0133.java:294)

@gsmet
Copy link
Member

gsmet commented Aug 25, 2022

@geoand the RESTEasy Reactive TCK started to fail in several pull requests.

@geoand
Copy link
Contributor

geoand commented Aug 25, 2022

Before or after the change you made?

@geoand
Copy link
Contributor

geoand commented Aug 26, 2022

Checking it out

@geoand
Copy link
Contributor

geoand commented Aug 26, 2022

It seems like #26495 is the culprit.

@Sanne
Copy link
Member Author

Sanne commented Aug 26, 2022

It seems like #26495 is the culprit.

does it imply I can merge this one?

@geoand
Copy link
Contributor

geoand commented Aug 26, 2022

Yup

@Sanne Sanne merged commit 3a568d2 into quarkusio:main Aug 26, 2022
@quarkus-bot quarkus-bot bot added this to the 2.13 - main milestone Aug 26, 2022
@Sanne Sanne deleted the pgUpgrade branch August 26, 2022 12:26
@Sanne
Copy link
Member Author

Sanne commented Aug 26, 2022

thanks!

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/persistence
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants