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

Enable External Infinispan Feature #858

Merged

Conversation

pruivo
Copy link
Contributor

@pruivo pruivo commented Jun 20, 2024

  • New option to enable external infinispan feature (disabled by default)
  • Added to the nightly runs to get some numbers

It has been a while since I looked at those GH action files. Please double-check if I did it right.
I tested a deployment with KC_EXTERNAL_INFINISPAN=true and confirmed no embedded caches were present in Keycloak Servers.

* New option to enable external infinispan feature (disabled by default)
* Added to the nightly runs to get some numbers

Signed-off-by: Pedro Ruivo <[email protected]>
@ahus1 ahus1 requested a review from ryanemerson June 24, 2024 12:24
@ahus1 ahus1 marked this pull request as draft June 24, 2024 12:25
@pruivo pruivo marked this pull request as ready for review June 25, 2024 10:00
@pruivo
Copy link
Contributor Author

pruivo commented Jun 25, 2024

I didn't add the functional tests to the GitHub actions. I forgot. It can be merged and hope it does not break horreum :)

cc @ahus1 @ryanemerson

createCluster: false
secrets: inherit

run-scaling-benchmark-with-external-infinispan:
Copy link
Contributor

Choose a reason for hiding this comment

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

We're going to need a way to differentiate benchmark runs executed with "remote-cache" and normal A/P. This is also required for Active/Active. I've created #862 to address this in a separate PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

It appears I'm blind, this just applies to A/A then 🎉

@ryanemerson ryanemerson merged commit c3d2b23 into keycloak:main Jun 25, 2024
3 checks passed
@ryanemerson
Copy link
Contributor

Thanks @pruivo

@pruivo pruivo deleted the t_enable_external_infinispa_feature branch June 25, 2024 13:52
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.

None yet

2 participants