Skip to content
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 OTel java agent with JMX Metric Insights to kafka #654

Merged
merged 11 commits into from
Dec 24, 2022

Conversation

svrnm
Copy link
Member

@svrnm svrnm commented Dec 21, 2022

Changes

This PR adds the java agent to the kafka broker. While this will not provide tracing, it will use JMX Metric Insights generate metrics and export them to prometheus:

Screenshot 2022-12-21 at 15 27 02

Screenshot 2022-12-21 at 19 03 26

Additional info

We plan to have a blog post about JMX Metric Insights on opentelemetry.io eventually, so calling out that the demo is making use of it would be a nice feature:-)

Merge Requirements

  • CHANGELOG.md updated to document new feature additions
  • Appropriate documentation updates in the docs folder

Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
@svrnm svrnm requested a review from a team as a code owner December 21, 2022 18:06
Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
Signed-off-by: svrnm <neumanns@cisco.com>
Copy link
Member

@mviitane mviitane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Runs fine. Nice to get the additional Kafka metrics. Kafka seems to consume a massive amount of memory, but I guess there's no easy solution for that.

Copy link
Contributor

@puckpuck puckpuck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fantastic. TYVM!

@puckpuck puckpuck merged commit e4d34c7 into open-telemetry:main Dec 24, 2022
@puckpuck puckpuck added the helm-update-required Requires an update to the Helm chart when released label Dec 24, 2022
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
…#654)

* Add otel agent with JMX metric insights to kafka

Signed-off-by: svrnm <neumanns@cisco.com>

* Increase memory limit for kafka

* Remove pb files

Signed-off-by: svrnm <neumanns@cisco.com>

* Add docs for kafka service

Signed-off-by: svrnm <neumanns@cisco.com>

* Revert removal of demo.proto

Signed-off-by: svrnm <neumanns@cisco.com>

* Update CHANGELOG

Signed-off-by: svrnm <neumanns@cisco.com>

* Fix markdownlint error

Signed-off-by: svrnm <neumanns@cisco.com>

Signed-off-by: svrnm <neumanns@cisco.com>
Co-authored-by: Carter Socha <43380952+cartersocha@users.noreply.github.com>
Co-authored-by: Pierre Tessier <pierre@pierretessier.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
helm-update-required Requires an update to the Helm chart when released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants