From: Xuehan Xu <xxhdx1985126@gmail.com>
Date: Wed, 27 Jan 2021 11:06:34 +0000 (+0800)
Subject: crimson/osd: objects without omap header may not necessarily be a error
X-Git-Tag: v17.1.0~3136^2~1
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ee8a068f4589d7ffcbc24d1720571248bbb98564;p=ceph.git

crimson/osd: objects without omap header may not necessarily be a error

There are circumstances in which objects don't have omap headers, so this should
not be treated as an error

Signed-off-by: Xuehan Xu <xxhdx1985126@gmail.com>
---

diff --git a/src/crimson/osd/replicated_recovery_backend.cc b/src/crimson/osd/replicated_recovery_backend.cc
index 70c268e83aff..218550fa7321 100644
--- a/src/crimson/osd/replicated_recovery_backend.cc
+++ b/src/crimson/osd/replicated_recovery_backend.cc
@@ -432,8 +432,7 @@ ReplicatedRecoveryBackend::read_metadata_for_push_op(
       }))
   ).then_unpack([&new_progress, push_op](auto bl, auto attrs) {
     if (bl.length() == 0) {
-      logger().error("read_metadata_for_push_op: fail to read omap header");
-      return eversion_t{};
+      logger().warn("read_metadata_for_push_op: fail to read omap header");
     } else if (attrs.empty()) {
       logger().error("read_metadata_for_push_op: fail to read attrs");
       return eversion_t{};