From: Radoslaw Zarzynski <rzarzyns@redhat.com>
Date: Sat, 14 Mar 2026 19:53:50 +0000 (+0000)
Subject: crimson/osd: improve !is_omap() logging
X-Git-Tag: v21.0.0~3^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=bc0f47174e2c8bf2b50262d0d382bb11a60d5545;p=ceph.git

crimson/osd: improve !is_omap() logging

Signed-off-by: Radoslaw Zarzynski <rzarzyns@redhat.com>
---

diff --git a/src/crimson/osd/pg_backend.cc b/src/crimson/osd/pg_backend.cc
index 5bf96d612046..8047b17c15c0 100644
--- a/src/crimson/osd/pg_backend.cc
+++ b/src/crimson/osd/pg_backend.cc
@@ -1359,6 +1359,7 @@ maybe_get_omap_vals_by_keys(
     return crimson::os::with_store<&crimson::os::FuturizedStore::Shard::omap_get_values>(
       store, coll, ghobject_t{oi.soid}, keys_to_get, 0);
   } else {
+    logger().error("{} on {}: backend has no OMAP support",__func__, oi.soid);
     return crimson::ct_error::enodata::make();
   }
 }
@@ -1380,6 +1381,7 @@ maybe_do_omap_iterate(
     return crimson::os::with_store<&crimson::os::FuturizedStore::Shard::omap_iterate>(
       store, coll, ghobject_t{oi.soid}, start_from, callback, 0, nullptr);
   } else {
+    logger().error("{} on {}: backend has no OMAP support",__func__, oi.soid);
     return crimson::ct_error::enodata::make();
   }
 }
@@ -1420,6 +1422,7 @@ PGBackend::omap_get_header(
   } else {
     // no omap? return empty data but not ENOENT. This is imporant for
     // the case when the object is being creating due to to may_write().
+    logger().error("{} on {}: backend has no OMAP support",__func__, os.oi.soid);
     return seastar::now();
   }
 }
@@ -1550,6 +1553,7 @@ PGBackend::omap_cmp(
       return  do_omap_val_cmp(out, assertions);
     });
   } else {
+    logger().error("{} on {}: backend has no OMAP support", __func__, os.oi.soid);
     return crimson::ct_error::ecanceled::make();
   }
 }