Skip to content

Per-object Prometheus metrics: avoid duplicate HELP, TYPE metadata lines (backport #15610)#15632

Merged
michaelklishin merged 3 commits intov4.2.xfrom
mergify/bp/v4.2.x/pr-15610
Mar 4, 2026
Merged

Per-object Prometheus metrics: avoid duplicate HELP, TYPE metadata lines (backport #15610)#15632
michaelklishin merged 3 commits intov4.2.xfrom
mergify/bp/v4.2.x/pr-15610

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify bot commented Mar 4, 2026

I am not sure if this is the optimal approach but it is best place/manner of addressing this that I could find without affecting aggregated metrics.

Raft metrics can and do come from different Ra systems, namely Khepri and quorum queues. We need to format them as a "single" metric to avoid duplicate HELP, TYPE metadata lines.

Since quorum queues have dozens of metrics, we filter out a set of Raft-related ones specifically that combine well with the Raft metrics from Khepri.

Closes #15600.


This is an automatic backport of pull request #15610 done by [Mergify](https://mergify.com).

michaelklishin and others added 3 commits March 4, 2026 17:42
for Raft metrics.

Raft metrics can and do come from different Ra systems, namely Khepri and quorum queues.
We need to format them as a "single" metric to avoid duplicate HELP,
TYPE metadata lines.

Since quorum queues have dozens of metrics, we filter out a set of Raft-related
ones specifically that combine well with the Raft metrics from Khepri.

Closes #15600.

(cherry picked from commit e374715)
Previously a missing metric was ignored.

(cherry picked from commit e34e5f0)
@michaelklishin michaelklishin added this to the 4.2.5 milestone Mar 4, 2026
@michaelklishin michaelklishin merged commit 1ae3696 into v4.2.x Mar 4, 2026
580 of 583 checks passed
@michaelklishin michaelklishin deleted the mergify/bp/v4.2.x/pr-15610 branch March 4, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants