Skip to content

Conversation

dominikschubert
Copy link
Member

Adds support for the route.destination-cidr-block filter in EC2.DescribeRouteTables

See: https://docs.aws.amazon.com/AWSEC2/latest/APIReference/API_DescribeRouteTables.html

Can be removed once https://github.com/getmoto/moto/pull/5932/files is merged upstream and we have updated our fork.

@dominikschubert dominikschubert self-assigned this Feb 16, 2023
@dominikschubert dominikschubert temporarily deployed to localstack-ext-tests February 16, 2023 08:58 — with GitHub Actions Inactive
Copy link
Member

@whummer whummer left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for the quick fix @dominikschubert! (based on @viren-nadkarni 's upstream PR) 🚀

@github-actions
Copy link

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   1h 28m 11s ⏱️ - 5m 17s
1 727 tests +1  1 369 ✔️ +1  358 💤 ±0  0 ±0 
2 441 runs  +1  1 743 ✔️ +1  698 💤 ±0  0 ±0 

Results for commit 9fcf3c8. ± Comparison against base commit ce0f939.

@coveralls
Copy link

Coverage Status

Coverage: 84.978% (-0.007%) from 84.984% when pulling 9fcf3c8 on ec2-routetable-filter-destinationcidrblock into ce0f939 on master.

@dominikschubert dominikschubert merged commit 7159b90 into master Feb 16, 2023
@dominikschubert dominikschubert deleted the ec2-routetable-filter-destinationcidrblock branch February 16, 2023 11:07
viren-nadkarni added a commit that referenced this pull request Mar 7, 2023
viren-nadkarni added a commit that referenced this pull request Mar 7, 2023
thrau pushed a commit that referenced this pull request Mar 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants