Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: Added support for S3 bucket object ownership #101

Merged

Conversation

carnei-ro
Copy link
Contributor

Description

Allow configuring "Object Ownership"

How Has This Been Tested?

  • I have tested and validated these changes using one or more of the provided examples/* projects

@utako
Copy link

utako commented Jul 21, 2021

would love to see this get merged soon.

@hiloboy0119
Copy link

I'm currently doing this outside of the module and trying to update Ownership Controls and Public Access Policy at the same time results in the error

A conflicting conditional operation is currently in progress against this resource. Please try again.

You will need to set up a dependency to keep them from running at the same time.

@antonbabenko antonbabenko changed the title feat: object_ownership feat: Added support for S3 bucket object ownership Aug 26, 2021
@antonbabenko antonbabenko merged commit c940e67 into terraform-aws-modules:master Aug 26, 2021
@antonbabenko
Copy link
Member

v2.8.0 has been just released.

@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 28, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants