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

[PR #2222/0dbac7a7 backport][stable-8] s3_bucket - remove default for accelerate_enabled #2223

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Aug 21, 2024

This is a backport of PR #2222 as merged into main (0dbac7a).

SUMMARY

fixes #2220

Removes the default for accelerate_enabled:

  • having the default explicitly set to false breaks idempotency when only passed partial configuration
  • having the default explicitly set results in the module always failing on non-AWS platforms
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

s3_bucket

ADDITIONAL INFORMATION

SUMMARY
fixes #2220
Removes the default for accelerate_enabled:

having the default explicitly set to false breaks idempotency when only passed partial configuration
having the default explicitly set results in the module always failing on non-AWS platforms

ISSUE TYPE

Bugfix Pull Request

COMPONENT NAME
s3_bucket
ADDITIONAL INFORMATION

Reviewed-by: Alina Buzachis
(cherry picked from commit 0dbac7a)
Copy link

github-actions bot commented Aug 21, 2024

Docs Build 📝

Thank you for contribution!✨

This PR has been merged and your docs changes will be incorporated when they are next published.

@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Aug 21, 2024
Copy link
Contributor

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/331ea7818469409d8df5fc46ed4e3d9a

✔️ ansible-galaxy-importer SUCCESS in 4m 31s
✔️ build-ansible-collection SUCCESS in 10m 39s
✔️ ansible-test-splitter SUCCESS in 4m 19s
✔️ integration-amazon.aws-1 SUCCESS in 7m 07s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit 5e6b35c into stable-8 Aug 21, 2024
34 of 37 checks passed
@softwarefactory-project-zuul softwarefactory-project-zuul bot deleted the patchback/backports/stable-8/0dbac7a748f143811d5d3d82e5dd38b5e3aba3f0/pr-2222 branch August 21, 2024 16:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit Merge the PR (SoftwareFactory)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant