Add _total suffix to ruler counter metrics

[yeya24]

Signed-off-by: Ben Ye [email protected]

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Checklist

• Documentation added
• Tests updated

[codecov-commenter]

Codecov Report

Merging #2582 into master will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2582      +/-   ##
==========================================
+ Coverage   63.00%   63.11%   +0.10%     
==========================================
  Files         169      169              
  Lines       15005    15005              
==========================================
+ Hits         9454     9470      +16     
+ Misses       4796     4781      -15     
+ Partials      755      754       -1     

Impacted Files	Coverage Δ
pkg/ruler/manager/memstore.go	68.25% <100.00%> (ø)
pkg/querier/queryrange/downstreamer.go	95.87% <0.00%> (-2.07%)	⬇️
pkg/promtail/targets/file/filetarget.go	66.28% <0.00%> (-0.58%)	⬇️
pkg/promtail/targets/file/tailer.go	75.00% <0.00%> (+4.16%)	⬆️
pkg/promtail/positions/positions.go	59.64% <0.00%> (+13.15%)	⬆️

[slim-bean]

Thanks Ben! could you also update the dashboards: ksonnet/lib/loki-cloud-mixin/ruler-dashboard.libsonnet with the fixed metric name?

[yeya24]

Hi @slim-bean, I am sorry I cannot find the file you mentioned. Is it in the production/ksonnet directory? I cannot find any ruler-dashboard.libsonnet file in Loki repo.

[slim-bean]

Sorry, looks like my search hit code in a different repo! you are good to go!

[slim-bean]

LGTM!