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

Migrate ec2_vpc_nat_gateway* modules and tests #652

Conversation

alinabuzachis
Copy link
Contributor

@alinabuzachis alinabuzachis commented Jul 20, 2021

Depends-On: ansible-collections/amazon.aws#410

Remove ec2_vpc_nat_gateway* modules and tests
These modules have been migrated to amazon.aws
Update runtime.yml with redirects to that collection
Update ignore files

@alinabuzachis alinabuzachis requested a review from jillr July 20, 2021 13:34
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch from ac8f4f3 to 040fd94 Compare July 21, 2021 12:32
@alinabuzachis
Copy link
Contributor Author

recheck

@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch 2 times, most recently from cab3aab to b4ccb16 Compare August 12, 2021 13:37
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch 6 times, most recently from 68c9ca2 to 2c8087c Compare August 13, 2021 13:35
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch from 2c8087c to 79ffb9c Compare August 23, 2021 14:32
@alinabuzachis
Copy link
Contributor Author

recheck

@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch from 79ffb9c to 3d2fc7c Compare August 23, 2021 17:11
@alinabuzachis
Copy link
Contributor Author

recheck

@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch from 3d2fc7c to 2be1d8e Compare August 23, 2021 19:47
@alinabuzachis alinabuzachis changed the title DNM Migrate ec2_vpc_nat_gateway* modules and tests Migrate ec2_vpc_nat_gateway* modules and tests Aug 24, 2021
@alinabuzachis alinabuzachis force-pushed the promote_ec2_vpc_nat_gateway branch from 2be1d8e to 971c572 Compare August 26, 2021 13:26
@alinabuzachis alinabuzachis requested a review from tremble August 26, 2021 13:28
@alinabuzachis
Copy link
Contributor Author

recheck

@tremble
Copy link
Contributor

tremble commented Aug 27, 2021

ansible-collections/amazon.aws#410 has merged. gating this one.

@tremble tremble added the gate label Aug 27, 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 96552f9 into ansible-collections:main Aug 27, 2021
alinabuzachis pushed a commit to alinabuzachis/community.aws that referenced this pull request May 25, 2022
Update ec2_instance.py

SUMMARY
vpc_subnet_id is not configurable under the network key, just tried a copy of the example and got this error->
TASK [manage_ec2_instances : Create EC2 instances for ansible node (control node)] **********************************************************************************************************
fatal: [localhost]: FAILED! => changed=false
  msg: No default subnet could be found - you must include a VPC subnet ID (vpc_subnet_id parameter) to create an instance

changing to match other examples
ISSUE TYPE

Docs Pull Request

COMPONENT NAME
ec2_instance
ADDITIONAL INFORMATION
n/a, only see one bad example here

Reviewed-by: Alina Buzachis <None>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review docs integration tests/integration module module plugins plugin (any type) tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants