-
- Downloads
Add a response-time metric (#3948)
* Refactor existing metrics support into MetricHelper This completes the refactor done at https://github.com/badges/shields/pull/3662#issuecomment-509011229 in anticipation of adding more metrics support, such as response size of an upstream service, or response time. * Clean up * Renames * Add response time metrics This adds around 30 new metrics to cover response times at a fairly granular level. We may be able to shrink the number of buckets with time, though I think using 30 metrics is probably okay given that I think may become our most important metric. * Fix
Showing
- core/base-service/base-non-memory-caching.js 8 additions, 4 deletionscore/base-service/base-non-memory-caching.js
- core/base-service/base-static.js 8 additions, 4 deletionscore/base-service/base-static.js
- core/base-service/base.js 8 additions, 16 deletionscore/base-service/base.js
- core/base-service/metric-helper.js 45 additions, 0 deletionscore/base-service/metric-helper.js
- core/base-service/redirector.js 8 additions, 4 deletionscore/base-service/redirector.js
- core/server/prometheus-metrics.js 61 additions, 6 deletionscore/server/prometheus-metrics.js
- core/server/server.js 8 additions, 9 deletionscore/server/server.js
Loading
Please register or sign in to comment