Skip to content

Conversation

@manovotn
Copy link
Contributor

@manovotn manovotn commented Nov 7, 2025

This is a revert of e8157ae

The newer version causes a regression when using Quartz in combination with PostgreSQL.
It was originally found in a CI run for quickstarts as we use this DB there, see #6588 (comment).

The Quartz PR causing this is quartz-scheduler/quartz#1344 (comment) where there is a report of the very same issue.

FYI there is already a pending fix (quartz-scheduler/quartz#1411) so we just need that to get merged and wait for newer version.

@manovotn manovotn requested a review from mkouba November 7, 2025 13:45
@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/scheduler labels Nov 7, 2025
@mkouba mkouba added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 7, 2025
@quarkus-bot
Copy link

quarkus-bot bot commented Nov 7, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit e352877.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.


Flaky tests - Develocity

⚙️ JVM Tests - JDK 17 Windows

📦 extensions/micrometer-opentelemetry/deployment

io.quarkus.micrometer.opentelemetry.deployment.compatibility.MicrometerTimedInterceptorTest.testTimeMethod_Uni - History

  • Stream has no elements - java.lang.IllegalArgumentException
java.lang.IllegalArgumentException: Stream has no elements
	at io.quarkus.micrometer.opentelemetry.deployment.common.MetricDataFilter.lambda$lastReading$2(MetricDataFilter.java:213)
	at java.base/java.util.Optional.orElseThrow(Optional.java:403)
	at io.quarkus.micrometer.opentelemetry.deployment.common.MetricDataFilter.lastReading(MetricDataFilter.java:213)
	at io.quarkus.micrometer.opentelemetry.deployment.common.MetricDataFilter.lastReadingDataPoint(MetricDataFilter.java:231)
	at io.quarkus.micrometer.opentelemetry.deployment.compatibility.MicrometerTimedInterceptorTest.testTimeMethod_Uni(MicrometerTimedInterceptorTest.java:174)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:534)

⚙️ MicroProfile TCKs Tests

📦 tcks/microprofile-lra

org.eclipse.microprofile.lra.tck.TckRecoveryTests.testCancelWhenParticipantIsUnavailable - History

  • Expecting the metric Compensated callback was called Expected: a value equal to or greater than <1> but: <0> was less than <1> - java.lang.AssertionError
java.lang.AssertionError: 
Expecting the metric Compensated callback was called
Expected: a value equal to or greater than <1>
     but: <0> was less than <1>
	at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)
	at org.eclipse.microprofile.lra.tck.TckRecoveryTests.assertMetricCallbackCalled(TckRecoveryTests.java:210)
	at org.eclipse.microprofile.lra.tck.TckRecoveryTests.testCancelWhenParticipantIsUnavailable(TckRecoveryTests.java:195)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)

@manovotn manovotn merged commit 0810c92 into quarkusio:main Nov 8, 2025
59 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.30 - main milestone Nov 8, 2025
@manovotn manovotn deleted the downgradeQuartz branch November 8, 2025 10:57
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 8, 2025
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/scheduler triage/flaky-test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants