7605 cloudwatch metric filters need to be serial

[lxop]

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

Fixes #7605

Changes proposed in this pull request:

• Creation/deletion of multiple Cloudwatch Log Metric Filters on any individual Cloudwatch Log Group is performed serially.

Output from acceptance testing:

$ make testacc TEST=./aws TESTARGS='-run=TestAccAWSCloudWatchLogMetricFilter_basic'
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./aws -v -parallel 20 -run=TestAccAWSCloudWatchLogMetricFilter_basic -timeout 120m
=== RUN   TestAccAWSCloudWatchLogMetricFilter_basic
=== PAUSE TestAccAWSCloudWatchLogMetricFilter_basic
=== CONT  TestAccAWSCloudWatchLogMetricFilter_basic
--- PASS: TestAccAWSCloudWatchLogMetricFilter_basic (50.17s)
PASS
ok      github.com/terraform-providers/terraform-provider-aws/aws       50.269s

[bflad]

LGTM, thanks for this fix, @lxop 🚀 I was able to coerce the OperationAbortedException with the new testing and without the code update. With the code it successfully runs as expected.

Output from acceptance testing:

--- PASS: TestAccAWSCloudWatchLogMetricFilter_basic (58.11s)

[bflad]

This has been released in version 2.2.0 of the Terraform 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!