Hotfix: Change type of search_total_cycles/search_total_ms_run metrics

Initially added in oliver006#953 as gauges, but usage shows that they're counters. Related: oliver006#942
nantiferov · Oct 29, 2024 · c9ea61f · c9ea61f
1 parent 8efb8fa
commit c9ea61f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/exporter/exporter.go b/exporter/exporter.go
@@ -299,8 +299,6 @@ func NewRedisExporter(uri string, opts Options) (*Exporter, error) {
 			"search_global_idle":         "search_global_idle",
 			"search_global_total":        "search_global_total",
 			"search_bytes_collected":     "search_collected_bytes",
-			"search_total_cycles":        "search_total_cycles",
-			"search_total_ms_run":        "search_total_run_ms",
 			"search_dialect_1":           "search_dialect_1",
 			"search_dialect_2":           "search_dialect_2",
 			"search_dialect_3":           "search_dialect_3",
@@ -359,6 +357,8 @@ func NewRedisExporter(uri string, opts Options) (*Exporter, error) {
 
 			// Redis Modules metrics, RediSearch module
 			"search_total_indexing_time": "search_indexing_time_ms_total",
+			"search_total_cycles":        "search_cycles_total",
+			"search_total_ms_run":        "search_run_ms_total",
 		},
 	}
 

diff --git a/exporter/modules_test.go b/exporter/modules_test.go
@@ -41,8 +41,8 @@ func TestModules(t *testing.T) {
 			"search_global_idle":               false,
 			"search_global_total":              false,
 			"search_collected_bytes":           false,
-			"search_total_cycles":              false,
-			"search_total_run_ms":              false,
+			"search_cycles_total":              false,
+			"search_run_ms_total":              false,
 			"search_dialect_1":                 false,
 			"search_dialect_2":                 false,
 			"search_dialect_3":                 false,