-
Notifications
You must be signed in to change notification settings - Fork 159
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
Liveness/readiness endpoints #108
Comments
yeah, something like a health-check point would be great |
does the described implementation make sense? i checked the code and couldn't find anything "more suitable" 🤔 |
Sure do a PR and we can discuss
—
Stephane Maarek
Book a call with us
https://www.conduktor.io/
…--- original message ---
On 7 June 2021 at 8:53 AM WEST ***@***.*** wrote:
does the described implementation make sense? i checked the code and couldn't find anything "more suitable" 🤔
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
--- end of original message ---
|
eroznik
added a commit
to eroznik/kafka-security-manager
that referenced
this issue
Jun 25, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hello,
Just wondering if this would be a good addition to the project?
Liveness could probably check that ACLs were synced successfully in the last configured sync interval. Readiness would be no-op, just return a 200.. since there is no public API(for now).
Thoughts?
The text was updated successfully, but these errors were encountered: