-
Notifications
You must be signed in to change notification settings - Fork 867
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 kafka client metrics to the javaagent instrumentation #6533
Add kafka client metrics to the javaagent instrumentation #6533
Conversation
Closing and re-opening to trigger latest dep tests |
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.
@mateuszrzeszutek are these metrics on-by-default?
Yes, enabled by default. |
oh, sorry, I meant to bring this up in SIG meeting today, I was unsure whether this should be on by default or not, since these are not spec'd metrics, but I guess we're considering it to be similar to micrometer (which is enabled by default)? one other related concern is that I didn't see an easy way to disable them (without defining a metrics view)? |
I think it probably makes sense to have them enabled by default, since it's a bridge/shim-like instrumentation, like micrometer. Making it easy to turn this off sounds good though, I'll add a property for disabling them 👍 |
…etry#6533) * Add kafka client metrics to the javaagent instrumentation * Don't override user-set metrics reporter * Add kafka metric reporter config property
…etry#6533) * Add kafka client metrics to the javaagent instrumentation * Don't override user-set metrics reporter * Add kafka metric reporter config property
…etry#6533) * Add kafka client metrics to the javaagent instrumentation * Don't override user-set metrics reporter * Add kafka metric reporter config property
Resolves #5941