interpolate vpc name in routes

.gitignore

@@ -11,3 +11,4 @@ backend.tf
 backend-config.hcl
 credentials.json
 key.json
+terraform-ls.tf

CHANGELOG.md

@@ -4,6 +4,8 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+- **incompatible change** routes in the `net-vpc` module now interpolate the VPC name to ensure uniqueness, upgrading from a previous version will drop and recreate routes
+
 ## [2.0.0] - 2020-06-11
 
 - new `data-solutions` section and `cmek-via-centralized-kms` example 

modules/net-vpc/main.tf

@@ -154,7 +154,7 @@ resource "google_compute_route" "gateway" {
   for_each         = local.routes_gateway
   project          = var.project_id
   network          = google_compute_network.network.name
-  name             = each.key
+  name             = "${var.name}-${each.key}"
   description      = "Terraform-managed."
   dest_range       = each.value.dest_range
   priority         = each.value.priority
@@ -166,7 +166,7 @@ resource "google_compute_route" "ilb" {
   for_each     = local.routes_ilb
   project      = var.project_id
   network      = google_compute_network.network.name
-  name         = each.key
+  name         = "${var.name}-${each.key}"
   description  = "Terraform-managed."
   dest_range   = each.value.dest_range
   priority     = each.value.priority
@@ -178,7 +178,7 @@ resource "google_compute_route" "instance" {
   for_each          = local.routes_instance
   project           = var.project_id
   network           = google_compute_network.network.name
-  name              = each.key
+  name              = "${var.name}-${each.key}"
   description       = "Terraform-managed."
   dest_range        = each.value.dest_range
   priority          = each.value.priority
@@ -192,7 +192,7 @@ resource "google_compute_route" "ip" {
   for_each    = local.routes_ip
   project     = var.project_id
   network     = google_compute_network.network.name
-  name        = each.key
+  name        = "${var.name}-${each.key}"
   description = "Terraform-managed."
   dest_range  = each.value.dest_range
   priority    = each.value.priority
@@ -204,7 +204,7 @@ resource "google_compute_route" "vpn_tunnel" {
   for_each            = local.routes_vpn_tunnel
   project             = var.project_id
   network             = google_compute_network.network.name
-  name                = each.key
+  name                = "${var.name}-${each.key}"
   description         = "Terraform-managed."
   dest_range          = each.value.dest_range
   priority            = each.value.priority

tests/modules/net_vpc/test_plan.py

@@ -86,5 +86,5 @@ def test_vpc_routes(plan_runner):
     _, resources = plan_runner(FIXTURES_DIR, routes=_var_routes)
     assert len(resources) == 3
     resource = [r for r in resources if r['values']
-                ['name'] == 'next-hop-test'][0]
+                ['name'] == 'my-vpc-next-hop-test'][0]
     assert resource['values']['next_hop_%s' % next_hop_type]