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

Removed BFD settings from net-vpn-ha module as it is not supported #2244

Merged
merged 3 commits into from
Apr 28, 2024

Conversation

apichick
Copy link
Collaborator

Removed BFD settings from net-vpn-ha module as it is not supported

Please read here:

https://cloud.google.com/network-connectivity/docs/router/concepts/bfd#bfd_limitations

You can only enable BFD in BGP sessions that you configure for VLAN attachments in Dedicated Interconnect or Partner Interconnect. BFD is not supported in BGP sessions that are configured for HA VPN tunnels or for Router appliance, which is part of Network Connectivity Center.


Checklist

I applicable, I acknowledge that I have:

  • Read the contributing guide
  • Ran terraform fmt on all modified files
  • Regenerated the relevant README.md files using tools/tfdoc.py
  • Made sure all relevant tests pass

@wiktorn wiktorn added the incompatible change Pull request that breaks compatibility with previous version label Apr 27, 2024
@apichick apichick merged commit a95e681 into master Apr 28, 2024
13 checks passed
@apichick apichick deleted the vpn-bfd branch April 28, 2024 10:11
dibaskar-google pushed a commit to dibaskar-google/cloud-foundation-fabric that referenced this pull request May 14, 2024
…oogleCloudPlatform#2244)

* Removed bfd settings from net-vpn-ha as it is not supported

* Removed bfd settings from net-vpn-ha as it is not supported
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
incompatible change Pull request that breaks compatibility with previous version on:modules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants