Merge pull request #3387 from terraform-providers/b-ecs-retry-read-on…

…-creation

resource/aws_ecs_service: Retry DescribeServices after creation

aws/resource_aws_ecs_service.go

@@ -356,14 +356,24 @@ func resourceAwsEcsServiceRead(d *schema.ResourceData, meta interface{}) error {
 			}
 			return resource.NonRetryableError(err)
 		}
+
+		if d.IsNewResource() && len(out.Services) < 1 {
+			return resource.RetryableError(fmt.Errorf("No ECS service found: %q", d.Id()))
+		}
+
+		service := out.Services[0]
+		if d.IsNewResource() && *service.Status == "INACTIVE" {
+			return resource.RetryableError(fmt.Errorf("ECS service currently INACTIVE: %q", d.Id()))
+		}
+
 		return nil
 	})
 	if err != nil {
 		return err
 	}
 
 	if len(out.Services) < 1 {
-		log.Printf("[DEBUG] Removing ECS service %s (%s) because it's gone", d.Get("name").(string), d.Id())
+		log.Printf("[WARN] Removing ECS service %s (%s) because it's gone", d.Get("name").(string), d.Id())
 		d.SetId("")
 		return nil
 	}
@@ -372,7 +382,7 @@ func resourceAwsEcsServiceRead(d *schema.ResourceData, meta interface{}) error {
 
 	// Status==INACTIVE means deleted service
 	if *service.Status == "INACTIVE" {
-		log.Printf("[DEBUG] Removing ECS service %q because it's INACTIVE", *service.ServiceArn)
+		log.Printf("[WARN] Removing ECS service %q because it's INACTIVE", *service.ServiceArn)
 		d.SetId("")
 		return nil
 	}