From 28c73c602f1a6cd392f889c50d7d9f62e378f240 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Tue, 22 Nov 2016 18:54:19 -0600 Subject: [PATCH] os/filestore: blacklist same ENOENT failures as bluestore FileStore doesn't care per se, but they indicate buggy calling code, and we want to catch all bad callers! Signed-off-by: Sage Weil --- src/os/filestore/FileStore.cc | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/os/filestore/FileStore.cc b/src/os/filestore/FileStore.cc index d3f632192b5b..f2be630f711b 100644 --- a/src/os/filestore/FileStore.cc +++ b/src/os/filestore/FileStore.cc @@ -2942,7 +2942,14 @@ void FileStore::_do_transaction( if (r == -ENOENT && !(op->op == Transaction::OP_CLONERANGE || op->op == Transaction::OP_CLONE || op->op == Transaction::OP_CLONERANGE2 || - op->op == Transaction::OP_COLL_ADD)) + op->op == Transaction::OP_COLL_ADD || + op->op == Transaction::OP_SETATTR || + op->op == Transaction::OP_SETATTRS || + op->op == Transaction::OP_RMATTR || + op->op == Transaction::OP_OMAP_SETKEYS || + op->op == Transaction::OP_OMAP_RMKEYS || + op->op == Transaction::OP_OMAP_RMKEYRANGE || + op->op == Transaction::OP_OMAP_SETHEADER)) // -ENOENT is normally okay // ...including on a replayed OP_RMCOLL with checkpoint mode ok = true; -- 2.47.3