Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

Commit

Permalink
Avoid using total suffix for gauge metrics and apply rate for counters
Browse files Browse the repository at this point in the history
Latest mixtool linter which relies on grafana-linter pkg is failing when gauge metrics with name ending with `total`[1].

[1] https://github.com/grafana/dashboard-linter/blob/44d415fb6bdc4d8e6585e514c448174d4de1ff02/lint/rule_target_counter_agg.go#L30

Signed-off-by: Arunprasad Rajkumar <ar.arunprasad@gmail.com>
  • Loading branch information
arajkumar committed Oct 31, 2022
1 parent 0606634 commit b597f0e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions docs/mixin/dashboards/promscale.json
Original file line number Diff line number Diff line change
Expand Up @@ -2291,7 +2291,7 @@
},
"editorMode": "code",
"exemplar": false,
"expr": "max(promscale_sql_database_worker_maintenance_job_long_running_total{namespace=~\"$namespace\"})",
"expr": "max(promscale_sql_database_worker_long_running_maintenance_jobs{namespace=~\"$namespace\"})",
"interval": "",
"legendFormat": "total",
"range": true,
Expand Down Expand Up @@ -2513,7 +2513,7 @@
},
"editorMode": "code",
"exemplar": false,
"expr": "max(promscale_sql_database_worker_maintenance_job_long_running_total{namespace=~\"$namespace\"})",
"expr": "max(promscale_sql_database_worker_long_running_maintenance_jobs{namespace=~\"$namespace\"})",
"interval": "",
"legendFormat": "total",
"range": true,
Expand Down Expand Up @@ -2687,7 +2687,7 @@
},
"editorMode": "code",
"exemplar": false,
"expr": "max(promscale_sql_database_worker_maintenance_job_locks_total{namespace=~\"$namespace\"})",
"expr": "max(promscale_sql_database_worker_maintenance_job_locks{namespace=~\"$namespace\"})",
"interval": "",
"legendFormat": "total",
"range": true,
Expand Down
4 changes: 2 additions & 2 deletions pkg/pgmodel/metrics/database/metrics.go
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ var metrics = []metricQueryWrap{
prometheus.GaugeOpts{
Namespace: util.PromNamespace,
Subsystem: "sql_database",
Name: "worker_maintenance_job_locks_total",
Name: "worker_maintenance_job_locks",
Help: "Number of locks held by Promscale maintenance workers.",
},
),
Expand Down Expand Up @@ -319,7 +319,7 @@ var metrics = []metricQueryWrap{
prometheus.GaugeOpts{
Namespace: util.PromNamespace,
Subsystem: "sql_database",
Name: "worker_maintenance_job_long_running_total",
Name: "worker_long_running_maintenance_jobs",
Help: "Number of Promscale maintenance workers executing long running queries.",
},
),
Expand Down

0 comments on commit b597f0e

Please # to comment.