provider/aws: Add support for EC2 Placement Group

[radeksimko]

Test plan

$ make testacc TEST=./builtin/providers/aws TESTARGS='-run=PlacementGroup' 2>~/tf.log

go generate ./...
TF_ACC=1 go test ./builtin/providers/aws -v -run=PlacementGroup -timeout 90m
=== RUN   TestAccAWSPlacementGroup_basic
--- PASS: TestAccAWSPlacementGroup_basic (6.99s)
PASS
ok      github.com/hashicorp/terraform/builtin/providers/aws    7.012s

[stack72]

I think we should set a default value of Cluster here as it is the only supported value atm

[stack72]

Ah but we need to have ForceNew enabled if there is no update :(

[radeksimko]

I'm not quite convinced it's a good idea to make it default, but I don't know Amazon's plans for the future.

[stack72]

This looks good. The only issue I would have with it is that we allow any value as the strategy and then wait for AWS to validate the input but that's just $0.02 :)

[stack72]

I've tested this and it looks good - i guess this can be merged pretty easily :)

[stack72]

@phinze FYI, I checked this branch out and was able to manage placement groups with it. So I can very this is good

[phinze]

Looking good! Nice work as usual @radeksimko 😀

[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.