-
Notifications
You must be signed in to change notification settings - Fork 81
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
Metrics lacks clear documentation #147
Comments
Agree. I found myself scratching my head numerous time of what they actually mean. For example in documentation:
There is no information about exposed metric names or what they mean. Then I'm left to wonder in google and rely on something like this: If somebody would at least point towards the direction where to look up in code, I could offer to contribute to documentation. |
I think there are perhaps two separate issues here: documentation of metrics instrumentation maintained by the Micrometer team, and discovery of metrics instrumentation maintained by the third-party library/tool being instrumented. Generally the latter is in that library/tool's documentation. RabbitMQ metrics are not part of Micrometer core. It looks like the metrics mentioned are provided by the RabbitMQ Java client metrics and the Spring AMQP project. Any library can support metrics via Micrometer; trying to document something not maintained by the Micrometer team here would quickly be out-of-date and inaccurate. We could have a section in our documentation linking to popular libraries' documentation of Micrometer support, though. Is that something that would be useful to users? If you think so, we could focus this issue on that and open a separate issue for improving the documentation of metrics maintained in Micrometer core. |
Agree, micrometer core metrics documentation could be improved to know which metrics belongs to micrometer core, so it would be clear that others are third party metrics. |
If you could point where to look for metrics supported by Micrometer core, I could look into during the weekend.
Agree. Maybe both of these sections: Micrometer core metrics and 3rd party support library list could next to each other? |
I was discussing options with @jonatan-ivanov for this, and I was wondering if we can't use the JavaDocs site for this somehow, as long as we make sure the JavaDoc comments used to generate it are up-to-date and informative enough. I was hoping we could essentially point to the We could, of course, manually make documentation of all the provided metrics, but this would require a lot of work to keep up-to-date, and a significant amount of the information is version-dependent. The reference documentation currently is not versioned. Since the JavaDocs sites are versioned, that's another reason I would like to leverage that. Note: we currently have a limited selection of documentation of metrics we provide: JVM, Cache, OkHttpClient. We have a lot more undocumented. Some reorganization would probably be in order. Perhaps if we update the package descriptions (JavaDocs) to point readers at which classes to look for which metrics, it would be more usable. I noticed while looking into this that currently we have the license header as a JavaDoc, which was probably a mistake. As for the third-party libraries with Micrometer metrics, we would have to collect these and link to them. |
Looking back at the open issues here, it looks like this is a duplicate of micrometer-metrics/micrometer#4626 and #57. Let's continue the conversation in those respective issues. |
For example metrics related RabbitMq:
In my application I have both subscribers and publishers to RabbitMQ and it confuses me which metric is for my published messages and which is for consumed.
The text was updated successfully, but these errors were encountered: