Skip to content

Commit cf814b3

Browse files
jameseh96daverigby
authored andcommitted
Prefix audit stats in Prometheus
"enabled" and "dropped_events" are unambiguous under cbstats, where there is additional context of being in the audit stat group. In contrast, there is no such grouping under Prometheus, so it is not immediately clear what these stats are. Prefixing these stats with "audit_" for Prometheus clarifies what they are. (cbstats keys unchanged) Change-Id: I3b5588e9bc085d092f7d24b5e5504e3c8f8c330d Reviewed-on: http://review.couchbase.org/c/kv_engine/+/137652 Tested-by: Build Bot <[email protected]> Reviewed-by: Dave Rigby <[email protected]>
1 parent 5975846 commit cf814b3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

include/statistics/stats.def.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -382,8 +382,8 @@ STAT(cmd_mutation_10s_count, count, , )
382382
// us suffix would be confusing in Prometheus as the stat is scaled to seconds
383383
STAT(cmd_mutation_10s_duration_us, microseconds, cmd_mutation_10s_duration, )
384384
STAT(total_resp_errors, count, , )
385-
STAT(enabled, none, , )
386-
STAT(dropped_events, count, , )
385+
STAT(enabled, none, audit_enabled, )
386+
STAT(dropped_events, count, audit_dropped_events, )
387387

388388
// Vbucket aggreagated stats
389389
#define VB_AGG_STAT(name, unit, familyName, ...) \

0 commit comments

Comments
 (0)