Skip to content

Conversation

@dglozano
Copy link
Contributor

@dglozano dglozano commented Jul 4, 2024

In the latest Keycloak version (v25.0.1) the cache options are not build options anymore. They now have to be provided during runtime.

Closes #31050

@dglozano dglozano requested review from a team as code owners July 4, 2024 08:27
@ahus1 ahus1 self-assigned this Jul 4, 2024
@ahus1
Copy link
Contributor

ahus1 commented Jul 4, 2024

Thank you for this change to the docs, which is correct.

To be able to merge your contribution, your commit needs to be signed off as outlined in the CONTRIBUTING.md. Please add a line

Signed-off-by: Author Name <[email protected]>

to your commit. Thanks!

@dglozano
Copy link
Contributor Author

dglozano commented Jul 4, 2024

Sorry about that 😅 Done now 🙌

In the latest Keycloak version (v25.0.1) the cache options are not build options anymore. They now have to be provided during runtime.

Signed-off-by: Diego Garcia Lozano <[email protected]>
Copy link
Contributor

@ahus1 ahus1 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 for the change!

@ahus1
Copy link
Contributor

ahus1 commented Jul 4, 2024

@dglozano - out of curiosity: Did you make this change using the GitHub web UI? I found a link how we as a project can enforce the sign-off to be present automatically. I hope to update it later this month.

@ahus1 ahus1 merged commit 2ff0d4e into keycloak:main Jul 4, 2024
@dglozano
Copy link
Contributor Author

dglozano commented Jul 4, 2024

@ahus1 Yes I did it using the Github web UI 😄 To fix it I amended the commit locally and forced push

@dglozano dglozano deleted the patch-1 branch July 4, 2024 12:30
@ahus1
Copy link
Contributor

ahus1 commented Jul 4, 2024

Yes I did it using the Github web UI

Thank you for confirming my suspicion here. I hope to have everything in place for users of the GitHub web UI later this month.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Caching docs should name parameter runtime parameters, not build parameters

2 participants