Skip to content

[opentelemetry-demo]: add kafkametricsreceiver#2125

Merged
TylerHelmuth merged 9 commits into
open-telemetry:mainfrom
Red-GV:kafka-exporter
Apr 10, 2026
Merged

[opentelemetry-demo]: add kafkametricsreceiver#2125
TylerHelmuth merged 9 commits into
open-telemetry:mainfrom
Red-GV:kafka-exporter

Conversation

@Red-GV
Copy link
Copy Markdown
Contributor

@Red-GV Red-GV commented Mar 25, 2026

This PR adds the Kafka Prometheus exporter in order to retrieve more metrics from the Kafka deployment.
This PR adds the Kafka Metrics Receiver to the otel collector configuration to export metrics from Kafka.

@Red-GV Red-GV requested review from a team, julianocosta89 and puckpuck as code owners March 25, 2026 14:29
@Red-GV Red-GV marked this pull request as draft March 25, 2026 14:57
@Red-GV Red-GV marked this pull request as ready for review March 25, 2026 18:34
@Red-GV Red-GV changed the title feat: add Kafka Prometheus exporter feat: add kafkametricsreceiver Mar 30, 2026
@Red-GV Red-GV changed the title feat: add kafkametricsreceiver [opentelemetry-demo]: add kafkametricsreceiver Mar 30, 2026
Copy link
Copy Markdown
Member

@julianocosta89 julianocosta89 left a comment

Choose a reason for hiding this comment

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

LGTM

@julianocosta89
Copy link
Copy Markdown
Member

@TylerHelmuth can we get that one merged?

@cyrille-leclerc
Copy link
Copy Markdown
Member

I think that we have a bug on the docker-compose deployment, I propose to fix it before merging the helm chart version.
open-telemetry/opentelemetry-demo#3158 (comment)

@Red-GV
Copy link
Copy Markdown
Contributor Author

Red-GV commented Apr 9, 2026

@cyrille-leclerc @julianocosta89 I believe the fix was done here: open-telemetry/opentelemetry-demo#3219

So is this guy now good to go in again?

Copy link
Copy Markdown
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.

I added a small little nit, but approve this either way.

Comment thread charts/opentelemetry-demo/values.yaml Outdated
Red-GV and others added 8 commits April 10, 2026 09:44
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
Co-authored-by: Pierre Tessier <pierre@pierretessier.com>
Signed-off-by: Gerard Vanloo <gerard.vanloo@ibm.com>
@TylerHelmuth TylerHelmuth merged commit 0dcc605 into open-telemetry:main Apr 10, 2026
4 checks passed
@Red-GV Red-GV deleted the kafka-exporter branch May 6, 2026 15:47
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.

5 participants