Skip to content

Comments

rgw: check for duplicate rule priorities in replication configuration#61803

Merged
cbodley merged 1 commit intoceph:mainfrom
clwluvw:duplicate-prio
Mar 18, 2025
Merged

rgw: check for duplicate rule priorities in replication configuration#61803
cbodley merged 1 commit intoceph:mainfrom
clwluvw:duplicate-prio

Conversation

@clwluvw
Copy link
Member

@clwluvw clwluvw commented Feb 13, 2025

When multiple replication rules target the same bucket, the rule with the highest priority should be applied. While this logic is already in place, the configuration does not enforce priority settings, which could lead to confusion. This behavior aligns with AWS's implementation for multiple destination buckets as outlined https://aws.amazon.com/blogs/aws/new-amazon-s3-replication-adds-support-for-multiple-destination-buckets/

Fixes: https://tracker.ceph.com/issues/69946

When multiple replication rules target the same bucket, the rule
with the highest priority should be applied. While this logic is
already in place, the configuration does not enforce priority
settings, which could lead to confusion. This behavior aligns with
AWS's implementation for multiple destination buckets as outlined
https://aws.amazon.com/blogs/aws/new-amazon-s3-replication-adds-support-for-multiple-destination-buckets/

Fixes: https://tracker.ceph.com/issues/69946
Signed-off-by: Seena Fallah <[email protected]>
@clwluvw
Copy link
Member Author

clwluvw commented Feb 13, 2025

jenkins test make check

1 similar comment
@clwluvw
Copy link
Member Author

clwluvw commented Feb 14, 2025

jenkins test make check

@cbodley cbodley merged commit 2e89cd2 into ceph:main Mar 18, 2025
13 checks passed
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.

3 participants