Skip to content

Fixed memberlist support for the compactor ring #3095

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

pracucci
Copy link
Contributor

What this PR does:
Similarly to #3058, in this PR I'm fixing memberlist for the compactor ring. We don't have any integration test on compactor yet, so I've manually tested it. Will create an issue about adding compactor integration tests too.

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

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]
Signed-off-by: Marco Pracucci <marco@pracucci.com>
@pracucci pracucci merged commit bd88361 into cortexproject:master Aug 27, 2020
@pracucci pracucci deleted the fix-memberlist-for-compactor branch August 27, 2020 06:18
@pracucci pracucci added the storage/blocks Blocks storage engine label Aug 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS storage/blocks Blocks storage engine
2 participants