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

[WFCORE-7058]: CachingRealm should start lazily in admin only mode. #6245

Merged
merged 1 commit into from
Nov 20, 2024

Conversation

ehsavoie
Copy link
Contributor

  • updating the start mode to LAZY if the server is in admin-mode
  • adding a proper error message if the caching realm service is not UP when the user tries to clear the cache.

Issue: https://issues.redhat.com/browse/WFCORE-7058

 * updating the start mode to LAZY if the server is in admin-mode
 * adding a proper error message if the caching realm service is not UP
   when the user tries to clear the cache.

Issue: https://issues.redhat.com/browse/WFCORE-7058

Signed-off-by: Emmanuel Hugonnet <ehugonne@redhat.com>
Copy link
Contributor

@darranl darranl 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 @ehsavoie

@wildfly-ci
Copy link

Core -> WildFly Preview Integration Build 14116 outcome was UNKNOWN using a merge of 1e8448c
Summary: Canceled (Error while applying patch; cannot find commit bc33e8c in the https://github.com/wildfly/wildfly-core.git repository, possible reason: refs/pull/6245/merge branch was updated and the commit selected for the ... Build time: 00:00:13

@yersan yersan added the ready-for-merge This PR is ready to be merged and fulfills all requirements label Nov 19, 2024
@yersan yersan merged commit 9143e98 into wildfly:main Nov 20, 2024
13 checks passed
@yersan
Copy link
Collaborator

yersan commented Nov 20, 2024

Thanks @ehsavoie @darranl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-for-merge This PR is ready to be merged and fulfills all requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants