]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
os/bluestore: be verbose about objects that existing on rmcoll 29217/head
authorSage Weil <sage@redhat.com>
Thu, 2 May 2019 16:39:31 +0000 (11:39 -0500)
committerNathan Cutler <ncutler@suse.com>
Tue, 23 Jul 2019 15:00:41 +0000 (17:00 +0200)
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>
(cherry picked from commit e387cbc45037db78db5dfbb01483b8d76e1c276f)

Conflicts:
src/os/bluestore/BlueStore.cc
- mimic does not have 87e8231d3fa61bc95ea0cda4d731e30f3d4de9d5

src/os/bluestore/BlueStore.cc

index 6282c03b321415904c62c47676cb9042286cfc4b..da8df0e8215fef6b5ba9a6641e544c49c3eaefb6 100644 (file)
@@ -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;
         }
       }