Hotfix: Change type of search_total_cycles/search_total_ms_run metrics

[nantiferov]

Initially added in #953 as gauges, but usage shows that they're counters.
I'm not sure if there are many users of this feature, but technically metrics rename might break some dashboards/alerts.

Related: #942

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.05%. Comparing base (8efb8fa) to head (c9ea61f).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #962   +/-   ##
=======================================
  Coverage   82.05%   82.05%           
=======================================
  Files          18       18           
  Lines        2636     2636           
=======================================
  Hits         2163     2163           
  Misses        386      386           
  Partials       87       87           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[coveralls]

Pull Request Test Coverage Report for Build 11575892083

Details

• 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 85.406%

---

Totals
Change from base Build 11529082829:	0.0%
Covered Lines:	2259
Relevant Lines:	2645

---

💛 - Coveralls

[nantiferov]

Dashboard with rate:

And without:

Current state doesn't block anything, but I think it would be correct to define these metrics as counters and fix naming:

[oliver006]

I'm not sure if there are many users of this feature, but technically metrics rename might break some dashboards/alerts.

Yeah, I think this is still early so we can just change it without too much ado. Appreciate the concern though.

[oliver006]

https://github.com/oliver006/redis_exporter/releases/tag/v1.66.0

[nantiferov]

Thank you for review and release 👍