-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Add Jolokia #450
Add Jolokia #450
Conversation
add jolokia
Fix jolokia path
Thanks for the PR, however - I don't think we're interested in adding jolokia to the project at this time as it is outside the core Apache Kafka offerings (which is what this image packages). It also seems a fairly inactive project judging by the github repo: https://github.com/rhuss/jolokia I'll leave the PR open for a bit to allow for further comment, but I suspect it's easier just to mount the JAR and configure |
Thanks for the review. |
Fix kafkacat to version 5.0.0 image (wurstmeister#452)
+1 |
Jolokia is active, exists commits in the last month. |
Thanks @alvarolmedo - but my comments still stand. This is not a universal solution - and not something that we want to maintain at this point in time. |
Jolikia is very famous project to monitor JVM application. About Kafka, we use elastic-beat https://www.elastic.co/guide/en/beats/metricbeat/current/metricbeat-module-kafka.html. As Docker build principle, all Jolikia related stuff should come at the end of the Dockerfile. What about to create a tag suffit such as "with-jolikia" ? |
Or maybe, a simple wiki post, or a Dockerfile example such as :
|
Add Jilokia HTTP Server to monitoring metric of kafka