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

[#286] Fix: Terraform plan can't be applied because of VPC endpoint misconfiguration #309

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Nihisil
Copy link
Contributor

@Nihisil Nihisil commented Nov 29, 2024

What happened 👀

Insight 📝

VPC endpoints are a good way to optimize because they lower traffic costs. However, they also come with hourly charges, and in cases where there is not much traffic, using a VPC endpoint might end up being more expensive than just paying for the traffic. I believe the decision to add a VPC endpoint should be made on a per-project basis and should not be included in the template by default. For most projects, it makes sense to have VPC endpoints for development with regular deployments and debug logs, but for staging and production, it might be better not to use VPC endpoints.

Proof Of Work 📹

Before the fix we weren't able to apply TF plan generated from our code:

image

After the fix TF plan can be applied without error ✅

Copy link
Member

@malparty malparty left a comment

Choose a reason for hiding this comment

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

Thanks for your insights. This makes sense indeed 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type : bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix: Terraform plan can't be applied because of VPC endpoint misconfiguration
4 participants