Page MenuHomeDevCentral

Restore metrics collector in management plugin
ClosedPublic

Authored by dereckson on May 30 2023, 19:24.
Tags
None
Referenced Files
F2825547: D3158.id8059.diff
Thu, Apr 18, 16:39
F2825546: D3158.id8058.diff
Thu, Apr 18, 16:39
F2825545: D3158.id8057.diff
Thu, Apr 18, 16:39
F2825544: D3158.id.diff
Thu, Apr 18, 16:38
F2823407: D3158.diff
Thu, Apr 18, 01:48
Unknown Object (File)
Sun, Apr 14, 23:37
Unknown Object (File)
Sun, Apr 14, 23:31
Unknown Object (File)
Sun, Apr 14, 12:37
Subscribers
None

Details

Summary

The base image disable explicitly fot the management plugin
the metrics collector and associated UI to avoid redundancy
with Prometheus metrics.

Metrics in the UI for queues is something useful to debug
in an unified place, even with dashboards on the side.

For a discussion on metrics collection, see also T1623 and T1633.

Test Plan

Deploy for white-rabbit and orange-rabbit

Diff Detail

Repository
rDRABBITMQ RabbitMQ Docker image
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson created this revision.

Still "Stats in management UI are disabled on this node"

This revision is now accepted and ready to land.May 30 2023, 19:27
dereckson added inline comments.
Dockerfile
7

with BuildKit I'm not sure that's a good idea to remove a file

Hmmm they prioritized the configuration to load in a specific order, so now it's 20-management_agent.disable_metrics_collector.conf

They change again the config, BOOM we lose it again.

Regardless of the previous remark on the lack of stability of upstream image, we can currently proceed.

Update configuration file name

This revision is now accepted and ready to land.May 30 2023, 19:34