Skip to content

Report RabbitMQ messages on the client queue as metric - #140

Merged
Hilbrand merged 3 commits into
aerius:mainfrom
Hilbrand:client-rmq-metrics
Oct 6, 2026
Merged

Hilbrand merged 3 commits into
aerius:mainfrom
Hilbrand:client-rmq-metrics

Conversation

@Hilbrand

Copy link
Copy Markdown
Member

Using a specific metric reporter in the taskmanager allows for easier filtering on client queue metrics in the dashboard.

Using a specific metric reporter in the taskmanager allows for easier filtering on client queue metrics in the dashboard.
@SerhatG

SerhatG commented Sep 29, 2026

Copy link
Copy Markdown
Member

I had some small questions and was searching for the telemetry doc but that hasn't been updated. Can you add that part? Probably all my questions will be answered after reading that. :)

@RonniLin

RonniLin commented Oct 5, 2026

Copy link
Copy Markdown

Maybe I missed part of the discussion, but why gather information about RabbitMQ indirectly when RabbitMQ exposes this information also? Is it because it is not reachable?

Also removed unused file BrokerConnectionEnum.
@Hilbrand

Hilbrand commented Oct 5, 2026

Copy link
Copy Markdown
Member Author

@RonniLin Yes RabbitMQ also provides this information. But with this we could opt not to gather the RabbitMQ metrics at all from RabbitMQ. But more importantly by providing the metrics via the TaskManager we add some additional information, which makes it simpler to process the metrics in the dashboard.

@Hilbrand
Hilbrand requested a review from BertScholten October 5, 2026 10:54

@BertScholten BertScholten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Hilbrand
Hilbrand merged commit c568367 into aerius:main Oct 6, 2026
1 check passed
@Hilbrand
Hilbrand deleted the client-rmq-metrics branch October 6, 2026 06:23
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.

4 participants