Project

General

Profile

Bug #11256

rgw: object set attrs clobbers object removal bucket index update

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

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

0%

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

Description

  • fix in master commit:4e6a66b and commit:edc0627

Associated revisions

Revision 7538319d (diff)
Added by Yehuda Sadeh about 9 years ago

rgw: generate new tag for object when setting object attrs

Fixes: #11256
Backport: firefly, hammer

Beforehand we were reusing the object's tag, which is problematic as
this tag is used for bucket index updates, and we might be clobbering a
racing update (like object removal).

Signed-off-by: Yehuda Sadeh <>

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

cls_rgw: use multimap to keep pending operations in bucket index

Fixes: #11256
Multiple concurrent requests might be sent using the same tag, need the
entry map to be able to hold multiple entries.

Signed-off-by: Yehuda Sadeh <>

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

rgw: generate new tag for object when setting object attrs

Fixes: #11256
Backport: firefly, hammer

Beforehand we were reusing the object's tag, which is problematic as
this tag is used for bucket index updates, and we might be clobbering a
racing update (like object removal).

Signed-off-by: Yehuda Sadeh <>

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

cls_rgw: use multimap to keep pending operations in bucket index

Fixes: #11256
Multiple concurrent requests might be sent using the same tag, need the
entry map to be able to hold multiple entries.

Signed-off-by: Yehuda Sadeh <>

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

rgw: generate new tag for object when setting object attrs

Fixes: #11256
Backport: firefly, hammer

Beforehand we were reusing the object's tag, which is problematic as
this tag is used for bucket index updates, and we might be clobbering a
racing update (like object removal).

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

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

cls_rgw: use multimap to keep pending operations in bucket index

Fixes: #11256
Multiple concurrent requests might be sent using the same tag, need the
entry map to be able to hold multiple entries.

Signed-off-by: Yehuda Sadeh <>
(cherry picked from commit 4e6a66b55e73c01347fc3330faa5c1307d29e9d3)

History

#1 Updated by Ian Colle almost 9 years ago

  • Status changed from New to 7

#2 Updated by Yehuda Sadeh almost 9 years ago

  • Status changed from 7 to Fix Under Review
  • Assignee changed from Yehuda Sadeh to Josh Durgin
  • Backport changed from firefly to firefly, hammer

#3 Updated by Yehuda Sadeh almost 9 years ago

  • Status changed from Fix Under Review to Pending Backport
fixes merged into hammer
  • hammer backport commit:7c7e651 and commit:cb75713

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

  • Description updated (diff)
  • Regression set to No

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

  • Description updated (diff)

#7 Updated by Nathan Cutler almost 9 years ago

both hammer and firefly backports appear to have been merged - does that mean this issue can be closed?

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

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF