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

Missing metrics for channels associated with Emitters #2332

Closed
abutch3r opened this issue Oct 18, 2023 · 1 comment · Fixed by #2342
Closed

Missing metrics for channels associated with Emitters #2332

abutch3r opened this issue Oct 18, 2023 · 1 comment · Fixed by #2342
Labels
bug Something isn't working

Comments

@abutch3r
Copy link

We have created an emitter that is sending messages to a kafka deployment via our own connector and we are pulling messages from Kafka on a separate channel. When we look at the metrics that are being produced. We have a single entry for mp_messaging that is for the incoming channel. But we are not seeing an entry at all the emitter channel.

When reviewing the specs for metrics. This part: metrics are produced for each channel declared by the application. There is no statement about how a channel associated with an emitter.

Our scenario can be seen as the following simple flows:
Emitter -> channelA -> Kafka connector -> Kafka
Kafka -> Kafka Connector -> channelB -> @Incoming

We are seeing metrics only for channelB.

When we used an @Outgoing and @Incoming we got metrics to both. so would indicate that it is the fact we had the xhannel associated with an Emitter that meant we were not seeing any metrics.

@cescoffier cescoffier added the bug Something isn't working label Oct 18, 2023
@cescoffier
Copy link
Contributor

That's a bug, we should report these messages too.

@cescoffier cescoffier self-assigned this Oct 23, 2023
cescoffier added a commit to cescoffier/smallrye-reactive-messaging that referenced this issue Oct 23, 2023
@cescoffier cescoffier removed their assignment Oct 23, 2023
ozangunalp added a commit to ozangunalp/smallrye-reactive-messaging that referenced this issue Oct 23, 2023
ozangunalp added a commit to ozangunalp/smallrye-reactive-messaging that referenced this issue Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants