gauge metric with suffix _total

[yeya24]

I am looking through some cortex metrics. I found that some gauge metrics with suffix _total like

	totalConfigs = promauto.NewGauge(prometheus.GaugeOpts{
		Namespace: "cortex",
		Name:      "scheduler_configs_total",
		Help:      "How many configs the scheduler knows about.",
	})
	totalRuleGroups = promauto.NewGauge(prometheus.GaugeOpts{
		Namespace: "cortex",
		Name:      "scheduler_groups_total",
		Help:      "How many rule groups the scheduler is currently evaluating",
	})

Does this break the convention https://prometheus.io/docs/practices/naming/ ? Is it better to change the suffix from _total to _count ?

[bboreham]

Yes. It’s generally a pain to rename metrics because it breaks historical reporting but in this case I don’t feel too bad.

[yeya24]

If we want to rename them I' d like to open a PR

[bboreham]

Please do.