From: Seena Fallah <seenafallah@gmail.com>
Date: Wed, 8 Jan 2025 21:25:13 +0000 (+0100)
Subject: rgw: skip on EACCES in RGWBucketSyncSingleEntryCR
X-Git-Tag: testing/wip-hyelloji-testing-20250121.140335~11^2
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=f28886124f77af4803e54934a7ddb82645e2f15d;p=ceph-ci.git

rgw: skip on EACCES in RGWBucketSyncSingleEntryCR

As of offloading source bucket perm check to the source cluster by (a3f40b4)
the http result of 403 would be translated to EACCES by rgw_http_error_to_errno()
so we need to also consider EACCES to be skipped in RGWBucketSyncSingleEntryCR().

Signed-off-by: Seena Fallah <seenafallah@gmail.com>
---

diff --git a/src/rgw/driver/rados/rgw_data_sync.cc b/src/rgw/driver/rados/rgw_data_sync.cc
index 22e08c039f3..1302f278f59 100644
--- a/src/rgw/driver/rados/rgw_data_sync.cc
+++ b/src/rgw/driver/rados/rgw_data_sync.cc
@@ -4520,7 +4520,7 @@ public:
           }
           tn->set_resource_name(SSTR(bucket_str_noinstance(bs.bucket) << "/" << key));
         }
-        if (retcode == -ERR_PRECONDITION_FAILED || retcode == -EPERM) {
+        if (retcode == -ERR_PRECONDITION_FAILED || retcode == -EPERM || retcode == -EACCES) {
 	  pretty_print(sc->env, "Skipping object s3://{}/{} in sync from zone {}\n",
 		       bs.bucket.name, key, zone_name);
           set_status("Skipping object sync: precondition failed (object contains newer change or policy doesn't allow sync)");