diff options
| author | Seena Fallah <seenafallah@gmail.com> | 2025-01-08 22:25:13 +0100 |
|---|---|---|
| committer | Seena Fallah <seenafallah@gmail.com> | 2025-01-08 22:25:13 +0100 |
| commit | f28886124f77af4803e54934a7ddb82645e2f15d (patch) | |
| tree | 8ff682868df7351c64599a08270d6d8aede91f82 | |
| parent | rgw: set retcode on verify_bucket_permission in RGWObjFetchCR (diff) | |
| download | ceph-f28886124f77af4803e54934a7ddb82645e2f15d.tar.xz ceph-f28886124f77af4803e54934a7ddb82645e2f15d.zip | |
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>
| -rw-r--r-- | src/rgw/driver/rados/rgw_data_sync.cc | 2 |
1 files changed, 1 insertions, 1 deletions
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)"); |