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

Cleanup boto3/botocore dependencies - remove tests for unsupported versions #675

Merged
merged 6 commits into from
Aug 10, 2021

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Aug 6, 2021

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

@ansibullbot ansibullbot added community_review feature This issue/PR relates to a feature request integration tests/integration module module needs_triage plugins plugin (any type) tests tests labels Aug 6, 2021
@tremble tremble force-pushed the boto3/minimums branch 2 times, most recently from 8c9cb32 to fbfa8f0 Compare August 6, 2021 14:20
@tremble
Copy link
Contributor Author

tremble commented Aug 7, 2021

recheck

@ansibullbot
Copy link

@tremble This PR was evaluated as a potentially problematic PR for the following reasons:

  • More than 50 changed files.

Such PR can only be merged by human. Contact a Core team member to review this PR on IRC: #ansible-devel on irc.freenode.net

click here for bot help

@ansibullbot ansibullbot added needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Aug 10, 2021
Copy link
Member

@markuman markuman left a comment

Choose a reason for hiding this comment

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

lgtm

@tremble tremble added the gate label Aug 10, 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!

@tremble tremble mentioned this pull request Aug 10, 2021
@ansible-zuul ansible-zuul bot merged commit 90dea24 into ansible-collections:main Aug 10, 2021
@tremble tremble deleted the boto3/minimums branch November 26, 2021 12:46
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…_contrib_script_compatible_ec2_tag_keys (ansible-collections#858)

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys

SUMMARY

Add version_added: 1.5.0 for exclude_filters, include_filters and use_contrib_script_compatible_ec2_tag_keys
include_filters and exclude_filters have been added ansible-collections#328 and released with amazon.aws 1.5.0
use_contrib_script_compatible_ec2_tag_keys has been added ansible-collections#331 and released with  amazon.aws 1.5.0
Let's update the aws_ec2 inventory plugin documentation with this information.
This should be a step towards closing this one ansible-collections#676 and ansible-collections#675
aws_ec2 documentation will be enriched with exhaustive examples in an upcoming PR.

ISSUE TYPE


Docs Pull Request

COMPONENT NAME

aws_ec2
ADDITIONAL INFORMATION

Reviewed-by: Mark Chappell <None>
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 integration tests/integration module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR needs_triage plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants