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
integration tests for ec2_vpc_net #33111
Conversation
cc @defionscode @willthames |
d737f9a
to
7a0c3e0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great in general - can you just update the ec2-policy.json (or merge #33122 and then update compute-policy.json) to add the DHCP permissions?
@willthames Thanks for the review, I will update compute-policy.json! |
cc347a8
to
49daca9
Compare
@willthames Updated the test policy, do you mind updating your review? Not mergeable yet because I'm waiting for my PR for CI permissions/clean up to be approved. |
fix typo ensure the DHCP option set is cleaned up
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests look really good and pass for me locally against us-east-2 after rerunning setup-iam.yml
SUMMARY
I'm porting this module to boto3 and adding a feature to expand existing VPCs in #33105. These tests should be reviewed/merged first to ensure backwards compatibility.
ISSUE TYPE
COMPONENT NAME
test/integration/targets/ec2_vpc_net
ANSIBLE VERSION