Skip to content
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: Do not use "min size = 1" as an example #17912

Merged
merged 4 commits into from Oct 2, 2017
Merged

Conversation

alfredodeza
Copy link
Contributor

Because it is easy to assume it is a normal setting, although there is significant potential for data loss.

Check rule added to ceph-medic at: ceph/ceph-medic#11 to warn when this is set to 1

Alfredo Deza added 4 commits September 22, 2017 17:10
Because `osd pool default min size = 1` is dangerous and may cause
data loss.

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Signed-off-by: Alfredo Deza <adeza@redhat.com>
Which prevents readin "min_size=1" and assuming it is an OK setting

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Which prevents readin "min_size=1" and assuming it is an OK setting

Signed-off-by: Alfredo Deza <adeza@redhat.com>
Copy link
Contributor

@badone badone left a comment

Choose a reason for hiding this comment

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

LGTM

@liewegas liewegas merged commit f778242 into master Oct 2, 2017
@liewegas liewegas deleted the docs-min-size branch October 2, 2017 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants