From: Yuval Lifshitz <ylifshit@ibm.com>
Date: Thu, 30 Oct 2025 08:03:26 +0000 (+0000)
Subject: rgw/logging: removed unused APIs from header
X-Git-Tag: testing/wip-vshankar-testing-20260219.125903~6^2~5^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=f92765ee0fb8135abb07f9cbbcf837305b4487f7;p=ceph-ci.git

rgw/logging: removed unused APIs from header

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

diff --git a/src/rgw/rgw_bucket_logging.cc b/src/rgw/rgw_bucket_logging.cc
index 23873957e76..39bb78b498c 100644
--- a/src/rgw/rgw_bucket_logging.cc
+++ b/src/rgw/rgw_bucket_logging.cc
@@ -328,14 +328,6 @@ int commit_logging_object(const configuration& conf,
       << ret << dendl;
     return ret;
   }
-  return commit_logging_object(conf, target_bucket, dpp, y, last_committed);
-}
-
-int commit_logging_object(const configuration& conf,
-    const std::unique_ptr<rgw::sal::Bucket>& target_bucket,
-    const DoutPrefixProvider *dpp,
-    optional_yield y,
-    std::string* last_committed) {
   std::string obj_name;
   if (const int ret = target_bucket->get_logging_object_name(obj_name, conf.target_prefix, y, dpp, nullptr); ret < 0) {
     ldpp_dout(dpp, 1) << "ERROR: failed to get name of logging object of logging bucket '" <<
diff --git a/src/rgw/rgw_bucket_logging.h b/src/rgw/rgw_bucket_logging.h
index 872aa8f548d..58a6f290272 100644
--- a/src/rgw/rgw_bucket_logging.h
+++ b/src/rgw/rgw_bucket_logging.h
@@ -186,27 +186,6 @@ int rollover_logging_object(const configuration& conf,
     std::string* last_committed,
     std::string* err_message = nullptr);
 
-// commit the pending log object to the log bucket
-// use this for cleanup, when new pending object is not needed
-// and target bucket is known
-// if "last_committed" is not null, it will be set to the name of the last committed object
-int commit_logging_object(const configuration& conf,
-    const std::unique_ptr<rgw::sal::Bucket>& target_bucket,
-    const DoutPrefixProvider *dpp,
-    optional_yield y,
-    std::string* last_committed);
-
-// commit the pending log object to the log bucket
-// use this for cleanup, when new pending object is not needed
-// and target bucket shoud be loaded based on the configuration
-// if "last_committed" is not null, it will be set to the name of the last committed object
-int commit_logging_object(const configuration& conf,
-    const DoutPrefixProvider *dpp,
-    rgw::sal::Driver* driver,
-    const std::string& tenant_name,
-    optional_yield y,
-    std::string* last_committed);
-
 // return the oid of the object holding the name of the temporary logging object
 // bucket - log bucket
 // prefix - logging prefix from configuration. should be used when multiple buckets log into the same log bucket