Revert "Allow project to be specified when importing instance groups (h…

…ashicorp#689)" (hashicorp#692)

This reverts commit e46f436.

google-beta/resource_compute_instance_group.go

@@ -379,17 +379,12 @@ func resourceComputeInstanceGroupDelete(d *schema.ResourceData, meta interface{}
 
 func resourceComputeInstanceGroupImportState(d *schema.ResourceData, meta interface{}) ([]*schema.ResourceData, error) {
 	parts := strings.Split(d.Id(), "/")
-	if len(parts) == 2 {
-		d.Set("zone", parts[0])
-		d.Set("name", parts[1])
-	} else if len(parts) == 3 {
-		d.Set("project", parts[0])
-		d.Set("zone", parts[1])
-		d.Set("name", parts[2])
-		d.SetId(parts[1] + "/" + parts[2])
-	} else {
-		return nil, fmt.Errorf("Invalid compute instance group specifier. Expecting {zone}/{name} or {project}/{zone}/{name}")
+	if len(parts) != 2 {
+		return nil, fmt.Errorf("Invalid compute instance group specifier. Expecting {zone}/{name}")
 	}
 
+	d.Set("zone", parts[0])
+	d.Set("name", parts[1])
+
 	return []*schema.ResourceData{d}, nil
 }

google-beta/resource_compute_instance_group_test.go

@@ -15,17 +15,15 @@ func TestAccComputeInstanceGroup_basic(t *testing.T) {
 	t.Parallel()
 
 	var instanceGroup compute.InstanceGroup
-	var resourceName = "google_compute_instance_group.basic"
 	var instanceName = fmt.Sprintf("instancegroup-test-%s", acctest.RandString(10))
-	var zone = "us-central1-c"
 
 	resource.Test(t, resource.TestCase{
 		PreCheck:     func() { testAccPreCheck(t) },
 		Providers:    testAccProviders,
 		CheckDestroy: testAccComputeInstanceGroup_destroy,
 		Steps: []resource.TestStep{
 			{
-				Config: testAccComputeInstanceGroup_basic(zone, instanceName),
+				Config: testAccComputeInstanceGroup_basic(instanceName),
 				Check: resource.ComposeTestCheckFunc(
 					testAccComputeInstanceGroup_exists(
 						"google_compute_instance_group.basic", &instanceGroup),
@@ -34,15 +32,9 @@ func TestAccComputeInstanceGroup_basic(t *testing.T) {
 				),
 			},
 			{
-				ResourceName:      resourceName,
-				ImportState:       true,
-				ImportStateVerify: true,
-			},
-			{
-				ResourceName:      resourceName,
+				ResourceName:      "google_compute_instance_group.basic",
 				ImportState:       true,
 				ImportStateVerify: true,
-				ImportStateId:     fmt.Sprintf("%s/%s/%s", getTestProjectFromEnv(), zone, instanceName),
 			},
 		},
 	})
@@ -315,7 +307,7 @@ func testAccComputeInstanceGroup_hasCorrectNetwork(nInstanceGroup string, nNetwo
 	}
 }
 
-func testAccComputeInstanceGroup_basic(zone, instance string) string {
+func testAccComputeInstanceGroup_basic(instance string) string {
 	return fmt.Sprintf(`
 	data "google_compute_image" "my_image" {
 		family  = "debian-9"
@@ -342,7 +334,7 @@ func testAccComputeInstanceGroup_basic(zone, instance string) string {
 	resource "google_compute_instance_group" "basic" {
 		description = "Terraform test instance group"
 		name = "%s"
-		zone = "%s"
+		zone = "us-central1-c"
 		instances = [ "${google_compute_instance.ig_instance.self_link}" ]
 		named_port {
 			name = "http"
@@ -357,7 +349,7 @@ func testAccComputeInstanceGroup_basic(zone, instance string) string {
 	resource "google_compute_instance_group" "empty" {
 		description = "Terraform test instance group empty"
 		name = "%s-empty"
-		zone = "%s"
+		zone = "us-central1-c"
 		named_port {
 			name = "http"
 			port = "8080"
@@ -366,7 +358,7 @@ func testAccComputeInstanceGroup_basic(zone, instance string) string {
 			name = "https"
 			port = "8443"
 		}
-	}`, instance, instance, zone, instance, zone)
+	}`, instance, instance, instance)
 }
 
 func testAccComputeInstanceGroup_rename(instance, instanceGroup, backend, health string) string {

website/docs/r/compute_instance_group.html.markdown

@@ -175,9 +175,8 @@ This resource provides the following
 
 ## Import
 
-Instance group can be imported using the `zone` and `name` with an optional `project`, e.g.
+Instance group can be imported using the `zone` and `name`, e.g.
 
 ```
 $ terraform import google_compute_instance_group.webservers us-central1-a/terraform-webservers
-$ terraform import google_compute_instance_group.webservers big-project/us-central1-a/terraform-webservers
 ```