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

kraken: osd: os/bluestore: fix statfs to not include DB partition in free space #13284

Merged
merged 1 commit into from Apr 15, 2017

Conversation

shinobu-x
Copy link
Contributor

@shinobu-x shinobu-x commented Feb 6, 2017

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>
(cherry picked from commit c66d5ba)
@tchaikov tchaikov added this to the kraken milestone Feb 7, 2017
@tchaikov tchaikov changed the title os/bluestore: fix statfs to not include DB partition in free space kraken: os/bluestore: fix statfs to not include DB partition in free space Feb 7, 2017
@smithfarm smithfarm changed the title kraken: os/bluestore: fix statfs to not include DB partition in free space kraken: osd: os/bluestore: fix statfs to not include DB partition in free space Feb 21, 2017
@smithfarm smithfarm added the core label Feb 21, 2017
@smithfarm
Copy link
Contributor

@liewegas This passed a rados suite at http://tracker.ceph.com/issues/19009#note-3 with one reproducible failure in rados/objectstore/objectstore.yaml that appears to be bluestore-related.

Do you think the test failure could be related to this PR?

Log of failed test: http://qa-proxy.ceph.com/teuthology/smithfarm-2017-04-11_06:46:27-rados-wip-kraken-backports-distro-basic-smithi/1012969/teuthology.log

@smithfarm
Copy link
Contributor

This passed another rados run at http://tracker.ceph.com/issues/19009#note-12

The rados/objectstore/objectstore.yaml failure was not reproduced.

@smithfarm smithfarm merged commit 8eb1044 into ceph:kraken Apr 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants