Skip to content

Conversation

alexqyle
Copy link
Contributor

@alexqyle alexqyle commented Jul 22, 2025

What this PR does:

Add zone label to metric ring_members. So end user could know ring members by state by zone. This would be useful to make sure ring members are balanced across zones.

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]

@alexqyle alexqyle changed the title Add zone label to ring_members metric Add zone label to ring_members metric. Jul 22, 2025
alexqyle added 2 commits July 22, 2025 14:37
Signed-off-by: Alex Le <[email protected]>
Signed-off-by: Alex Le <[email protected]>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 24, 2025
@danielblando danielblando merged commit 9861229 into cortexproject:master Jul 24, 2025
17 checks passed
rubywtl pushed a commit to rubywtl/cortex that referenced this pull request Jul 28, 2025
* add zone label to ring_members metric

Signed-off-by: Alex Le <[email protected]>

* update changelog

Signed-off-by: Alex Le <[email protected]>

* fix nil pointer

Signed-off-by: Alex Le <[email protected]>

---------

Signed-off-by: Alex Le <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ring lgtm This PR has been approved by a maintainer size/L
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants