Page MenuHomeDevCentral

Restore metrics collector in management plugin
ClosedPublic

Authored by dereckson on May 30 2023, 19:24.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 04:39
Unknown Object (File)
Sat, Dec 21, 04:17
Unknown Object (File)
Tue, Dec 17, 16:44
Unknown Object (File)
Thu, Dec 12, 06:20
Unknown Object (File)
Wed, Dec 11, 02:44
Unknown Object (File)
Wed, Dec 11, 02:44
Unknown Object (File)
Wed, Dec 11, 02:44
Unknown Object (File)
Sun, Dec 8, 14:44
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
No Lint Coverage
Unit
No Test Coverage
Branch
main
Build Status
Buildable 4981
Build 5262: arc lint + arc unit

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