Project

General

Profile

Bug #7110

rgw: cannot set acls on pre-dumpling buckets

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

Status:
Resolved
Priority:
Normal
Assignee:
Yehuda Sadeh
Target version:
-
% Done:

0%

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

Associated revisions

Revision a5f8cc7e (diff)
Added by Yehuda Sadeh over 9 years ago

rgw: convert bucket info if needed

Fixes: #7110
In dumpling, the bucket info was separated into bucket entry point and
bucket instance objects. When setting bucket attrs we only ended up
updating the bucket instance object. However, pre-dumpling buckets still
keep everything at the entry-point object, so acl changes didn't affect
anything (because we never updated the entry point). This change just
converts the bucket info into the new format.

Signed-off-by: Yehuda Sadeh <>

Revision ee9fc590 (diff)
Added by Yehuda Sadeh over 9 years ago

rgw: convert bucket info if needed

Fixes: #7110
In dumpling, the bucket info was separated into bucket entry point and
bucket instance objects. When setting bucket attrs we only ended up
updating the bucket instance object. However, pre-dumpling buckets still
keep everything at the entry-point object, so acl changes didn't affect
anything (because we never updated the entry point). This change just
converts the bucket info into the new format.

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

Revision 72168f71 (diff)
Added by Yehuda Sadeh over 9 years ago

rgw: convert bucket info if needed

Fixes: #7110
In dumpling, the bucket info was separated into bucket entry point and
bucket instance objects. When setting bucket attrs we only ended up
updating the bucket instance object. However, pre-dumpling buckets still
keep everything at the entry-point object, so acl changes didn't affect
anything (because we never updated the entry point). This change just
converts the bucket info into the new format.

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

History

#1 Updated by Ian Colle over 9 years ago

  • Status changed from New to Won't Fix

#2 Updated by Ian Colle over 9 years ago

  • Status changed from Won't Fix to 12

Buckets created on versions prior to Dumpling cannot have ACLs set, even after upgrading to Dumpling.

#3 Updated by Josh Durgin over 9 years ago

The fix in wip-7110-dumpling looks good to me.

#4 Updated by Ian Colle over 9 years ago

  • Status changed from 12 to 7
  • Assignee set to Yehuda Sadeh

#5 Updated by Sage Weil over 9 years ago

  • Status changed from 7 to Resolved

#6 Updated by Yehuda Sadeh over 9 years ago

  • Backport set to dumpling, emperor

still need to backport it to dumpling and emperor

#7 Updated by Yehuda Sadeh over 9 years ago

  • Status changed from Resolved to Pending Backport

#8 Updated by Ian Colle over 9 years ago

Landed to next 744895326c718efca720bf71e164d65e4eb29d32

#9 Updated by Ian Colle over 9 years ago

  • Status changed from Pending Backport to Resolved

Landed to dumpling - ee9fc590a556354b54a174a4790c4142949d7409
Landed to emperor - 72168f717b2632f2942fcabf69798ff59ab89411

Also available in: Atom PDF