os/kstore: rmcoll fix to satisfy store_test #11533

Merged
merged 3 commits into from Oct 18, 2016

Projects

None yet

2 participants

@ifed01
Contributor
ifed01 commented Oct 18, 2016

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

ifed01 added some commits Oct 18, 2016
@ifed01 ifed01 os/kstore: uniform Collection implementation with BlueStore one
Signed-off-by:Igor Fedotov <ifedotov@mirantis.com>
4a9c8e0
@ifed01 ifed01 os/kstore: fix KStore rm_coll handler to behave similar to bluestore one
Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
5b36a93
@ifed01 ifed01 os/bluestore: proper set resulting pnext to avoid returning local var
Signed-off-by: Igor Fedotov <ifedotov@mirantis.com>
ebf81e3
src/test/objectstore/store_test.cc
@@ -2768,6 +2768,7 @@ TEST_P(StoreTest, SimpleCloneTest) {
ObjectStore::Transaction t;
t.remove_collection(cid);
cerr << "Invalid rm coll" << std::endl;
+// apply_transaction(store, &osr, std::move(t));
src/os/kstore/KStore.cc
+ vector<ghobject_t> *ls, ghobject_t *pnext)
+{
+/* if (!c->exists)
+ return -ENOENT;*/
@liewegas liewegas merged commit 15acca0 into ceph:master Oct 18, 2016

2 checks passed

Signed-off-by all commits in this PR are signed
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment