upstream/opnsense-src
1
0
Fork 0
mirror of https://github.com/opnsense/src.git synced 2026-06-09 08:43:19 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Be a bit more liberal about sysctl naming.

Browse source 
On the systems on which I tested this exporter, I never ran into metrics
that were named in such a way that they couldn't be exported to
Prometheus metrics directly. Now it turns out that on systems with NUMA,
the sysctl tree contains metrics named dev.${driver}.${index}.%domain.
For these metrics, the % in the name is problematic, as Prometheus
doesn't allow this symbol to be used.

Remove the assertions that were originally put in place to prevent the
exporter from generating malformed output and add code to deal with it
accordingly. For metric names, convert any unsupported character to an
underscore. For label values, perform string escaping.

PR:		https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=221035
Reported by:	lifanov@
This commit is contained in:
Ed Schouten 2017-07-29 08:35:07 +00:00
parent 23e148e9a4
commit 4f6a15dd39
1 changed files with 16 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
usr.sbin/prometheus_sysctl_exporter/prometheus_sysctl_exporter.c
View file

@ -1,5 +1,5 @@
/*-
* Copyright (c) 2016 Nuxi, https://nuxi.nl/
* Copyright (c) 2016-2017 Nuxi, https://nuxi.nl/
*
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions
@ -32,6 +32,7 @@ __FBSDID("$FreeBSD$");
#include <sys/sysctl.h>
#include <assert.h>
#include <ctype.h>
#include <err.h>
#include <errno.h>
#include <math.h>
@ -384,11 +385,12 @@ oidname_print(const struct oidname *on, const struct oidformat *of,
label = on->labels;
for (i = 0; i < on->oid.len; ++i) {
if (*label == '\0') {
assert(name[strspn(name,
"abcdefghijklmnopqrstuvwxyz"
"ABCDEFGHIJKLMNOPQRSTUVWXYZ"
"0123456789_")] == '\0');
fprintf(fp, "_%s", name);
fputc('_', fp);
while (*name != '\0') {
/* Map unsupported characters to underscores. */
fputc(isalnum(*name) ? *name : '_', fp);
++name;
}
}
name += strlen(name) + 1;
label += strlen(label) + 1;
@ -404,15 +406,18 @@ oidname_print(const struct oidname *on, const struct oidformat *of,
separator = '{';
for (i = 0; i < on->oid.len; ++i) {
if (*label != '\0') {
assert(name[strspn(name,
"abcdefghijklmnopqrstuvwxyz"
"ABCDEFGHIJKLMNOPQRSTUVWXYZ"
"0123456789_-")] == '\0');
assert(label[strspn(label,
"abcdefghijklmnopqrstuvwxyz"
"ABCDEFGHIJKLMNOPQRSTUVWXYZ"
"0123456789_")] == '\0');
fprintf(fp, "%c%s=\"%s\"", separator, label, name);
fprintf(fp, "%c%s=\"", separator, label);
while (*name != '\0') {
/* Escape backslashes and double quotes. */
if (*name == '\\' || *name == '"')
fputc('\\', fp);
fputc(*name++, fp);
}
fputc('"', fp);
separator = ',';
}
name += strlen(name) + 1;