rgw: trim bilog: send bucket param instead of source-bucket param

For backward compatibility, so that older rgws could handle it appropriately.

Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>

diff --git a/src/rgw/rgw_rest_log.cc b/src/rgw/rgw_rest_log.cc
index 735e652014a0cad3fa2c00e3c547d6d54517e891..c0e0aa6b86d0cc8cb843a5b0b4bb9cf3ad41cff3 100644 (file)
--- a/src/rgw/rgw_rest_log.cc
+++ b/src/rgw/rgw_rest_log.cc
@@ -866,7 +866,8 @@ void RGWOp_BILog_Status::execute()
 
   rgw_bucket source_bucket;
 
-  if (source_key.empty()) {
+  if (source_key.empty() ||
+      source_key == key) {
     source_bucket = info.bucket;
   } else {
     http_ret = rgw_bucket_parse_bucket_key(s->cct, source_key, &source_bucket, nullptr);

diff --git a/src/rgw/rgw_trim_bilog.cc b/src/rgw/rgw_trim_bilog.cc
index abfba41331b0688d3438e2a9c7b250ad8c4a26ab..8ccc6d54743876817d16e9fe4eeaf6d64a60e8a1 100644 (file)
--- a/src/rgw/rgw_trim_bilog.cc
+++ b/src/rgw/rgw_trim_bilog.cc
@@ -500,7 +500,8 @@ int BucketTrimInstanceCR::operate()
           { "type", "bucket-index" },
           { "status", nullptr },
           { "options", "merge" },
-          { "source-bucket", bucket_instance.c_str() },
+          { "bucket", bucket_instance.c_str() }, /* equal to source-bucket when `options==merge` and source-bucket
+                                                    param is not provided */
           { "source-zone", zone_id.c_str() },
           { nullptr, nullptr }
         };