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

test/store_test: add deferred test case setup to support explicit min… #13415

Merged
merged 1 commit into from Feb 28, 2017

Conversation

ifed01
Copy link
Contributor

@ifed01 ifed01 commented Feb 14, 2017

…_alloc_size specification.

Fixes: http://tracker.ceph.com/issues/18857

Signed-off-by: Igor Fedotov ifedotov@mirantis.com

@ifed01
Copy link
Contributor Author

ifed01 commented Feb 14, 2017

[ OK ] ObjectStore/StoreTestSpecificAUSize.OnodeSizeTracking/2 (5939 ms)
[----------] 9 tests from ObjectStore/StoreTestSpecificAUSize (10143170 ms total)

[----------] Global test environment tear-down
[==========] 69 tests from 2 test cases ran. (10631240 ms total)
[ PASSED ] 69 tests.

@xiexingguo
Copy link
Member

retest this please

@@ -100,6 +100,7 @@ bool sorted(const vector<ghobject_t> &in, bool bitwise) {
return true;
}


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

extra line.

Copy link
Contributor

@varadakari varadakari left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@ifed01 ifed01 force-pushed the wip-fix-storetest branch 2 times, most recently from 9b80e4d to 3587c02 Compare February 16, 2017 17:27
@tchaikov
Copy link
Contributor

@ifed01 needs rebase.

@ifed01
Copy link
Contributor Author

ifed01 commented Feb 27, 2017

@tchaikov - rebased.

…_alloc_size specification.

Fixes: ceph#18857

Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
@liewegas liewegas merged commit 3212ba7 into ceph:master Feb 28, 2017
@ifed01 ifed01 deleted the wip-fix-storetest branch March 1, 2017 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants