Update preemptible argument for Terraform 0.12

aaron-lane · aaron-lane · commit ceebc362d431 · 2019-07-31T10:56:02.000-04:00
diff --git a/modules/instance_template/main.tf b/modules/instance_template/main.tf
@@ -98,7 +98,6 @@ resource "google_compute_instance_template" "tpl" {
   }
 
   scheduling {
-    preemptible = "${var.preemptible}"
+    preemptible = var.preemptible
   }
 }
-
diff --git a/modules/instance_template/variables.tf b/modules/instance_template/variables.tf
@@ -42,8 +42,9 @@ variable "labels" {
 }
 
 variable "preemptible" {
+  type        = bool
   description = "Allow the instance to be preempted"
-  default     = "false"
+  default     = false
 }
 
 #######
@@ -134,4 +135,3 @@ variable "service_account" {
   })
   description = "Service account to attach to the instance. See https://www.terraform.io/docs/providers/google/r/compute_instance_template.html#service_account."
 }
-

Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,6 @@ resource "google_compute_instance_template" "tpl" {`
`98`	`98`	`}`
`99`	`99`
`100`	`100`	`scheduling {`
`101`		`- preemptible = "${var.preemptible}"`
	`101`	`+ preemptible = var.preemptible`
`102`	`102`	`}`
`103`	`103`	`}`
`104`		`-`
Original file line number	Diff line number	Diff line change
`@@ -42,8 +42,9 @@ variable "labels" {`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`variable "preemptible" {`
	`45`	`+ type = bool`
`45`	`46`	`description = "Allow the instance to be preempted"`
`46`		`- default = "false"`
	`47`	`+ default = false`
`47`	`48`	`}`
`48`	`49`
`49`	`50`	`#######`
`@@ -134,4 +135,3 @@ variable "service_account" {`
`134`	`135`	`})`
`135`	`136`	`description = "Service account to attach to the instance. See https://www.terraform.io/docs/providers/google/r/compute_instance_template.html#service_account."`
`136`	`137`	`}`
`137`		`-`