provider/aws: Fix ECS cluster+service bugs

[radeksimko]

Since these bugs were more or less related to each other and the SDK has changed a little bit too, it would be more difficult to send these as separate PRs.
At least I tried to separate modifications per commits, so it should be easy to review.

Closes:

• ecs_cluster destroy failed - "ClusterContainsContainerInstancesException" #2691
• aws_ecs_cluster rename causes crash #2427

Test plan

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run=Ecs'

go generate ./...
TF_ACC=1 go test ./builtin/providers/aws -v -run=Ecs -timeout 90m
=== RUN TestAccAWSEcsCluster_basic
--- PASS: TestAccAWSEcsCluster_basic (2.40s)
=== RUN TestAccAWSEcsServiceWithARN
--- PASS: TestAccAWSEcsServiceWithARN (14.66s)
=== RUN TestAccAWSEcsServiceWithFamilyAndRevision
--- PASS: TestAccAWSEcsServiceWithFamilyAndRevision (20.99s)
=== RUN TestAccAWSEcsServiceWithRenamedCluster
--- PASS: TestAccAWSEcsServiceWithRenamedCluster (38.28s)
=== RUN TestAccAWSEcsTaskDefinition_basic
--- PASS: TestAccAWSEcsTaskDefinition_basic (3.08s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    79.429s

[phinze]

LGTM

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.