Upgrade Infinispan to 14.0.13.Final#21565
Conversation
|
Waiting for Infinispan 14.0.12 Docker image to become available as this is needed for the tests: https://quay.io/repository/infinispan/server?tab=tags&tag=latest |
5cedd3e to
ae9b2f6
Compare
|
@ahus1 My suggestion would be to use |
|
@pruivo - yes, using the BOM would be a good thing. Still I don't know what else breaks/changes around that. When moving to the BOM, I'd also see some versions set in the "quarkus/runtime/pom.xml" and it would require more thought to clean that up IMHO. Let's keep the fingers crossed that a small change is possible for this upgrade. The most confusing thing is that it was putting the right dependencies in the Quarkus distribution of Keycloak, but used the wrong library in some module's JUnit tests. Definitely something to clean up in a follow-up PR. |
|
Model tests fail with
due to https://issues.redhat.com/browse/ISPN-15036. I'm considering this issue now blocked until another Infinispan micro release. Needing some help from @mhajas / @martin-kanis, possibly @pruivo / @ryanemerson to apply a possible workaround if we decide we don't want to wait. |
ae9b2f6 to
fa8be4e
Compare
d9f78d2 to
18b38e3
Compare
|
Waiting for files to appear on Maven central |
18b38e3 to
c7ea25b
Compare
This is actually related to our recent discussion in #21363. I can imagine there might be some conflicts between deps coming from Quarkus BOM and deps coming from ISPN BOM. Question also is productization. I still do think that ultimately we should align with Quarkus to avoid conflicts but I understand we need to adopt ISPN releases faster. |
vmuzikar
left a comment
There was a problem hiding this comment.
Changes LGTM from cloud native perspective.
c7ea25b to
2a47d7d
Compare
Closes #21564