ambry
ambry copied to clipboard
Add account level metrics for container metrics
Summary
Add account level metric to aggregate its containers' metrics.
Details
Some accounts have lots of containers and it's very hard those those accounts to aggregate all the metrics for 40x, 50x and latency etc. This PR adds an AccountMetrics class and have its ContainerMetrics to record metrics for AccountMetrics.
This PR also adds a configuration to only enable some accounts to have this AccountMetrics as most accounts have very small set of containers that they just don't need extra metrics.
Test
Unit test
Codecov Report
Attention: Patch coverage is 70.17544% with 17 lines in your changes missing coverage. Please review.
Project coverage is 70.26%. Comparing base (
52ba813) to head (a6fb9fc). Report is 128 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #2926 +/- ##
============================================
+ Coverage 64.24% 70.26% +6.01%
- Complexity 10398 11982 +1584
============================================
Files 840 866 +26
Lines 71755 73711 +1956
Branches 8611 8870 +259
============================================
+ Hits 46099 51793 +5694
+ Misses 23004 19212 -3792
- Partials 2652 2706 +54
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.