-
Notifications
You must be signed in to change notification settings - Fork 851
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 jmx exporter #2522
Comments
I'm not opposed to this, but it's unlikely that the maintainers of this project will have time to do this in the near future. Would you be willing to contribute code for this? |
i will discuss internally and get back. |
@maneeshbhunwal123 Some time has passed, any more thoughts on this? Writing to JMX feels like an unusual use case to me. Were you thinking about writing to a remote MBean or local to the same JVM? In any case, would be interested to hear what your use case is, and if this request is still relevant/meaningful. Thanks. |
we are publishing to local mbean. there is a sidecar process running which keep on reading values from mbean, and send it to remote server. Where it can be plotted using grafana. |
Interesting and unusual use case, thanks for sharing. Are you still considering writing an exporter for this? Just curious. |
for now we used codehale metric registory because both aggregation and jmx reporter was not there at that point of time. I can not commit to writing exporter at this time. But i believe it should not take too much time, as i have seen codehale jmx repoter code, and it was straight foreward. |
A JMX metric exporter is outside of the defined project scope. If you still have interest in this, consider contributing it to |
Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Send opentelemetery collected metrics to jmx
Describe alternatives you've considered
Additional context
There are few exporters written already but i dont see support for writing to jmx mean. Can we add a exporter for this as well?
The text was updated successfully, but these errors were encountered: