aws_service_discovery_public_dns_namespace name length limit

[Frogvall]

Fixes #5339

Changes proposed in this pull request:

• Trim name if needed to comply to the 64 character limit to RequestId

Output from acceptance testing:

• Did make acceptance test. Spent a few hours trying to make it run in our company environment, but just wouldn't work.

Basically just does what was done in #4702, and as was suggested by @bflad

[bflad]

Thanks @Frogvall! I fixed up the acceptance testing so we can get this merged and released this week. 🚀

make testacc TEST=./aws TESTARGS='-run=TestAccAWSServiceDiscoveryPublicDnsNamespace_'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -run=TestAccAWSServiceDiscoveryPublicDnsNamespace_ -timeout 120m
=== RUN   TestAccAWSServiceDiscoveryPublicDnsNamespace_basic
--- PASS: TestAccAWSServiceDiscoveryPublicDnsNamespace_basic (13.60s)
=== RUN   TestAccAWSServiceDiscoveryPublicDnsNamespace_longname
--- PASS: TestAccAWSServiceDiscoveryPublicDnsNamespace_longname (12.56s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	26.782s

[Frogvall]

Great, thank you!

[bflad]

This has been released in version 1.33.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

[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 feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!