Skip to content
This repository has been archived by the owner on Sep 16, 2020. It is now read-only.

Commit

Permalink
Revert "Centralise logic for VM tag name."
Browse files Browse the repository at this point in the history
This reverts commit d2217ee.
  • Loading branch information
bot committed Nov 11, 2016
1 parent 4e5a4c9 commit 5759c09
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 9 deletions.
4 changes: 2 additions & 2 deletions cf_internal_networking.tf
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,10 @@ resource "google_compute_firewall" "cf-internal" {
protocol = "udp"
}

target_tags = ["${var.vm_tag}"]
target_tags = ["${var.env_name}-vms"]

source_tags = [
"${var.vm_tag}", // Allows VMs deployed by OpsMan to talk to one another
"${var.env_name}-vms", // Allows VMs deployed by OpsMan to talk to one another
"${var.env_name}-ops-manager-external",
] // Allows bosh-init running on the OpsMan VM to SSH to the deployed BOSH Director
}
2 changes: 1 addition & 1 deletion outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ output "service_account_key" {
}

output "vm_tag" {
value = "${var.vm_tag}"
value = "${var.env_name}-vms"
}

output "network_name" {
Expand Down
6 changes: 0 additions & 6 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,6 @@ variable "env_name" {
type = "string"
}

variable "vm_tag" {
type = "string"
description = "tag used allow internal network access"
default = "${var.env_name}-vms"
}

variable "region" {
type = "string"
}
Expand Down

0 comments on commit 5759c09

Please sign in to comment.