diff options
author | 2020-03-31 20:03:18 +0200 | |
---|---|---|
committer | 2020-03-31 20:03:18 +0200 | |
commit | a29a594e5c2283496af0577bd5c64dc2b439dee6 (patch) | |
tree | d5d13667c459328221d233ba78220fc961306cc6 /plugin/acl/metrics.go | |
parent | e2ece87035f67bbf6012b9feb8b7e79c9dba1353 (diff) | |
download | coredns-a29a594e5c2283496af0577bd5c64dc2b439dee6.tar.gz coredns-a29a594e5c2283496af0577bd5c64dc2b439dee6.tar.zst coredns-a29a594e5c2283496af0577bd5c64dc2b439dee6.zip |
Update more metrics names (#3799)
The subsystem of a metric must be it's plugin name. "dns" is reserved
for the core. Use a const convention to avoid copy-paste mistakes in
naming.
Fix dns64 and acl plugin.
Signed-off-by: Ben Kochie <superq@gmail.com>
Diffstat (limited to 'plugin/acl/metrics.go')
-rw-r--r-- | plugin/acl/metrics.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/plugin/acl/metrics.go b/plugin/acl/metrics.go index 1f235b85b..719f24675 100644 --- a/plugin/acl/metrics.go +++ b/plugin/acl/metrics.go @@ -10,15 +10,15 @@ var ( // RequestBlockCount is the number of DNS requests being blocked. RequestBlockCount = prometheus.NewCounterVec(prometheus.CounterOpts{ Namespace: plugin.Namespace, - Subsystem: "dns", - Name: "acl_blocked_requests_total", + Subsystem: pluginName, + Name: "blocked_requests_total", Help: "Counter of DNS requests being blocked.", }, []string{"server", "zone"}) // RequestAllowCount is the number of DNS requests being Allowed. RequestAllowCount = prometheus.NewCounterVec(prometheus.CounterOpts{ Namespace: plugin.Namespace, - Subsystem: "dns", - Name: "acl_allowed_requests_total", + Subsystem: pluginName, + Name: "allowed_requests_total", Help: "Counter of DNS requests being allowed.", }, []string{"server"}) ) |