Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pacific: rgw/putobj: RadosWriter uses part head object for multipart parts #55586

Merged
merged 1 commit into from Feb 15, 2024

Conversation

cbodley
Copy link
Contributor

@cbodley cbodley commented Feb 14, 2024

backport tracker: https://tracker.ceph.com/issues/64427


backport of #55582
parent tracker: https://tracker.ceph.com/issues/63642

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@cbodley
Copy link
Contributor Author

cbodley commented Feb 14, 2024

cbodley wants to merge 10,000 commits

oops

the cleanup logic in the RadosWrite destructor was using the wrong
`head_obj` to avoid races between cleanup and part re-uploads. it
pointed at the final location of the multipart upload, rather than the
head object of the current part

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

Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit bbbf5c5)

Conflicts:
	rgw_putobj_processor.* uses sal::RGWObject instead of rgw_obj
@cbodley cbodley removed the request for review from a team February 14, 2024 16:29
@yuriw yuriw merged commit 60a1053 into ceph:pacific Feb 15, 2024
7 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants