From d21940117a5a6367dde366090a8f960f43254352 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 2 May 2019 11:39:31 -0500 Subject: [PATCH] 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 --- src/os/bluestore/BlueStore.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) 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; } } -- 2.47.3