-
Notifications
You must be signed in to change notification settings - Fork 7.9k
New operator failing on health checks #30709
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
Conversation
|
@vmuzikar @keycloak/cloud-native I believe it's sufficient to resolve the parent issue. Are you okay with that, or do you have any suggestions? |
|
@vmuzikar Might be better now. PTAL I added more general info to the advanced configuration and kept the more specific message related to the mgmt interface to the related subsection about mgmt interface. |
vmuzikar
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mabartos Thank you for this docs enhancement!
Added some suggestions to the wording but I'm not a native speaker so I don't consider it blocking. :D
Closes keycloak#30355 Co-authored-by: Václav Muzikář <[email protected]> Signed-off-by: Martin Bartoš <[email protected]>
|
@vmuzikar Thanks! Looks better :)) Updated |
Closes #30355