os/bluestore: fix statfs to not include DB partition in free space#13140
Merged
liewegas merged 1 commit intoceph:masterfrom Jan 27, 2017
Merged
os/bluestore: fix statfs to not include DB partition in free space#13140liewegas merged 1 commit intoceph:masterfrom
liewegas merged 1 commit intoceph:masterfrom
Conversation
f3fda6f to
bcd4502
Compare
If we report the DB space as vailable, ceph thinks the OSD can store more data and will not mark the cluster as full as easily. And in reality, we can't actually store data in this space--only metadata. Avoid the problem by not reporting it as available. Fixes: http://tracker.ceph.com/issues/18599 Signed-off-by: Sage Weil <sage@redhat.com>
bcd4502 to
c66d5ba
Compare
Member
Author
|
Ah, on vstart the db partition is tiny (64mb) which is why it wasn't obvious this was working. |
ifed01
approved these changes
Jan 27, 2017
Contributor
|
@liewegas , our QA also finds this issue, thanks for the fix! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
If we report the DB space as vailable, ceph thinks the OSD can store more
data and will not mark the cluster as full as easily. And in reality, we
can't actually store data in this space--only metadata. Avoid the problem
by not reporting it as available.
Fixes: http://tracker.ceph.com/issues/18599
Signed-off-by: Sage Weil sage@redhat.com