Ignore vpc endpoint routes

[ojongerius]

Fixes #2063

[radeksimko]

Thanks for noticing.

I feel we should add a proper support for VPC endpoints, but there's no support in the upstream AWS Go SDK yet.

I will keep asking the AWS support.

[ojongerius]

@radeksimko cheers!

Terraform support for S3 VPC endpoints would be amazing, it makes a HUGE difference the amount of traffic flowing through our routers. Entertainingly enough we noticed Cloudformation currently also chokes on route tables containing S3 VPC endpoint routes object.

This patch at least fixes the current situation.

[gjohnson]

Looks like it was added upstream this week. Be awesome to have this support.

[radeksimko]

@gjohnson Yes, "we ask for it, we get it". 😃

I already started working on EFS which was added in the same commit and unless someone will be quicker, I may start working on VPC endpoints as well after finishing EFS, so far it's open for everyone (it always is though :) ).

[gjohnson]

@radeksimko I may have time to add endpoints while your on EFS, ping you on the PR if I get to it.

[radeksimko]

@ojongerius Thanks, but @gjohnson came up with slightly cleaner solution to this issue.
We also now support VPC endpoints, so these don't have to be managed outside Terraform.

See #2695 which has been just merged.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.