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
[cloud] ec2_vpc_route_table: ignore routes without DestinationCidrBlock - fixes #37003 #37010
Conversation
c98fa6a
to
a88bd9a
Compare
Add module warnings rather than silently skipping
Should there be a test for this change? It does look good to me but a test would give me greater confidence before merging. |
Added a test to ensure a VPC endpoint associated with a route table doesn't result in traceback (which is what happens now but not prior to 2.5) with the ec2_vpc_route_table module. It's going to need CI permissions added. |
… not result in a traceback
Looks good, pending CI updates. |
bot_status |
Componentslib/ansible/modules/cloud/amazon/ec2_vpc_route_table.py test/integration/targets/ec2_vpc_route_table/tasks/main.yml Metadatawaiting_on: ansible |
This is definitely a backport contender. I'll cherry-pick |
…ck - fixes ansible#37003 (ansible#37010) * [cloud] ec2_vpc_route_table: ignore routes without DestinationCidrBlock Add module warnings rather than silently skipping * Permit warnings for routes tables containing vpc endpoints to be turned off * Add tests to ensure a VPC endpoint associated with a route table does not result in a traceback (cherry picked from commit da3f7a8)
…ck - fixes ansible#37003 (ansible#37010) * [cloud] ec2_vpc_route_table: ignore routes without DestinationCidrBlock Add module warnings rather than silently skipping * Permit warnings for routes tables containing vpc endpoints to be turned off * Add tests to ensure a VPC endpoint associated with a route table does not result in a traceback
…ck - fixes ansible#37003 (ansible#37010) * [cloud] ec2_vpc_route_table: ignore routes without DestinationCidrBlock Add module warnings rather than silently skipping * Permit warnings for routes tables containing vpc endpoints to be turned off * Add tests to ensure a VPC endpoint associated with a route table does not result in a traceback (cherry picked from commit 246ff9c)
…ck - fixes #37003 (#37010) (#39210) * [cloud] ec2_vpc_route_table: ignore routes without DestinationCidrBlock Add module warnings rather than silently skipping * Permit warnings for routes tables containing vpc endpoints to be turned off * Add tests to ensure a VPC endpoint associated with a route table does not result in a traceback (cherry picked from commit da3f7a8)
SUMMARY
Fixes #37003. Regression introduced when porting the module from boto to boto3.
ISSUE TYPE
COMPONENT NAME
lib/ansible/modules/cloud/amazon/ec2_vpc_route_table.py
ANSIBLE VERSION