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

[Enhancement]: Explicitly add JmxReporter to metric.reporters when JMX Exporter is enabled #11019

Open
fvaleri opened this issue Jan 8, 2025 · 2 comments
Assignees

Comments

@fvaleri
Copy link
Contributor

fvaleri commented Jan 8, 2025

Related problem

In Kafka 4.0.0, the org.apache.kafka.common.metrics.JmxReporter will still be enabled by default, but the user will need to explicitly add this plugin to the metric.reporters property when setting any other custom metrics plugin. This means that, with the current Cluster Operator reconciliation logic, the JMX Prometheus Exporter would not work when Cruise Control is also enabled.

Suggested solution

Update the Cluster Operator Kafka cluster configuration reconciliation to explicitly add the org.apache.kafka.common.metrics.JmxReporter to metric.reporters when JMX Prometheus Exporter is enabled.

Alternatives

No response

Additional context

No response

@OwenCorrigan76
Copy link
Contributor

I'm interested in woking on this. I'll discuss with @fvaleri after the community call on Jan 9th.

@im-konge
Copy link
Member

im-konge commented Jan 9, 2025

Triaged on 9.1.2025: This makes sense to have fixed and it seems that @OwenCorrigan76 already volunteered for fixing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants