os/bluestore: Fix fsck

Fixed a case when a single extra onode shard causes a flood of shard errors.

Signed-off-by: Adam Kupczyk <akupczyk@ibm.com>

diff --git a/src/os/bluestore/BlueStore.cc b/src/os/bluestore/BlueStore.cc
index 5e64dd5282ce73c728e57f16e4f044a657713949..1112cf7a0070c6a1a55599483bf784fdd07fe124 100644 (file)
--- a/src/os/bluestore/BlueStore.cc
+++ b/src/os/bluestore/BlueStore.cc
@@ -10602,16 +10602,12 @@ void BlueStore::_fsck_check_objects(
           ++errors;
           continue;
         }
-        while (expecting_shards.front() > it->key()) {
+        if (expecting_shards.front() > it->key()) {
           derr << "fsck error:   saw " << pretty_binary_string(it->key())
             << dendl;
           derr << "fsck error:   exp "
             << pretty_binary_string(expecting_shards.front()) << dendl;
           ++errors;
-          expecting_shards.pop_front();
-          if (expecting_shards.empty()) {
-            break;
-          }
         }
         continue;
       }