[Bug 259607] prometheus_sysctl_exporter: Need better encoding support for sysctl OIDs
Date: Sat, 20 Aug 2022 01:11:09 UTC
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=259607
--- Comment #6 from commit-hook@FreeBSD.org ---
A commit in branch stable/12 references this bug:
URL:
https://cgit.FreeBSD.org/src/commit/?id=b524667411e93d1e828bbaca03d96bb8637a5357
commit b524667411e93d1e828bbaca03d96bb8637a5357
Author: Alan Somers <asomers@FreeBSD.org>
AuthorDate: 2022-04-18 21:29:37 +0000
Commit: Alan Somers <asomers@FreeBSD.org>
CommitDate: 2022-08-20 01:03:31 +0000
prometheus_sysctl_exporter: fix metric aliasing
When exporting sysctls to Prometheus, the exporter replaces "." with
"_". This caused several metrics to alias, confusing the Prometheus
server. Fix it by:
* Renaming the "tcp_log_bucket" UMA zone to "tcp_log_id_bucket". Also,
rename "tcp_log_node" to "tcp_log_id_node" for consistency.
* Not exporting sysctls with "(LEGACY)" in the description. That is
used by ZFS sysctls that have been replaced by others, many of which
alias to the same Prometheus metric name (like "vfs.zfs.arc_max" and
"vfs.zfs.arc.max").
PR: 259607
Reported by: delphij
Sponsored by: Axcient
Reviewed by: delphij,rew,thj
Differential Revision: https://reviews.freebsd.org/D34952
(cherry picked from commit 8c47d8f53854825d8e8591ccd06e32b2c798f81c)
sys/netinet/tcp_log_buf.c | 33 +++++++++++-----------
.../prometheus_sysctl_exporter.c | 11 ++++++--
2 files changed, 26 insertions(+), 18 deletions(-)
--
You are receiving this mail because:
You are on the CC list for the bug.