Skip to content

Upgrade to Quarkus 3.12#3700

Merged
sophokles73 merged 5 commits intomasterfrom
upgrade_to_quarkus_3.12
Nov 19, 2025
Merged

Upgrade to Quarkus 3.12#3700
sophokles73 merged 5 commits intomasterfrom
upgrade_to_quarkus_3.12

Conversation

@sophokles73
Copy link
Contributor

Also removed/replaced invocations of arbitrary deprecated API and upgraded to latest Jaeger backend.

@sophokles73 sophokles73 requested review from mattkaem and removed request for dejanb and kaniyan October 17, 2025 09:03
@DominikBasnerSotec
Copy link
Contributor

DominikBasnerSotec commented Oct 22, 2025

Thx for the update! I checked that 3.12 has reload-period for certs inside, would enable better cert update handling. Already excited for the update

@sophokles73
Copy link
Contributor Author

Thx for the update! I checked that 3.12 has reload-period for certs inside, would enable better cert update handling. Already excited for the update

Note that this will only be an intermediary step towards the latest Quarkus LTS version, though. We are upgrading step by step to make sure that we are correctly adopting the more interesting/severe changes that Quarkus has introduced. The Hono 2.7.0 release will then be based on Quarkus 3.27 ...

@sophokles73 sophokles73 force-pushed the upgrade_to_quarkus_3.12 branch 2 times, most recently from d9e3e51 to 90ec9d5 Compare October 29, 2025 12:11
@sophokles73
Copy link
Contributor Author

@mattkaem would you mind taking another look?

@mattkaem
Copy link
Contributor

mattkaem commented Nov 6, 2025

@mattkaem would you mind taking another look?

I don't see any changes since my last review, except for the rebases of course. So I think once we close the comments from the last review this can be merged.

@sophokles73
Copy link
Contributor Author

@mattkaem would you mind taking another look?

I don't see any changes since my last review, except for the rebases of course. So I think once we close the comments from the last review this can be merged.

I was under the impression that I had already fixed that. My bad. I have now pushed a corresponding change...

Copy link
Contributor

@mattkaem mattkaem left a comment

Choose a reason for hiding this comment

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

LGTM

Thank you for upgrading our Quarkus version!

Also removed obsolete native-image.properties and moved reflection and
resource definitions for GraalVM's native image builder to dedicated
config files as recommended by GraalVM's documentation.
Removed/replaced invocations of arbitrary deprecated API.
@sophokles73 sophokles73 force-pushed the upgrade_to_quarkus_3.12 branch from 1765235 to 6d73cda Compare November 10, 2025 09:01
@sophokles73 sophokles73 merged commit ab58a16 into master Nov 19, 2025
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants