From: Yuval Lifshitz <ylifshit@ibm.com>
Date: Thu, 5 Jun 2025 07:48:07 +0000 (+0000)
Subject: rgw/logging: log only object ACls in journal mode
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=ce2d09efebb14c79ad1f8540d0824c477ba0b836;p=ceph.git

rgw/logging: log only object ACls in journal mode

Fixes: https://tracker.ceph.com/issues/71563

Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
---

diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
index 92fade2bfbb66..349f23615f35e 100644
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@ -6370,17 +6370,20 @@ void RGWPutACLs::execute(optional_yield y)
     return;
   }
 
-  const auto etag = s->object->get_attrs()[RGW_ATTR_ETAG].to_str();
-  op_ret = rgw::bucketlogging::log_record(driver,
-      rgw::bucketlogging::LoggingType::Journal,
-      s->object.get(),
-      s,
-      canonical_name(),
-      etag,
-      s->object->get_size(),
-      this, y, false, false);
-  if (op_ret < 0) {
-    return;
+  if (!rgw::sal::Object::empty(s->object)) {
+    // in journal mode we log only object ACLs
+    const auto etag = s->object->get_attrs()[RGW_ATTR_ETAG].to_str();
+    op_ret = rgw::bucketlogging::log_record(driver,
+        rgw::bucketlogging::LoggingType::Journal,
+        s->object.get(),
+        s,
+        canonical_name(),
+        etag,
+        s->object->get_size(),
+        this, y, false, false);
+    if (op_ret < 0) {
+      return;
+    }
   }
 
   bufferlist bl;