Skip to content

Adding tags to ha_vpn_gateway_gcp_to_gcp#952

Open
Gorlami96 wants to merge 2 commits into
terraform-google-modules:mainfrom
Gorlami96:ha-vpn-terraform
Open

Adding tags to ha_vpn_gateway_gcp_to_gcp#952
Gorlami96 wants to merge 2 commits into
terraform-google-modules:mainfrom
Gorlami96:ha-vpn-terraform

Conversation

@Gorlami96

@Gorlami96 Gorlami96 commented May 13, 2026

Copy link
Copy Markdown

Description

Fixes b/358215501

Added tags and rearranged terraform code for VPN resource to be more in line with the gcloud documentation.

Note: If you are not associated with Google, open an issue for discussion before submitting a pull request.

Checklist

Readiness

  • Yes, merge this PR after it is approved
  • No, don't merge this PR after it is approved

Style

Testing

Intended location

  • Yes, this sample will be (or already is) included on cloud.google.com
    Location(s):

  • No, this sample won't be included on cloud.google.com
    Reason:

API enablement

  • If the sample needs an API enabled to pass testing, I have added the service to the Test setup file

Review

  • If this sample adds a new directory, I have added codeowners to the CODEOWNERS file

@Gorlami96 Gorlami96 requested review from a team as code owners May 13, 2026 06:54
@snippet-bot

snippet-bot Bot commented May 13, 2026

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 5 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@Gorlami96

Copy link
Copy Markdown
Author

Not sure why the previous PR was closed #951

Created a new one. Please let me know if anything is missing!

@Gorlami96 Gorlami96 marked this pull request as draft July 16, 2026 06:28
@Gorlami96 Gorlami96 marked this pull request as ready for review July 16, 2026 06:28
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