test/apisix
1
0
Fork 0
mirror of https://gitee.com/iresty/apisix.git synced 2024-12-14 00:41:35 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

chore: should improve the prometheus metric naming (#2073)

Browse Source 
By looking at name like `apisix_bandwidth` or `apisix_http_latency`,
it is hard to guess their units. Also, it is not so clear for where does
the overhead come from for `apisix_http_overhead` at the first glance.
This commit is contained in:
罗泽轩 2020-08-18 20:42:05 +08:00 committed by GitHub
parent cf882bda1e
commit 36bb0a8d3e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
apisix/plugins/prometheus/exporter.lua
View File

@ -69,6 +69,14 @@ function _M.init()
clear_tab(metrics)
-- Newly added metrics should follow the naming best pratices described in
-- https://prometheus.io/docs/practices/naming/#metric-names
-- For example,
-- 1. Add unit as the suffix
-- 2. Add `_total` as the suffix if the metric type is counter
-- 3. Use base unit
-- We keep the old metric names for the compatibility.
-- across all services
prometheus = base_prometheus.init("prometheus-metrics", "apisix_")
metrics.connections = prometheus:gauge("nginx_http_current_connections",
@ -93,11 +101,12 @@ function _M.init()
{"code", "route", "service", "node"})
metrics.latency = prometheus:histogram("http_latency",
"HTTP request latency per service in APISIX",
"HTTP request latency in milliseconds per service in APISIX",
{"type", "service", "node"}, DEFAULT_BUCKETS)
metrics.overhead = prometheus:histogram("http_overhead",
"HTTP request overhead per service in APISIX",
"HTTP request overhead added by APISIX in milliseconds per service " ..
"in APISIX",
{"type", "service", "node"}, DEFAULT_BUCKETS)
metrics.bandwidth = prometheus:counter("bandwidth",