Skip to content

Comments

rgw: reject PutBucketReplication policies with mismatched statuses, skip sync on update#62323

Merged
ivancich merged 3 commits intoceph:mainfrom
clwluvw:bucket-replication-mismatch
Mar 20, 2025
Merged

rgw: reject PutBucketReplication policies with mismatched statuses, skip sync on update#62323
ivancich merged 3 commits intoceph:mainfrom
clwluvw:bucket-replication-mismatch

Conversation

@clwluvw
Copy link
Member

@clwluvw clwluvw commented Mar 15, 2025

PutBucketReplication is now rejecting policies that name a destination bucket with a different status.
If the status changes after PutBucketReplication, the corresponding bucket sync processes will skip those entries.

Fixes: https://tracker.ceph.com/issues/70486 (partially - the status change is tracked in #62147)

@clwluvw clwluvw changed the title Bucket replication mismatch rgw: reject PutBucketReplication policies with mismatched statuses, skip sync on update Mar 15, 2025
@clwluvw clwluvw marked this pull request as ready for review March 15, 2025 10:29
@clwluvw clwluvw requested a review from a team as a code owner March 15, 2025 10:29
clwluvw added 3 commits March 15, 2025 12:21
Reject PutBucketReplication calls if versioning is not identical
between the source and destination buckets. This check also applies
to object lock configurations to ensure consistency.

Fixes: https://tracker.ceph.com/issues/70486
Signed-off-by: Seena Fallah <[email protected]>
As instance field is preserved while replicating, this can make
confusion for versioned buckets having non-versioned objects and
non-versioned buckets having versioned objects.

Fixes: https://tracker.ceph.com/issues/70486
Signed-off-by: Seena Fallah <[email protected]>
replicating lock enabled objects require the destination bucket to
have object lock enabled.

Fixes: https://tracker.ceph.com/issues/70486
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw clwluvw force-pushed the bucket-replication-mismatch branch from d6ec9af to 826d493 Compare March 15, 2025 11:21
@ivancich ivancich added the wip-eric-testing-1 for ivancich testing label Mar 19, 2025
@ivancich ivancich merged commit 5f16ece into ceph:main Mar 20, 2025
11 checks passed
@ivancich ivancich removed needs-qa wip-eric-testing-1 for ivancich testing labels Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants