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

s3_bucket - be more forgiving when missing 'get' permissions for a parameter we're not trying to set #1406

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Mar 3, 2023

SUMMARY

S3 permissions are painfully complex. Be as forgiving as possible when missing a 'get' permission on a parameter we're not trying to set.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

s3_bucket

ADDITIONAL INFORMATION

Related to #1352

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.
https://ansible.softwarefactory-project.io/zuul/buildset/997605a103564022ad2c8311f5d68965

✔️ ansible-galaxy-importer SUCCESS in 5m 23s
✔️ build-ansible-collection SUCCESS in 12m 44s
✔️ ansible-test-sanity-aws-ansible-python38 SUCCESS in 12m 24s (non-voting)
✔️ ansible-test-sanity-aws-ansible-2.12-python38 SUCCESS in 12m 31s
✔️ ansible-test-sanity-aws-ansible-2.13-python38 SUCCESS in 12m 28s
✔️ ansible-test-sanity-aws-ansible-2.14 SUCCESS in 12m 18s
✔️ ansible-test-units-amazon-aws-python36 SUCCESS in 10m 28s
✔️ ansible-test-units-amazon-aws-python38 SUCCESS in 11m 36s
✔️ ansible-test-units-amazon-aws-python39 SUCCESS in 10m 43s
✔️ ansible-test-units-amazon-aws-python310 SUCCESS in 12m 10s
✔️ cloud-tox-py3 SUCCESS in 8m 24s
✔️ ansible-test-changelog SUCCESS in 4m 16s
✔️ ansible-test-splitter SUCCESS in 4m 46s
✔️ integration-amazon.aws-1 SUCCESS in 6m 32s
Skipped 43 jobs

@tremble tremble marked this pull request as ready for review March 6, 2023 12:18
@tremble tremble added the mergeit Merge the PR (SoftwareFactory) label Mar 6, 2023
@softwarefactory-project-zuul
Copy link
Contributor

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

✔️ ansible-galaxy-importer SUCCESS in 3m 59s
✔️ build-ansible-collection SUCCESS in 41m 44s
✔️ ansible-test-splitter SUCCESS in 16m 09s
✔️ integration-amazon.aws-1 SUCCESS in 6m 35s
Skipped 43 jobs

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit fe5a71f into ansible-collections:main Mar 6, 2023
@github-actions
Copy link

github-actions bot commented Mar 6, 2023

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 deleted the s3_bucket/permissions branch September 27, 2023 06:59
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.

2 participants