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

mon/PGMap: factor mon_osd_full_ratio into MAX AVAIL calc #12923

Merged
merged 1 commit into from Feb 9, 2017

Conversation

Projects
None yet
3 participants
@liewegas
Member

liewegas commented Jan 13, 2017

If we only fill OSDs to 95%, we should factor that into
the MAX AVAIL calculation for the pool.

Fixes: http://tracker.ceph.com/issues/18522
Signed-off-by: Sage Weil sage@redhat.com

@liewegas liewegas requested a review from jecluis Jan 13, 2017

@smithfarm

This comment has been minimized.

Contributor

smithfarm commented Jan 13, 2017

We might be able to squeeze this one into hammer, yet, but at some point we're going to have to say "stop".

@jdurgin

This comment has been minimized.

Member

jdurgin commented Jan 28, 2017

Should add an explanation of max_avail to doc/rados/operations/monitoring.rst at least - it'll be odd since it's the only one of our several space reporting interfaces that takes this ratio into account. Did this come up because openstack uses the 'ceph df' command, or for some other reason?

mon/PGMap: factor mon_osd_full_ratio into MAX AVAIL calc
If we only fill OSDs to 95%, we should factor that into
the MAX AVAIL calculation for the pool.

Fixes: http://tracker.ceph.com/issues/18522
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas

This comment has been minimized.

Member

liewegas commented Feb 3, 2017

updated

@jdurgin

jdurgin approved these changes Feb 4, 2017

@jdurgin jdurgin added the needs-qa label Feb 4, 2017

@liewegas liewegas merged commit 9c881e9 into ceph:master Feb 9, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@liewegas liewegas deleted the liewegas:wip-df-avail branch Feb 9, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment