Skip to content

Comments

Add nhcb samples metric in distributor#7248

Merged
yeya24 merged 5 commits intocortexproject:masterfrom
yeya24:add-metric-nhcb
Feb 22, 2026
Merged

Add nhcb samples metric in distributor#7248
yeya24 merged 5 commits intocortexproject:masterfrom
yeya24:add-metric-nhcb

Conversation

@yeya24
Copy link
Contributor

@yeya24 yeya24 commented Feb 8, 2026

What this PR does:

Add nhcb dimension to cortex_distributor_received_samples_total and cortex_distributor_samples_in_total to keep track of number of nhcb samples.

Note that cortex_distributor_received_samples_total{type="nhcb"} is a subset of cortex_distributor_received_samples_total{type="histogram"} so doing sum(cortex_distributor_received_samples_total) will actually double count of nhcb samples.

Which issue(s) this PR fixes:
Fixes #

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 10, 2026
Signed-off-by: yeya24 <benye@amazon.com>
Signed-off-by: yeya24 <benye@amazon.com>
Signed-off-by: yeya24 <benye@amazon.com>
Signed-off-by: yeya24 <benye@amazon.com>
Signed-off-by: yeya24 <benye@amazon.com>
Copy link
Member

@SungJin1212 SungJin1212 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@yeya24 yeya24 merged commit 458e6b6 into cortexproject:master Feb 22, 2026
91 of 95 checks passed
@dosubot
Copy link

dosubot bot commented Feb 22, 2026

Related Documentation

Checked 0 published document(s) in 1 knowledge base(s). No updates required.

How did I do? Any feedback?  Join Discord

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/distributor lgtm This PR has been approved by a maintainer size/L type/feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants