]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
ubi: fix data source type
authorFlorian Eckert <fe@dev.tdt.de>
Wed, 24 Jun 2020 10:56:33 +0000 (12:56 +0200)
committerFlorian Eckert <fe@dev.tdt.de>
Wed, 24 Jun 2020 10:57:05 +0000 (12:57 +0200)
Currently the values are stored in the ubi plugin as data source type
`counter`. But this makes no sense, because the values change very slowly
and I don't want to know the rate of change. It is better to store the
value as data source type `gauge`. Then I can see the current value.

Signed-off-by: Florian Eckert <fe@dev.tdt.de>
src/types.db
src/ubi.c

index c41c22e63a1ce8193095ff983de9e6604580ee40..414f8b287c384471bddd4a11916101f83ec9fc93 100644 (file)
@@ -7,7 +7,7 @@ apache_scoreboard       value:GAUGE:0:65535
 ath_nodes               value:GAUGE:0:65535
 ath_stat                value:DERIVE:0:U
 backends                value:GAUGE:0:65535
-bad_peb_count           value:COUNTER:0:U
+bad_peb_count           value:GAUGE:0:U
 bitrate                 value:GAUGE:0:4294967295
 blocked_clients         value:GAUGE:0:U
 bool                    value:GAUGE:0:1
@@ -141,7 +141,7 @@ job_stats               value:DERIVE:0:U
 latency                 value:GAUGE:0:U
 links                   value:GAUGE:0:U
 load                    shortterm:GAUGE:0:5000, midterm:GAUGE:0:5000, longterm:GAUGE:0:5000
-max_ec                  value:COUNTER:0:U
+max_ec                  value:GAUGE:0:U
 media                   value:GAUGE:0:18446744073709551615
 memory_bandwidth        value:DERIVE:0:U
 md_disks                value:GAUGE:0:U
index c8a5c851e6ae2ed99b56ee462189783418ef412c..75ef99a05a9e50932b4b1320422a7cc62372334e 100644 (file)
--- a/src/ubi.c
+++ b/src/ubi.c
@@ -70,13 +70,13 @@ static int ubi_config(const char *key, const char *value) {
 } /* int ubi_config */
 
 static void ubi_submit(const char *dev_name, const char *type,
-                       counter_t value) {
+                       gauge_t value) {
   value_list_t vl = VALUE_LIST_INIT;
 
   if (ignorelist_match(ignorelist, dev_name) != 0)
     return;
 
-  vl.values = &(value_t){.counter = value};
+  vl.values = &(value_t){.gauge = value};
   vl.values_len = 1;
   sstrncpy(vl.plugin, PLUGIN_NAME, sizeof(vl.plugin));
   sstrncpy(vl.type_instance, dev_name, sizeof(vl.type_instance));
@@ -107,7 +107,7 @@ static int ubi_read_dev_attr(const char *dev_name, const char *attr) {
     return -1;
   }
 
-  ubi_submit(dev_name, attr, (counter_t)val);
+  ubi_submit(dev_name, attr, (gauge_t)val);
 
   return 0;
 } /* int ubi_read_dev_attr */