-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Feature/jolokia plugin #337
Conversation
2470bc0
to
33b02cc
Compare
33b02cc
to
abbdc3f
Compare
@saiello can you please write a README for this plugin? you can use the CPU readme as an example: https://github.com/influxdb/telegraf/blob/master/plugins/system/CPU_README.md |
@saiello, I'm not super familiar with JMX/jolokia/MBeans, but it seems like this information is more generically called an |
return ` | ||
context = "/jolokia/read" | ||
|
||
[jolokia.tags] |
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.
this should be space-indented
@saiello nevermind about renaming the plugin, I get it now, |
Once you write a README and address my comments I'll merge it, thanks a bunch @saiello! |
@sparrc I added a README to jolokia plugin. I hope to have fixed the indentation as well 😁 Thanks |
thank you @saiello! |
I wrote a simple JVM plugin based on jolokia rest api.
I needed to add some new methods in testutil/accumulator.go hence I refactor the oldest to use them.