rgw/logging: do not create empty temporary objects

Fixes: https://tracker.ceph.com/issues/73845
Signed-off-by: Yuval Lifshitz <ylifshit@ibm.com>
(cherry picked from commit 256eb4cd27a1389a93b0e743d0ae65f69f7417a5)

diff --git a/src/rgw/driver/rados/rgw_sal_rados.cc b/src/rgw/driver/rados/rgw_sal_rados.cc
index 3ef6f24dc6c2282476141bb666dc904841a80a47..5f0f962de22621f0612d203779b9c86722d814cd 100644 (file)
--- a/src/rgw/driver/rados/rgw_sal_rados.cc
+++ b/src/rgw/driver/rados/rgw_sal_rados.cc
@@ -1313,23 +1313,8 @@ int RadosBucket::commit_logging_object(const std::string& obj_name,
         << ". error: " << ret << dendl;
       return ret;
     }
+    mtime = ceph::real_time::clock::now();
     ldpp_dout(dpp, 20) << "INFO: temporary logging object '" << temp_obj_name << "' does not exist. committing it empty" << dendl;
-    // creating an empty object
-    if (ret = rgw_put_system_obj(dpp, store->svc()->sysobj,
-                   data_pool,
-                   temp_obj_name,
-                   bl_data, // empty bufferlist
-                   true, // exclusive
-                   nullptr,
-                   ceph::real_time::clock::now(),
-                   y); ret < 0) {
-      if (ret == -EEXIST) {
-        ldpp_dout(dpp, 5) << "WARNING: race detected in committing an empty logging object '" << temp_obj_name << dendl;
-      } else {
-        ldpp_dout(dpp, 1) << "ERROR: failed to commit empty logging object '" << temp_obj_name << "'. error: " << ret << dendl;
-      }
-      return ret;
-    }
   }
 
   uint64_t size = bl_data.length();