aws_placement_group enforces an incorrect partition_count limit

[gmarkey]

Terraform and AWS Provider Version

Terraform v1.13.4
on darwin_arm64
+ provider registry.terraform.io/hashicorp/aws v6.20.0
+ provider registry.terraform.io/hashicorp/null v3.2.4
+ provider registry.terraform.io/hashicorp/vault v5.4.0
+ provider registry.terraform.io/mongey/kafka v0.13.1

Affected Resource(s) or Data Source(s)

• aws_placement_group

Expected Behavior

Placement groups using partition strategy should not enforce an upper limit of 7 and instead allow the AWS API to enforce limits, as this limit is mutable.
https://github.com/hashicorp/terraform-provider-aws/blob/main/internal/service/ec2/ec2_placement_group.go#L56

Actual Behavior

Provider errors out when the partition count is greater than hard coded limit of 7.

Relevant Error/Panic Output

Error: expected partition_count to be in the range (0 - 7), got 15

Sample Terraform Configuration

Click to expand configuration

resource "aws_placement_group" "partitioned" {
  name            = "partitioned-placement-group"
  strategy        = "partition"
  partition_count = 8
}

Steps to Reproduce

1. Attempt to create an aws_placement_group with strategy = "partition" and partition_count greater than 7.

Debug Logging

n/a

GenAI / LLM Assisted Development

n/a

Important Facts and References

https://github.com/hashicorp/terraform-provider-aws/blob/main/internal/service/ec2/ec2_placement_group.go#L56

AWS allow modifying this limit, although it does not seem to be very publicly advertised, nor is it actually listed under service quotas - modification requires opening a support case (this functionality may be limited to enterprise support customers).

Would you like to implement a fix?

No

[github-actions]

Community Guidelines

This comment is added to every new Issue to provide quick reference to how the Terraform AWS Provider is maintained. Please review the information below, and thank you for contributing to the community that keeps the provider thriving! 🚀

Voting for Prioritization

• Please vote on this Issue by adding a 👍 reaction to the original post to help the community and maintainers prioritize it.
• Please see our prioritization guide for additional information on how the maintainers handle prioritization.
• Please do not leave +1 or other comments that do not add relevant new information or questions; they generate extra noise for others following the Issue and do not help prioritize the request.

Volunteering to Work on This Issue

• If you are interested in working on this issue, please leave a comment.
• If this would be your first contribution, please review the contribution guide.
• For new resources and data sources, use skaff to generate scaffolding with comments detailing common expectations.

[stefanfreitag]

Hi @gmarkey 👋 ,

thanks for raising the issue. I can have a look into this one.

[ChaosCypher]

Opened a PR to address this here - #45042

[stefanfreitag]

Hi @ChaosCypher !
Thanks for picking it up :)

[revtron]

hi @stefanfreitag
👋 I noticed this issue about the hardcoded partition limit.
I’d like to work on this and submit a PR removing the static limit
in favor of API-based validation. Is anyone already working on this?

[ChaosCypher]

hi @stefanfreitag 👋 I noticed this issue about the hardcoded partition limit. I’d like to work on this and submit a PR removing the static limit in favor of API-based validation. Is anyone already working on this?

#45042