From: Sage Weil Date: Thu, 2 May 2019 16:39:31 +0000 (-0500) Subject: os/bluestore: be verbose about objects that existing on rmcoll X-Git-Tag: v13.2.7~216^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F29217%2Fhead;p=ceph.git os/bluestore: be verbose about objects that existing on rmcoll This is always a bug (OSD doesn't try to remove a collection unless it thinks it is empty), and not seeing it at default debug levels makes it hard to track down. Signed-off-by: Sage Weil (cherry picked from commit e387cbc45037db78db5dfbb01483b8d76e1c276f) Conflicts: src/os/bluestore/BlueStore.cc - mimic does not have 87e8231d3fa61bc95ea0cda4d731e30f3d4de9d5 --- diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc index 6282c03b321..da8df0e8215 100644 --- a/src/os/bluestore/BlueStore.cc +++ b/src/os/bluestore/BlueStore.cc @@ -12177,8 +12177,8 @@ int BlueStore::_remove_collection(TransContext *txc, const coll_t &cid, ceph_assert((*c)->exists); if ((*c)->onode_map.map_any([&](OnodeRef o) { if (o->exists) { - dout(10) << __func__ << " " << o->oid << " " << o - << " exists in onode_map" << dendl; + dout(1) << __func__ << " " << o->oid << " " << o + << " exists in onode_map" << dendl; return true; } ++nonexistent_count; @@ -12204,7 +12204,7 @@ int BlueStore::_remove_collection(TransContext *txc, const coll_t &cid, auto onode = (*c)->onode_map.lookup(*it); exists = !onode || onode->exists; if (exists) { - dout(10) << __func__ << " " << *it + dout(1) << __func__ << " " << *it << " exists in db" << dendl; } }