Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reapply "Upgrade to Infinispan 15.0.3.Final" #813

Merged
merged 1 commit into from
May 14, 2024

Conversation

ryanemerson
Copy link
Contributor

The Operator PR finally got merged, so we should be good to reapply my original fix: redhat-openshift-ecosystem/community-operators-prod#4512

@ryanemerson ryanemerson requested a review from mhajas May 14, 2024 08:17
Copy link
Contributor

@mhajas mhajas left a comment

Choose a reason for hiding this comment

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

Thank you @ryanemerson, I can confirm today's clusters already contain the correct version of ISPN operators == 2.4.1.

I believe it is safe to merge this now even when today's kcb deployment still contains 14.0.27 ISPN.

@ryanemerson
Copy link
Contributor Author

I believe it is safe to merge this now even when today's kcb deployment still contains 14.0.27 ISPN.

+1. If we require 15.0.x then it's just a case of redeploying Infinispan/Keycloak with the latest code on our clusters.

@@ -22,8 +22,6 @@ data:
cacheContainer:
metrics:
namesAsTags: true
gauges: true
histograms: {{ .Values.metrics.histograms }}
Copy link
Contributor

Choose a reason for hiding this comment

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

any reason to remove this line?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have no idea why I did that. I should have just removed gauges as we're using the default value. Reverted

This reverts commit 1be39e9.

Signed-off-by: Ryan Emerson <remerson@redhat.com>
@mhajas mhajas merged commit c56d363 into keycloak:main May 14, 2024
2 checks passed
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.

3 participants