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

Update the Prometheus JMX Collector and Client #909

Merged

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented May 31, 2024

This PR updates the Prometheus JMX Collector to 1.0.1. This new version has many changes to how it treats the metrics with some keywords such as _total. This required an update to the metrics configuration YAMl to maintain the compatibility of the produced metrics (essentially, the counters using _total suffix need to be mapped to counters in the rules).

In addition, the Prometheus Simple Client is now archived and not anymore supported by the Prometheus JMX Exporter. So it also moves the bridge to use the current Prometheus Client.

Signed-off-by: Jakub Scholz <www@scholzj.com>
@scholzj scholzj added this to the 0.30.0 milestone May 31, 2024
@scholzj scholzj requested a review from ppatierno June 2, 2024 11:15
@scholzj scholzj marked this pull request as ready for review June 2, 2024 11:15
Copy link
Member

@ppatierno ppatierno left a comment

Choose a reason for hiding this comment

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

LGTM. I assume you checked that metrics are coming as expected.

@scholzj scholzj merged commit 46e8df1 into strimzi:main Jun 4, 2024
13 checks passed
@scholzj scholzj deleted the update-prometheus-jmx-collector-to-1.0.1 branch June 4, 2024 08:30
antonio-pedro99 pushed a commit to antonio-pedro99/strimzi-kafka-bridge that referenced this pull request Jul 22, 2024
Signed-off-by: Jakub Scholz <www@scholzj.com>
Signed-off-by: Antonio Pedro <tonio.pedro99@gmail.com>
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.

2 participants