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 aws_config tests #440

Merged
merged 2 commits into from
Feb 24, 2021
Merged

Conversation

tremble
Copy link
Contributor

@tremble tremble commented Feb 24, 2021

SUMMARY
  • use the standard role/resource names
  • test using updated policy (unsupported should work when using the additional 'unsupported' policies in aws-terminator

mattclay/aws-terminator#133

ISSUE TYPE
  • Tests Pull Request
COMPONENT NAME

aws_config

ADDITIONAL INFORMATION

@jillr jillr merged commit 8cb1164 into ansible-collections:main Feb 24, 2021
@tremble tremble deleted the tests/aws_config branch June 29, 2021 12:55
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* More explanation as to why aws_config is 'unsupported'

* Make sure aws_config tests create resources with the 'usual' names.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request Jul 19, 2021
* More explanation as to why aws_config is 'unsupported'

* Make sure aws_config tests create resources with the 'usual' names.
danielcotton pushed a commit to danielcotton/community.aws that referenced this pull request Nov 23, 2021
* More explanation as to why aws_config is 'unsupported'

* Make sure aws_config tests create resources with the 'usual' names.
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
…s#429)

Disable ec2_key, ec2_snapshot and ec2_group tests

SUMMARY
Work around ansible-collections#428 and ansible-collections#440 ansible-collections#441
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_key
ec2_snapshot
ec2_group
ADDITIONAL INFORMATION

Reviewed-by: Jill R <None>
Reviewed-by: None <None>
Reviewed-by: Paul Belanger <None>
Reviewed-by: Mark Chappell <None>
abikouo pushed a commit to abikouo/community.aws that referenced this pull request Oct 24, 2023
…collections#878)

ec2_group - Add egress_rules and purge_egress_rules aliases

SUMMARY
Fix ansible-collections#440 - exception running with --diff and --check when creating a new SG by adding a rule referencing a missing SG.
Add egress_rules and purge_egress_rules aliases - while cleaning up the tests I kept writing "egress_rules" rather than "rules_egress".
Cleanup and re-enable ec2_group tests

Remove "Classic Networking" tests (we don't have this in CI and it'll be gone in August)
Fix Group/VPC deletion in tests (cross-referenced rules weren't being deleted, wipe the rules first)

ISSUE TYPE

Feature Pull Request

COMPONENT NAME
ec2_group
ADDITIONAL INFORMATION
fixes: ansible-collections#440

Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants