Skip to content

Comments

rgw: set dest bucket id in bucket sync checkpoint#61406

Merged
ivancich merged 1 commit intoceph:mainfrom
clwluvw:checkpoint-dstbucket
Jan 17, 2025
Merged

rgw: set dest bucket id in bucket sync checkpoint#61406
ivancich merged 1 commit intoceph:mainfrom
clwluvw:checkpoint-dstbucket

Conversation

@clwluvw
Copy link
Member

@clwluvw clwluvw commented Jan 15, 2025

bucket_source_sync_checkpoint() requires bucket id to lookup for full-sync object and the dest pipe doesn't have it. by replcaning the bucket key in the pipe with the bucket info we will ensure the bucket id is set.

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

bucket_source_sync_checkpoint() requires bucket id to lookup for
full-sync object and the dest pipe doesn't have it. by replcaning
the bucket key in the pipe with the bucket info we will ensure the
bucket id is set.

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

clwluvw commented Jan 16, 2025

jenkins test make check

@ivancich ivancich added the wip-eric-testing-2 for ivancich testing label Jan 16, 2025
@ivancich ivancich merged commit 3b96a09 into ceph:main Jan 17, 2025
13 checks passed
@ivancich ivancich removed needs-qa wip-eric-testing-2 for ivancich testing labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants