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

nautilus: os/bluestore: default bluestore_block_size 1T -> 100G #32283

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

neha-ojha
Copy link
Member

Fixes: https://tracker.ceph.com/issues/42913

This makes vstart way faster. This option is only really relevant
for dev test environments. We bumped it up back in dbdd1d9
from 10G just to make ENOSPC less common in dev/test. Let's see if 100G
is a better balance.

Signed-off-by: Sage Weil sage@redhat.com
(cherry picked from commit 57890fc)

Conflicts:
src/common/options.cc: we did not cherry-pick
dbdd1d9 to nautilus, so we'll directly
go from 10G->100G

This makes vstart *way* faster.  This option is only really relevant
for dev test environments.  We bumped it up back in dbdd1d9
from 10G just to make ENOSPC less common in dev/test.  Let's see if 100G
is a better balance.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 57890fc)

 Conflicts:
	src/common/options.cc: we did not cherry-pick
        dbdd1d9 to nautilus, so we'll directly
        go from 10G->100G
@liewegas
Copy link
Member

Not sure this matters.. the reason to go from 10G -> 100G was to make ENOSPC less common for a developer using vstart. And we aren't usually developing on the nautilus branch.

@neha-ojha
Copy link
Member Author

Not sure this matters.. the reason to go from 10G -> 100G was to make ENOSPC less common for a developer using vstart. And we aren't usually developing on the nautilus branch.

The cram test 15f360d#diff-bce7896e1ecd14da7871a0509c3fcb9d fails in nautilus without this, maybe we can increase it just for that test or not run this test in nautilus?

@liewegas
Copy link
Member

Ah, ok. It's fine to merge this then, generally harmless!

@yuriw
Copy link
Contributor

yuriw commented Dec 17, 2019

@yuriw yuriw merged commit 9e82c65 into ceph:nautilus Dec 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants