]> git.ipfire.org Git - thirdparty/collectd.git/commitdiff
memory plugin: Align metrics with OpenTelemetry recommendations. 4219/head
authorFlorian Forster <octo@collectd.org>
Sun, 17 Dec 2023 14:04:59 +0000 (15:04 +0100)
committerFlorian Forster <octo@collectd.org>
Wed, 3 Jan 2024 10:32:33 +0000 (11:32 +0100)
* The "state" label has been renamed to "system.memory.state".
* Metric descriptions and units have been added.

src/memory.c

index 77e85a0feafb6d475aef421624bb6cde281d6c59..7ecea01429a53831d368687dbba3d3393bb75fc9 100644 (file)
@@ -168,6 +168,8 @@ static int memory_config(oconfig_item_t *ci) /* {{{ */
 static int memory_dispatch(gauge_t values[COLLECTD_MEMORY_TYPE_MAX]) {
   metric_family_t fam_absolute = {
       .name = "system.memory.usage",
+      .help = "Reports memory in use by state",
+      .unit = "By",
       .type = METRIC_TYPE_GAUGE,
   };
   gauge_t total = 0;
@@ -180,8 +182,9 @@ static int memory_dispatch(gauge_t values[COLLECTD_MEMORY_TYPE_MAX]) {
     total += values[i];
 
     if (values_absolute) {
-      metric_family_append(&fam_absolute, "state", memory_type_names[i],
-                           (value_t){.gauge = values[i]}, NULL);
+      metric_family_append(&fam_absolute, "system.memory.state",
+                           memory_type_names[i], (value_t){.gauge = values[i]},
+                           NULL);
     }
   }
 
@@ -206,6 +209,8 @@ static int memory_dispatch(gauge_t values[COLLECTD_MEMORY_TYPE_MAX]) {
 
   metric_family_t fam_percent = {
       .name = "system.memory.utilization",
+      .help = "Reports memory in use by state",
+      .unit = "1",
       .type = METRIC_TYPE_GAUGE,
   };
   for (size_t i = 0; i < COLLECTD_MEMORY_TYPE_MAX; i++) {
@@ -213,7 +218,8 @@ static int memory_dispatch(gauge_t values[COLLECTD_MEMORY_TYPE_MAX]) {
       continue;
     }
 
-    metric_family_append(&fam_percent, "state", memory_type_names[i],
+    metric_family_append(&fam_percent, "system.memory.state",
+                         memory_type_names[i],
                          (value_t){.gauge = 100.0 * values[i] / total}, NULL);
   }