#1297 stats:show displays counter stat type wrongly

Reporter pep.
Owner MattJ
Created
Updated
Stars ★ (1)
Tags
  • Status-NeedInfo
  • Patch
  • Type-Defect
  • Priority-Medium
  1. pep. on

    What steps will reproduce the problem? 1. mod_measure_cpu loaded (or any other thing that provides a counter) 2. `stats:show()` in admin telnet What is the expected output? > | cpu.clock:counter 279.700536 > | cpu.percent:amount 0.20719375057321 What do you see instead? > | cpu.clock:amount 279.700536 > | cpu.percent:amount 0.20719375057321 What version of the product are you using? On what operating system? prosody: 7fa273f8869e prosody-modules: e0fb97f47212 Please provide any additional information below. I am not entirely sure that's how it should be, it just seemed like a mistake. Please discard if so, otherwise here is a patch. # HG changeset patch # User Maxime “pep” Buquet <pep@bouah.net> # Date 1547656867 0 # Wed Jan 16 16:41:07 2019 +0000 # Node ID 17f038db9695892e7de21978b5a08886b6b5ac94 # Parent 00f7af0be849549c1e65c485f214ca90c97981a9 util.statistics: Fix counter method name diff --git a/util/statistics.lua b/util/statistics.lua --- a/util/statistics.lua +++ b/util/statistics.lua @@ -51,7 +51,7 @@ local function new_registry(config) end; counter = function (name, initial) local v = initial or 0; - registry[name..":amount"] = function () return "amount", v; end + registry[name..":counter"] = function () return "counter", v; end return function (delta) v = v + delta; end;

  2. Zash on

    I believe it's intentional, that a counter is defined as an amount that is updated by a delta. Need MattJ to confirm this.

    Changes
    • tags Patch Status-NeedInfo
    • owner MattJ

New comment

Not published. Used for spam prevention and optional update notifications.