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

osd: bluestore: improve fs-type verification and tidy up #7651

Merged
merged 18 commits into from Feb 26, 2016

Conversation

xiexingguo
Copy link
Member

No description provided.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
For the sake of performance.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
We use internal blue-fs for such support.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
So we don't bother to start a real trans if it is a noop.

Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
Signed-off-by: xie xingguo <xie.xingguo@zte.com.cn>
@xiexingguo
Copy link
Member Author

Dropped those changes you disagree with and fixed the rest. Also make sure it passed my local test(script: rm -rf c store_test_temp_dir && make ceph_test_objectstore && CEPH_ARGS="--log-file c --debug-bluestore 20 --no-log-to-stderr" ./ceph_test_objectstore), below is the result:

2016-02-23 20:27:35.617754 7f37362794c0 -1 WARNING: experimental feature 'rocksdb' is enabled
Please be aware that this feature is experimental, untested,
unsupported, and may result in data corruption, data loss,
and/or irreparable damage to your cluster.  Do not use
feature with important data.

==> rm -r store_test_temp_dir
[       OK ] ObjectStore/StoreTest.SetAllocHint/3 (111 ms)
[----------] 160 tests from ObjectStore/StoreTest (3068328 ms total)

[----------] Global test environment tear-down
[==========] 160 tests from 1 test case ran. (3068328 ms total)
[  PASSED  ] 160 tests.

@liewegas liewegas changed the title os/bluestore: improve fs-type verification and tidy up osd: bluestore: improve fs-type verification and tidy up Feb 26, 2016
liewegas added a commit that referenced this pull request Feb 26, 2016
osd: bluestore: improve fs-type verification and tidy up

Reviewed-by: Sage Weil <sage@redhat.com>
@liewegas liewegas merged commit 0499ca8 into ceph:master Feb 26, 2016
@xiexingguo xiexingguo deleted the xxg-wip-fixtypecheck branch February 27, 2016 00:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants