Project

General

Profile

Actions

Feature #61887

open

s3: GetBucketLocation should also return placement target

Added by Casey Bodley 11 months ago. Updated 18 days ago.

Status:
Fix Under Review
Priority:
Normal
Target version:
-
% Done:

0%

Source:
Tags:
s3 placement low-hanging-fruit
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

rgw extends CreateBucket to allow the LocationConstraint to specify a placement target: https://docs.ceph.com/en/latest/radosgw/placement/#s3-bucket-placement

the GetBucketLocation API only returns the 'region' part of that LocationConstraint, so there is currently no way for an s3 client to query the placement target

the GetBucketLocation xml response only includes the <LocationConstraint> element, so we can't extend the response to add another field. a response header like x-rgw-bucket-placement-target is probably the best option for this

Actions #1

Updated by Casey Bodley 11 months ago

  • Tags changed from s3 placement to s3 placement low-hanging-fruit
Actions #2

Updated by Shreyansh Sancheti 8 months ago

Casey Bodley wrote:

rgw extends CreateBucket to allow the LocationConstraint to specify a placement target: https://docs.ceph.com/en/latest/radosgw/placement/#s3-bucket-placement

the GetBucketLocation API only returns the 'region' part of that LocationConstraint, so there is currently no way for an s3 client to query the placement target

the GetBucketLocation xml response only includes the <LocationConstraint> element, so we can't extend the response to add another field. a response header like x-rgw-bucket-placement-target is probably the best option for this

Can you provide more details ? Is it, that this should return the placement-target and should reflect while querying over the api endpoint.

Actions #3

Updated by Md Mahamudur Rahaman Sajib 18 days ago

  • Assignee set to Md Mahamudur Rahaman Sajib
Actions #4

Updated by Casey Bodley 18 days ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 57152
Actions

Also available in: Atom PDF