Skip to content

Use directories.new when getting S3 directory

Jan Provaznik requested to merge s3-directories-get into master

What does this MR do?

Same as https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/6682:

Calling Fog::Storage::AWS::Directories#get requires the ListAllMyBuckets permission, but we can avoid that extra query and permission by initializing the directory with a specific bucket: https://stackoverflow.com/a/12288581/1992201

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-com/gl-infra/infrastructure/issues/5807

Does this MR meet the acceptance criteria?

Edited by Kamil Trzciński

Merge request reports