The maas_id label for prometheus metrics should use the server uuid
Bug #1828579 reported by
Alberto Donato
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
Medium
|
Alberto Donato | ||
Bug Description
The maas_id label in prometheus metrics should use the MAAS uuid (from the Config), since this value is the same across multiple machines of the same maas deployment.
Related branches
~ack/maas:prometheus-maas-id-use-uuid
- Blake Rouse (community): Approve
-
Diff: 297 lines (+61/-19)11 files modifiedsrc/maasserver/eventloop.py (+10/-0)
src/maasserver/prometheus/stats.py (+3/-5)
src/maasserver/prometheus/tests/test_stats.py (+1/-1)
src/maasserver/rpc/regionservice.py (+7/-4)
src/maasserver/rpc/tests/test_regionservice.py (+5/-1)
src/maasserver/testing/eventloop.py (+6/-0)
src/maasserver/tests/test_eventloop.py (+2/-2)
src/provisioningserver/prometheus/metrics.py (+10/-5)
src/provisioningserver/rpc/clusterservice.py (+2/-1)
src/provisioningserver/rpc/region.py (+1/-0)
src/provisioningserver/rpc/tests/test_clusterservice.py (+14/-0)
| Changed in maas: | |
| milestone: | 2.6.0rc1 → 2.6.0beta4 |
| Changed in maas: | |
| milestone: | 2.6.0beta4 → 2.6.0rc1 |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
