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

Bunch of modifications #1

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Bunch of modifications #1

wants to merge 6 commits into from

Conversation

DeLoWaN
Copy link

@DeLoWaN DeLoWaN commented Jun 23, 2021

Hello, I made a bunch of modifications on the module.

  • Remove security group definition: the purpose of this module is to only create a VPN connection. Additionnally, the GCP firewall rules were not present
  • Change static resource names to dynamic ones so that multiple VPN can be established
  • Compatible with Terraform > 0.13 only
  • Remove unused region names
  • Replace static routing AWS side with route propagation from Virtual Private Gateway
  • Add the possibility to route the GCP Cloud DNS CIDR through the VPN to allow Cloud DNS zone forwarding
  • Removes the requirement of route tables ids and allows to specify which gcp subnetwork will be routed
  • Updated Readme

Damien Gustave and others added 6 commits June 17, 2021 18:14
…only create a VPN connection. Additionnally, the GCP firewall rules were not present

* Change static resource names to dynamic ones so that multiple VPN can be established
* Compatible with Terraform > 0.13 only
* Remove unused region names
… Private Gateway

* Add the possibility to route the GCP Cloud DNS CIDR through the VPN to allow Cloud DNS zone forwarding
feat: add the possibilty to create a non-HA VPN (cut the cost)
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.

1 participant