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

Add additional storage classes to S3 lifecycle transition list. #270

Merged
merged 3 commits into from
Oct 20, 2020

Conversation

philltomlinson
Copy link
Contributor

No description provided.

Copy link
Contributor

@tremble tremble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately the integration tests are apparently broken (not because of this change)

Rather than blocking a very simple improvement on fixing the tests I think we can let this one through on the basis of the updated docs:

https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/s3.html#S3.BucketLifecycleConfiguration.put

@philltomlinson One small thing: Please add a "minor changes" changelog entry: https://docs.ansible.com/ansible/latest/community/development_process.html#changelogs-how-to

@tremble tremble merged commit 073353b into ansible-collections:main Oct 20, 2020
@tremble
Copy link
Contributor

tremble commented Oct 20, 2020

@philltomlinson Thank you for taking the time to raise this PR.

alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…ble-collections#270)

* Add additional storage classes to S3 lifecycle transition list.
* Add minor changes changelog for S3 lifecycle transition list change.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
…ble-collections#270)

* Add additional storage classes to S3 lifecycle transition list.
* Add minor changes changelog for S3 lifecycle transition list change.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…s#270)

* Add vpc_available, vpc_exists and subnet_available waiters
* ec2_vpc_subnet use retry decorators more consistently
* ec2_vpc_net: Use custom waiter that handles API Ratelimits
* Move Client/BotoCore Error catching inside get_classic_link function
* Catch WaiterError to give a better error message
* changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants