-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Zone aware replication #3820
Comments
We could immediately benefit from it in our setup. We have been planning to support multi-cluster deployments. |
Hello 👋 Looks like there was no activity on this issue for the last two months. |
Still valid. |
Hello 👋 Looks like there was no activity on this issue for the last two months. |
Hello 👋 Looks like there was no activity on this issue for the last two months. |
Closing for now as promised, let us know if you need this to be reopened! 🤗 |
Hello 👋 Looks like there was no activity on this issue for the last two months. |
Closing for now as promised, let us know if you need this to be reopened! 🤗 |
Hello 👋 Looks like there was no activity on this issue for the last two months. |
+1, multi-az is a highly available requirement for many companies |
This has been implemented already, I think: https://thanos.io/tip/components/receive.md/#az-aware-ketama-hashring-experimental. |
Is your proposal related to a problem?
Allow not just replication factor and quorum to be a deciding factor, but also that this quorum must be across availability zones.
Describe the solution you'd like
Make Thanos aware of the availability zones receive nodes are in, and allow quorum to pass if and only if
ceil(replication-factor/2)
availability zones were successfully replicated to.Describe alternatives you've considered
Leaving as is, or not using Thanos. Leaving as is does not tolerate an entire zone outage though, which is not included in most cloud provider's SLOs (as in cloud providers say, that there is nothing wrong with an entire availability zone being unavailable if the rest works).
@thanos-io/thanos-maintainers
The text was updated successfully, but these errors were encountered: