Add Tag Support to API Gateway REST API Resource

[DrFaust92]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" comments, they generate extra noise for pull request followers and do not help prioritize the request

Relates #8744

Release note for CHANGELOG:

- resource/resource_aws_api_gateway_rest_api: Add `tags` argument.

Output from acceptance testing:

$ make testacc TESTARGS='-run=TestAccAWSAPIGatewayRestApi_'

=== RUN   TestAccAWSAPIGatewayRestApi_basic
=== PAUSE TestAccAWSAPIGatewayRestApi_basic
=== CONT  TestAccAWSAPIGatewayRestApi_basic
--- PASS: TestAccAWSAPIGatewayRestApi_basic (254.58s)
=== RUN   TestAccAWSAPIGatewayRestApi_tags
=== PAUSE TestAccAWSAPIGatewayRestApi_tags
=== CONT  TestAccAWSAPIGatewayRestApi_tags
--- PASS: TestAccAWSAPIGatewayRestApi_tags (72.37s)
=== RUN   TestAccAWSAPIGatewayRestApi_EndpointConfiguration
=== PAUSE TestAccAWSAPIGatewayRestApi_EndpointConfiguration
=== CONT  TestAccAWSAPIGatewayRestApi_EndpointConfiguration
--- PASS: TestAccAWSAPIGatewayRestApi_EndpointConfiguration (71.19s)
=== RUN   TestAccAWSAPIGatewayRestApi_EndpointConfiguration_Private
=== PAUSE TestAccAWSAPIGatewayRestApi_EndpointConfiguration_Private
=== CONT  TestAccAWSAPIGatewayRestApi_EndpointConfiguration_Private
--- PASS: TestAccAWSAPIGatewayRestApi_EndpointConfiguration_Private (237.72s)
=== RUN   TestAccAWSAPIGatewayRestApi_api_key_source
=== PAUSE TestAccAWSAPIGatewayRestApi_api_key_source
=== CONT  TestAccAWSAPIGatewayRestApi_api_key_source
--- PASS: TestAccAWSAPIGatewayRestApi_api_key_source (171.49s)
=== RUN   TestAccAWSAPIGatewayRestApi_policy
=== PAUSE TestAccAWSAPIGatewayRestApi_policy
=== CONT  TestAccAWSAPIGatewayRestApi_policy
--- PASS: TestAccAWSAPIGatewayRestApi_policy (420.34s)
=== RUN   TestAccAWSAPIGatewayRestApi_openapi
=== PAUSE TestAccAWSAPIGatewayRestApi_openapi
=== CONT  TestAccAWSAPIGatewayRestApi_openapi
--- PASS: TestAccAWSAPIGatewayRestApi_openapi (69.78s)
PASS
ok  	github.com/terraform-providers/terraform-provider-aws/aws	451.528s


...

[bflad]

One non-blocking nit, otherwise looks good, @DrFaust92 🚀

Output from acceptance testing:

--- PASS: TestAccAWSAPIGatewayRestApi_openapi (35.26s)
--- PASS: TestAccAWSAPIGatewayRestApi_tags (66.82s)
--- PASS: TestAccAWSAPIGatewayRestApi_api_key_source (67.18s)
--- PASS: TestAccAWSAPIGatewayRestApi_policy (124.97s)
--- PASS: TestAccAWSAPIGatewayRestApi_EndpointConfiguration_Private (171.00s)
--- PASS: TestAccAWSAPIGatewayRestApi_EndpointConfiguration (270.16s)
--- PASS: TestAccAWSAPIGatewayRestApi_disappears (389.77s)
--- PASS: TestAccAWSAPIGatewayRestApi_basic (438.28s)

[bflad]

Nit: Standard Go language conventions are to use lowercase mixed caps for variable naming.

[ghost]

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

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks!

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