-
Notifications
You must be signed in to change notification settings - Fork 816
Make ruler HA #310
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
Comments
Current plan:
We'll sort out the config mess later. Not personally going to do any of this any time soon, although I would personally like to do it. I also think this should wait until #555 is done. |
prometheus/prometheus#422 is relevant here. AFAICT, much of the discussion on the ticket assumes that Prometheus's metric store is the only acceptable place to store 'for' state. Cortex doesn't have that constraint, but ideally the work would be done such that the storage is behind a golang interface, such that the current in-memory approach and a hypothetical TSDB-backed approach would both work, in addition to whatever dynamodb / bigtable thing we do. |
Lots of this came up from a conversation I had w/ @juliusv a while back. The good ideas are probably his, the bad ones mine. |
Fixed in #1258. Ruler is now horizontally scalable, and rules are sharded using the ring. I believe this should be enough. |
Currently the ruler is not HA. There's only one instance that runs all the rules. If it goes down, rules are not being processed.
The text was updated successfully, but these errors were encountered: