Skip to content

Conversation

@Ladicek
Copy link
Contributor

@Ladicek Ladicek commented Mar 25, 2025

This is due to an upcoming change in Execution Model Validation [1] where we need the JsonRPCProvidersBuildItem produced always, not only in dev mode. JSON RPC providers can use execution model affecting annotations, so we need to know about them, otherwise a non-dev build would fail with an incorrect validation error.

[1] quarkusio/quarkus#46965

This is due to an upcoming change in Execution Model Validation [1]
where we need the `JsonRPCProvidersBuildItem` produced always, not
only in dev mode. JSON RPC providers can use execution model affecting
annotations, so we need to know about them, otherwise a non-dev build
would fail with an incorrect validation error.

[1] quarkusio/quarkus#46965
@Ladicek Ladicek requested a review from a team as a code owner March 25, 2025 11:46
@fjtirado fjtirado merged commit f92b97d into quarkiverse:main Mar 25, 2025
1 check passed
@Ladicek Ladicek deleted the always-produce-json-rpc-providers branch March 25, 2025 14:07
@melloware
Copy link
Contributor

@ricardozanini @fjtirado can you do a release. Without this Quarkus 3.23.0 is giving this stack trace...

2025-05-25 09:07:57,943 ERROR [io.qua.dev.run.jso.JsonRpcCodec] (executor-thread-7) Error in JsonRPC Call: java.lang.NoClassDefFoundError: io/quarkus/devui/runtime/config/ConfigDescription
        at io.quarkiverse.embedded.postgresql.devui.EmbeddedPostgreSQLJsonRpcService.getDatasourcePort(EmbeddedPostgreSQLJsonRpcService.java:18)
        at io.quarkiverse.embedded.postgresql.devui.EmbeddedPostgreSQLJsonRpcService_ClientProxy.getDatasourcePort(Unknown Source)

@ricardozanini
Copy link
Member

@melloware main is on 3.15.x thou: https://github.com/quarkiverse/quarkus-embedded-postgresql/blob/main/pom.xml#L30

Is that alright? We are not planning to push the Quarkus version atm.

@melloware
Copy link
Contributor

Correct I think you just need to publish as is with the fix already merged from @Ladicek

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants