diff options
author | tcambron <tcambron@digium.com> | 2015-12-09 09:48:29 -0600 |
---|---|---|
committer | Mark Michelson <mmichelson@digium.com> | 2015-12-09 15:57:35 -0600 |
commit | cd119ed4a2a3e5efcae005bb8fa105ffd658582c (patch) | |
tree | 7e1df62f736eb5f1b967fd4dabecade2439623c5 /res | |
parent | 142d4fefb8db0ac2c30b18f75dc415093fb77f27 (diff) |
res_chan_stats: Fix bug to send correct statistics to StatsD
Fixed a bug that originally would show a negative number of
active calls occuring in Asterisk. A gauge is persistent so
incrementing and decrementing it results in a more consistent
performance. Also changed to the call to StatsD to use
ast_statsd_log_string() so that a "+" could be sent to StatsD.
ASTERISK-25619 #close
Change-Id: Iaaeff5c4c6a46535366b4d16ea0ed0ee75ab2ee7
Diffstat (limited to 'res')
-rw-r--r-- | res/res_chan_stats.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/res/res_chan_stats.c b/res/res_chan_stats.c index df3471865..8d446b3b3 100644 --- a/res/res_chan_stats.c +++ b/res/res_chan_stats.c @@ -111,7 +111,7 @@ static void updates(void *data, struct stasis_subscription *sub, if (!update->old_snapshot && update->new_snapshot) { /* Initial cache entry; count a channel creation */ - ast_statsd_log("channels.count", AST_STATSD_COUNTER, 1); + ast_statsd_log_string("channels.count", AST_STATSD_GAUGE, "+1", 1.0); } else if (update->old_snapshot && !update->new_snapshot) { /* Cache entry removed. Compute the age of the channel and post * that, as well as decrementing the channel count. @@ -125,7 +125,7 @@ static void updates(void *data, struct stasis_subscription *sub, ast_statsd_log("channels.calltime", AST_STATSD_TIMER, age); /* And decrement the channel count */ - ast_statsd_log("channels.count", AST_STATSD_COUNTER, -1); + ast_statsd_log_string("channels.count", AST_STATSD_GAUGE, "-1", 1.0); } } |