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

common/options: make "blue{fs,store}_allocator" LEVEL_DEV #16645

Merged
merged 1 commit into from Jul 28, 2017

Conversation

Projects
None yet
3 participants
@tchaikov
Contributor

tchaikov commented Jul 28, 2017

See-also: http://tracker.ceph.com/issues/20660
Signed-off-by: Kefu Chai kchai@redhat.com

@tchaikov tchaikov added the cleanup label Jul 28, 2017

@tchaikov tchaikov requested a review from liewegas Jul 28, 2017

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Jul 28, 2017

Contributor
Contributor

tchaikov commented Jul 28, 2017

fyi, @shangzhong

@tchaikov tchaikov added the bluestore label Jul 28, 2017

@xiexingguo

This comment has been minimized.

Show comment
Hide comment
@xiexingguo

xiexingguo Jul 28, 2017

Member

may restrict bluestore_allocator for LEVEL_DEV too

Member

xiexingguo commented Jul 28, 2017

may restrict bluestore_allocator for LEVEL_DEV too

@tchaikov

This comment has been minimized.

Show comment
Hide comment
@tchaikov

tchaikov Jul 28, 2017

Contributor

@xiexingguo updated and repushed.

Contributor

tchaikov commented Jul 28, 2017

@xiexingguo updated and repushed.

@tchaikov tchaikov changed the title from common/options: make "bluefs_allocator" LEVEL_DEV to common/options: make "blue{fs,store}_allocator" LEVEL_DEV Jul 28, 2017

@liewegas

This comment has been minimized.

Show comment
Hide comment
@liewegas

liewegas Jul 28, 2017

Member

#16489 does the bluestore one.. didn't cover bluefs yet

Member

liewegas commented Jul 28, 2017

#16489 does the bluestore one.. didn't cover bluefs yet

@liewegas liewegas merged commit 597a6a6 into ceph:master Jul 28, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
make check (arm64) make check succeeded
Details

@tchaikov tchaikov deleted the tchaikov:wip-bluefs-allocator-is-dev-only-option branch Jul 28, 2017

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