Repository navigation
Upgrade to Quarkus 3.33.2 - #49401
Conversation
Closes keycloak#48311 Closes keycloak#46172 Closes keycloak#48805 Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
| <hibernate-orm.plugin.version>6.2.13.Final</hibernate-orm.plugin.version> | ||
| <hibernate.c3p0.version>6.2.13.Final</hibernate.c3p0.version> | ||
| <microprofile-metrics-api.version>5.1.2</microprofile-metrics-api.version> | ||
| <infinispan.version>16.0.8</infinispan.version> |
There was a problem hiding this comment.
Quarkus is using https://github.com/quarkusio/quarkus/blob/3.33.2/bom/application/pom.xml#L131 16.0.9. Should we stick to 16.0.8?
There was a problem hiding this comment.
We're not aligning Infinispan version with Quarkus. Maybe @ahus1 remembers the reasoning behind that.
There was a problem hiding this comment.
Ok, thanks. Maybe there could be a comment added in the future, because it seems random to me.
There was a problem hiding this comment.
It looks like the infinispan version, even micro ones, could be a little involved: https://github.com/keycloak/keycloak/pull/47018/changes cc @ryanemerson
There was a problem hiding this comment.
It looks like the infinispan version, even micro ones, could be a little involved: https://github.com/keycloak/keycloak/pull/47018/changes cc @ryanemerson
in general, I think the same can be said about each of our component, e.g. following bumps were not involved:
I agree it makes sense to leave this to experts, so I withdraw my question. Thanks
There was a problem hiding this comment.
We don't align with the Quarkus Infinispan version as we need to use the Infinispan version that is used downstream for Data Grid as this is a dependency for our HA multi-cluster capabilities with RHBK. #49206 has been created to track the next upgrade.
Ok, thanks. Maybe there could be a comment added in the future, because it seems random to me.
That could help 👍
There was a problem hiding this comment.
We don't align with the Quarkus Infinispan version as we need to use the Infinispan version that is used downstream for Data Grid as this is a dependency for our HA multi-cluster capabilities with RHBK. #49206 has been created to track the next upgrade.
I see. I'm still learning, thanks.
|
@shawkins Can I get your green tick, so we can merge? :) |
Closes keycloak#48311 Closes keycloak#46172 Closes keycloak#48805 Signed-off-by: Václav Muzikář <vmuzikar@ibm.com>
Closes #48311
Closes #46172
Closes #48805