Make it clear that we don't need to ForceSend nested fields in Comput…

…eInstanceFromTemplate (#2543)

Merged PR #2543.

third_party/terraform/resources/resource_compute_instance_from_template.go

@@ -127,7 +127,6 @@ func resourceComputeInstanceFromTemplateCreate(d *schema.ResourceData, meta inte
 	}
 
 	// Force send all top-level fields that have been set in case they're overridden to zero values.
-	// TODO: consider doing so for nested fields as well.
 	// Initialize ForceSendFields to empty so we don't get things that the instance resource
 	// always force-sends.
 	instance.ForceSendFields = []string{}

third_party/terraform/tests/resource_compute_instance_from_template_test.go

@@ -32,6 +32,7 @@ func TestAccComputeInstanceFromTemplate_basic(t *testing.T) {
 					// Check that fields were set based on the template
 					resource.TestCheckResourceAttr(resourceName, "machine_type", "n1-standard-1"),
 					resource.TestCheckResourceAttr(resourceName, "attached_disk.#", "1"),
+					resource.TestCheckResourceAttr(resourceName, "scheduling.0.automatic_restart", "false"),
 				),
 			},
 		},
@@ -255,6 +256,10 @@ resource "google_compute_instance_template" "foobar" {
 		foo = "bar"
 	}
 
+	scheduling {
+		automatic_restart = true
+	}
+
 	can_ip_forward = true
 }
 
@@ -269,6 +274,9 @@ resource "google_compute_instance_from_template" "foobar" {
 	labels = {
 		my_key       = "my_value"
 	}
+	scheduling {
+		automatic_restart = false
+	}
 }
 `, template, template, instance)
 }