fix(compute): Report compute_backpressure_throttling_seconds as counter

It was reported as `gauge`, but it's actually a `counter`. Also add `_total` suffix as that's the convention for counters.
neondatabase · Dec 17, 2024 · 12756e1 · 12756e1
1 parent b5833ef
commit 12756e1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/compute/etc/neon_collector.jsonnet b/compute/etc/neon_collector.jsonnet
@@ -3,7 +3,7 @@
   metrics: [
     import 'sql_exporter/checkpoints_req.libsonnet',
     import 'sql_exporter/checkpoints_timed.libsonnet',
-    import 'sql_exporter/compute_backpressure_throttling_seconds.libsonnet',
+    import 'sql_exporter/compute_backpressure_throttling_seconds_total.libsonnet',
     import 'sql_exporter/compute_current_lsn.libsonnet',
     import 'sql_exporter/compute_logical_snapshot_files.libsonnet',
     import 'sql_exporter/compute_logical_snapshots_bytes.libsonnet',

diff --git a/...backpressure_throttling_seconds.libsonnet → ...essure_throttling_seconds_total.libsonnet b/...backpressure_throttling_seconds.libsonnet → ...essure_throttling_seconds_total.libsonnet
@@ -1,10 +1,10 @@
 {
-  metric_name: 'compute_backpressure_throttling_seconds',
-  type: 'gauge',
+  metric_name: 'compute_backpressure_throttling_seconds_total',
+  type: 'counter',
   help: 'Time compute has spent throttled',
   key_labels: null,
   values: [
     'throttled',
   ],
-  query: importstr 'sql_exporter/compute_backpressure_throttling_seconds.sql',
+  query: importstr 'sql_exporter/compute_backpressure_throttling_seconds_total.sql',
 }
diff --git a/...mpute_backpressure_throttling_seconds.sql → ...backpressure_throttling_seconds_total.sql b/...mpute_backpressure_throttling_seconds.sql → ...backpressure_throttling_seconds_total.sql