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 new "require_boto(3|core)_at_least() helpers #446

Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Aug 6, 2021

SUMMARY

We currently have lots of different ways we output that we needed a library, add a helper for our standard use-case of minimum botocore versions for a feature.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/module_utils/core.py
plugins/modules/ec2_vol.py

ADDITIONAL INFORMATION

@ansibullbot
Copy link

@tremble tremble force-pushed the boto3/require_at_least branch 2 times, most recently from 5bbaa4e to 8ca2372 Compare August 6, 2021 16:16
@tremble tremble requested review from alinabuzachis and jillr August 6, 2021 16:18
@tremble tremble force-pushed the boto3/require_at_least branch from 8ca2372 to 9ea0249 Compare August 6, 2021 17:16
@tremble
Copy link
Contributor Author

tremble commented Aug 6, 2021

recheck

@tremble tremble added the gate label Aug 9, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

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

LGTM!

@ansible-zuul ansible-zuul bot merged commit 2574ca4 into ansible-collections:main Aug 9, 2021
ansible-zuul bot added a commit to ansible-collections/community.aws that referenced this pull request Aug 10, 2021
Cleanup boto3/botocore dependencies - remove tests for unsupported versions

SUMMARY
Testing for boto3/botocore support on specific functions is very inconsistent.  Now that we have minimum supported versions we can drop the tests for previous versions.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/aws_eks_cluster.py
plugins/modules/aws_msk_cluster.py
plugins/modules/aws_msk_config.py
plugins/modules/cloudformation_stack_set.py
plugins/modules/dynamodb_table.py
plugins/modules/dynamodb_ttl.py
plugins/modules/ec2_ami_copy.py
plugins/modules/ec2_asg.py
plugins/modules/ec2_launch_template.py
plugins/modules/ec2_transit_gateway.py
plugins/modules/ec2_transit_gateway_info.py
plugins/modules/ec2_vpc_peer.py
plugins/modules/ecs_ecr.py
plugins/modules/ecs_service.py
plugins/modules/ecs_task.py
plugins/modules/ecs_taskdefinition.py
plugins/modules/efs.py
plugins/modules/efs_info.py
plugins/modules/elb_target_group.py
plugins/modules/iam_role.py
plugins/modules/lambda.py
plugins/modules/rds_instance.py
plugins/modules/sqs_queue.py
ADDITIONAL INFORMATION
Depends-on: ansible-collections/amazon.aws#442
Depends-on: ansible-collections/amazon.aws#446

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 12, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 13, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 23, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit that referenced this pull request Aug 25, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit that referenced this pull request Aug 25, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit that referenced this pull request Aug 26, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit that referenced this pull request Aug 26, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
alinabuzachis added a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 26, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
jillr pushed a commit to alinabuzachis/amazon.aws that referenced this pull request Aug 27, 2021
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>

This commit was initially merged in https://github.com/ansible-collections/community.aws
See: ansible-collections/community.aws@52fb230
@tremble tremble deleted the boto3/require_at_least branch November 26, 2021 12:42
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
Cleanup boto3/botocore dependencies - remove tests for unsupported versions

SUMMARY
Testing for boto3/botocore support on specific functions is very inconsistent.  Now that we have minimum supported versions we can drop the tests for previous versions.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
plugins/modules/aws_eks_cluster.py
plugins/modules/aws_msk_cluster.py
plugins/modules/aws_msk_config.py
plugins/modules/cloudformation_stack_set.py
plugins/modules/dynamodb_table.py
plugins/modules/dynamodb_ttl.py
plugins/modules/ec2_ami_copy.py
plugins/modules/ec2_asg.py
plugins/modules/ec2_launch_template.py
plugins/modules/ec2_transit_gateway.py
plugins/modules/ec2_transit_gateway_info.py
plugins/modules/ec2_vpc_peer.py
plugins/modules/ecs_ecr.py
plugins/modules/ecs_service.py
plugins/modules/ecs_task.py
plugins/modules/ecs_taskdefinition.py
plugins/modules/efs.py
plugins/modules/efs_info.py
plugins/modules/elb_target_group.py
plugins/modules/iam_role.py
plugins/modules/lambda.py
plugins/modules/rds_instance.py
plugins/modules/sqs_queue.py
ADDITIONAL INFORMATION
Depends-on: ansible-collections#442
Depends-on: ansible-collections#446

Reviewed-by: Markus Bergholz <[email protected]>
Reviewed-by: Alina Buzachis <None>
Reviewed-by: None <None>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Sep 18, 2023
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>
abikouo pushed a commit to abikouo/amazon.aws that referenced this pull request Oct 24, 2023
* Solve RequestLimitExceeded error by adding the retry decorator

Signed-off-by: Alina Buzachis <[email protected]>

Co-authored-by: Alina Buzachis <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review feature This issue/PR relates to a feature request module_utils module_utils module module needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants