Skip to content

Conversation

siddarth2810
Copy link
Contributor

What this PR does:
The inmemory store will not work correctly with multiple distributors as each distributor can have a different state, causing injestion errors. The PR adds a warning to the docs to clarify this.

Which issue(s) this PR fixes:
Fixes #5145

Checklist

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

Copy link
Contributor

@danielblando danielblando left a comment

Choose a reason for hiding this comment

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

Thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 18, 2025
@friedrichg friedrichg enabled auto-merge (squash) July 18, 2025 16:44
@friedrichg friedrichg merged commit f0e0a3a into cortexproject:master Jul 18, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Distributor HA tracker ring should allow inmemory as a KV store
3 participants