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

rgw: policy: modify some operation permission keyword #20974

Merged
merged 1 commit into from May 9, 2018

Conversation

xiangxiang222
Copy link
Contributor

@xiangxiang222 xiangxiang222 commented Mar 20, 2018

according to AWS S3, the permission keyword should be s3ListBucketMultipartUploads
rather than s3:ListBucketMultiPartUploads.

all operation permission list in AWS S3 as follows:
https://docs.aws.amazon.com/AmazonS3/latest/dev/using-with-s3-actions.html?shortFooter=true

Fixes: http://tracker.ceph.com/issues/24061
Signed-off-by: xiangxiang xiangxiang@xsky.com

@cbodley cbodley added the rgw label Mar 20, 2018
…tipartUploads

according to AWS S3, the permission keyword should be s3:ListBucketMultipartUploads
rather than s3:ListBucketMultiPartUploads.

all operation permission list in AWS S3 as follows:
  https://docs.aws.amazon.com/AmazonS3/latest/dev/using-with-s3-actions.html?shortFooter=true

Signed-off-by: xiangxiang <xiangxiang@xsky.com>
@adamemerson adamemerson merged commit 89d1de6 into ceph:master May 9, 2018
@cbodley
Copy link
Contributor

cbodley commented May 9, 2018

@adamemerson we'll need a tracker issue for this, or the backport team won't know about it (cc @smithfarm)

@smithfarm
Copy link
Contributor

@adamemerson What @cbodley said - please follow the established workflow for flagging bugfixes for backport - i.e. create a tracker issue and fill out the "Backport" field with the stable versions you think it should be backported to - e.g. "luminous, jewel".

(I periodically ask @liewegas to delete backport-related github labels, but they keep coming back. . .)

@smithfarm
Copy link
Contributor

In this case, the status of the tracker issue should be set to "Pending Backport", since the fix is already merged into master.

@theanalyst
Copy link
Member

created http://tracker.ceph.com/issues/24095

@smithfarm
Copy link
Contributor

@theanalyst http://tracker.ceph.com/issues/24061 was already there but I neglected to crosslink it properly - should all be good now

@theanalyst
Copy link
Member

ah ok thanks for noticing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
5 participants