Skip to content

Conversation

ozangunalp
Copy link
Contributor

Release 4.29.0

Major Features & Improvements:

  • Kafka Transactions promoted to tech preview
  • AMQP connector refactor - channels moved from connector bean to separate classes
  • AMQP Outgoing channel attribute for lazy/eager connection
  • JMS Connection reuse - ability to reuse Connection on JMSContext
  • MQTT client ID fix - use correct ID for identifying MQTT clients

Bug Fixes:

  • Fixed flaky JMS disconnection test
  • Fixed context duplication not inheriting locals from owner context
  • Fixed Kafka transactions tech preview rev API

@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/reactive-messaging area/smallrye labels Sep 17, 2025
@geoand
Copy link
Contributor

geoand commented Sep 17, 2025

Should this be backported to 3.28?

This comment has been minimized.

@gastaldi gastaldi added the triage/needs-rebase This PR needs to be rebased first because it has merge conflicts label Sep 18, 2025
@gsmet
Copy link
Member

gsmet commented Sep 18, 2025

@brunobat could you have a look at the failure in OtlpUpstreamMetricsExporterErrorTest. It seems we got a NPE when starting Quarkus.

Copy link
Contributor

@brunobat brunobat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to wait to merge this.
This version of SR Reactive messaging depends on opentelemetry.version>1.48.0.
Quarkus is still on OTel SDK 1.44.1 because of #47157 (comment)

@ozangunalp
Copy link
Contributor Author

@brunobat no, no this version doesn't contain the OTel update.

It depends still on opentelemetry-instrumentation-bom-alpha:2.5.0-alpha, which itself is on opentelemetry-bom:1.39.0.

Copy link

quarkus-bot bot commented Sep 18, 2025

Status for workflow Quarkus CI

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

✅ 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 21

📦 extensions/smallrye-openapi/deployment

io.quarkus.smallrye.openapi.test.vertx.OpenApiHttpRootPathCorsTestCase.testCorsFilterProperties - History

  • 1 expectation failed. Expected status code <200> but was <500>. - java.lang.AssertionError
java.lang.AssertionError: 
1 expectation failed.
Expected status code <200> but was <500>.

	at java.base/jdk.internal.reflect.DirectConstructorHandleAccessor.newInstance(DirectConstructorHandleAccessor.java:62)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:502)
	at java.base/java.lang.reflect.Constructor.newInstance(Constructor.java:486)
	at org.codehaus.groovy.reflection.CachedConstructor.invoke(CachedConstructor.java:73)

⚙️ JVM Tests - JDK 17 Windows

📦 extensions/micrometer-opentelemetry/deployment

io.quarkus.micrometer.opentelemetry.deployment.compatibility.MicrometerTimedInterceptorTest.testTimeMethod - 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(MicrometerTimedInterceptorTest.java:77)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.QuarkusUnitTest.runExtensionMethod(QuarkusUnitTest.java:534)

📦 extensions/scheduler/deployment

io.quarkus.scheduler.test.PausedSchedulerTest.testSchedulerPauseResume - History

  • expected: <false> but was: <true> - org.opentest4j.AssertionFailedError
org.opentest4j.AssertionFailedError: expected: <false> but was: <true>
	at org.junit.jupiter.api.AssertionFailureBuilder.build(AssertionFailureBuilder.java:151)
	at org.junit.jupiter.api.AssertionFailureBuilder.buildAndThrow(AssertionFailureBuilder.java:132)
	at org.junit.jupiter.api.AssertFalse.failNotFalse(AssertFalse.java:63)
	at org.junit.jupiter.api.AssertFalse.assertFalse(AssertFalse.java:36)
	at org.junit.jupiter.api.AssertFalse.assertFalse(AssertFalse.java:31)
	at org.junit.jupiter.api.Assertions.assertFalse(Assertions.java:231)
	at io.quarkus.scheduler.test.PausedSchedulerTest.testSchedulerPauseResume(PausedSchedulerTest.java:47)

Copy link
Contributor

@brunobat brunobat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ozangunalp. I must have checked out the wrong tag.

@ozangunalp ozangunalp merged commit 2a7a009 into quarkusio:main Sep 18, 2025
57 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.29 - main milestone Sep 18, 2025
@gsmet gsmet modified the milestones: 3.29 - main, 3.28.1 Sep 23, 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/reactive-messaging area/smallrye triage/flaky-test triage/needs-rebase This PR needs to be rebased first because it has merge conflicts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants