-
Notifications
You must be signed in to change notification settings - Fork 512
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
Docs: Review performing rolling updates docs #1402
Conversation
Given series are sharded across all ingesters, Grafana Mimir tolerates up to one unavailable ingester. | ||
By default, ingesters run with a replication factor equal to `3`. | ||
Ingesters running with the replication factor of `3` require a quorum of two instances to successfully query any series samples. | ||
Given series are sharded across all ingesters. Grafana Mimir tolerates up to one unavailable ingester. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pracucci , not sure what 'Given' means here. Can you please clarify? Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Given" means "Since" in this context and the comma was intentional in the original sentence.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (modulo addressing the comment)
Given series are sharded across all ingesters, Grafana Mimir tolerates up to one unavailable ingester. | ||
By default, ingesters run with a replication factor equal to `3`. | ||
Ingesters running with the replication factor of `3` require a quorum of two instances to successfully query any series samples. | ||
Given series are sharded across all ingesters. Grafana Mimir tolerates up to one unavailable ingester. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Given" means "Since" in this context and the comma was intentional in the original sentence.
docs/sources/operating-grafana-mimir/performing-a-rolling-update.md
Outdated
Show resolved
Hide resolved
…te.md Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com>
* Docs: Review performing rolling updates docs * Update docs/sources/operating-grafana-mimir/performing-a-rolling-update.md Co-authored-by: Christopher Moyer <35463610+chri2547@users.noreply.github.com> Co-authored-by: Marco Pracucci <marco@pracucci.com>
Relates to #1104