]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
os/bluestore: be verbose about objects that existing on rmcoll
authorSage Weil <sage@redhat.com>
Thu, 2 May 2019 16:39:31 +0000 (11:39 -0500)
committerSage Weil <sage@redhat.com>
Mon, 6 May 2019 02:14:18 +0000 (21:14 -0500)
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 <sage@redhat.com>
src/os/bluestore/BlueStore.cc

index a558677f682c608e881a0e81c016b9865f8f6eff..e1149ed33a2a6a2fa9947933492f8d310b2cdb34 100644 (file)
@@ -13295,8 +13295,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;
@@ -13322,10 +13322,10 @@ 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
-                  << " exists in db, "
-                  << (!onode ? "not present in ram" : "present in ram")
-                  << dendl;
+          dout(1) << __func__ << " " << *it
+                 << " exists in db, "
+                 << (!onode ? "not present in ram" : "present in ram")
+                 << dendl;
         }
       }
       if (!exists) {