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

custom jmxfetch bean to track number of ingested beans, attributes, and metrics #472

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

carlosroman
Copy link
Contributor

This pr is part of a multi stage process to increase observability within jmxfetch. Another pr adds a check that uses proccess_regex to attach to jmxfetch's jvm in order to collect mbean data from our own application. In this pr I have added jmx MBeans that are registered for each instance of jmxfetch and track the number of beans, metrics, and attributes that each instance collects and exposes this under the jmxFetch domain where the check from my other pr will be able to pick up this data in addition to general jmfetch metrics.

…nd metrics (#453)

* added bean that tracks number of ingested beans

* restructured bean creation

* reverted compiler version

* updated some formatting

* cleaned up some formatting

* simplified quoting process

* removed constructor argument from jmxFetchTelem

* fixed formatting

* updated spacing

* cleaned up debug logs

* fixed long lines

* updated naming

* removed java object debug
@carlosroman carlosroman requested a review from a team as a code owner August 10, 2023 16:39
@scottopell scottopell merged commit e30e615 into master Aug 10, 2023
@carlosroman carlosroman deleted the carlosroman/return-3c03064-commit branch October 10, 2023 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants