Project

General

Profile

Bug #11622

rgw: broken manifest on multipart upload retry

Added by Yehuda Sadeh almost 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Immediate
Assignee:
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
hammer, firefly
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Specifically if the retry happens on the first part, but there might be other cases when this applies.

Associated revisions

Revision 389ae673 (diff)
Added by Yehuda Sadeh almost 9 years ago

rgw: merge manifests correctly when there's prefix override

Fixes: #11622
Backport: hammer, firefly

Prefix override happens in a manifest when a rados object does not
conform to the generic prefix set on the manifest. When merging
manifests (specifically being used in multipart objects upload), we need
to check if the rule that we try to merge has a prefix that is the same
as the previous rule. Beforehand we checked if both had the same
override_prefix setting, but that might not apply as both manifests
might have different prefixes.

Signed-off-by: Yehuda Sadeh <>

Revision cd8f1830 (diff)
Added by Yehuda Sadeh almost 9 years ago

rgw: merge manifests correctly when there's prefix override

Fixes: #11622
Backport: hammer, firefly

Prefix override happens in a manifest when a rados object does not
conform to the generic prefix set on the manifest. When merging
manifests (specifically being used in multipart objects upload), we need
to check if the rule that we try to merge has a prefix that is the same
as the previous rule. Beforehand we checked if both had the same
override_prefix setting, but that might not apply as both manifests
might have different prefixes.

Signed-off-by: Yehuda Sadeh <>
(cherry picked from commit 389ae6739ddc6239a4dd7c5f7f9bfc9b645b8577)

Revision 293affe9 (diff)
Added by Yehuda Sadeh almost 9 years ago

rgw: merge manifests correctly when there's prefix override

Fixes: #11622
Backport: hammer, firefly

Prefix override happens in a manifest when a rados object does not
conform to the generic prefix set on the manifest. When merging
manifests (specifically being used in multipart objects upload), we need
to check if the rule that we try to merge has a prefix that is the same
as the previous rule. Beforehand we checked if both had the same
override_prefix setting, but that might not apply as both manifests
might have different prefixes.

Signed-off-by: Yehuda Sadeh <>
(cherry picked from commit 389ae6739ddc6239a4dd7c5f7f9bfc9b645b8577)

History

#1 Updated by Sage Weil almost 9 years ago

  • Priority changed from Urgent to Immediate

#5 Updated by Loïc Dachary almost 9 years ago

  • Backport set to hammer, firefly

#6 Updated by Loïc Dachary almost 9 years ago

  • Status changed from 12 to Pending Backport

#7 Updated by Loïc Dachary almost 9 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF