-
Notifications
You must be signed in to change notification settings - Fork 9.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support CW Metric Streams include_linked_accounts_argument #29281
Support CW Metric Streams include_linked_accounts_argument #29281
Conversation
Community NoteVoting for Prioritization
For Submitters
|
# Conflicts: # internal/service/cloudwatch/metric_stream.go
…rom acceptance test cofiguration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀.
% make testacc TESTARGS='-run=TestAccCloudWatchMetricStream_' PKG=cloudwatch ACCTEST_PARALLELISM=3
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/cloudwatch/... -v -count 1 -parallel 3 -run=TestAccCloudWatchMetricStream_ -timeout 180m
=== RUN TestAccCloudWatchMetricStream_basic
=== PAUSE TestAccCloudWatchMetricStream_basic
=== RUN TestAccCloudWatchMetricStream_disappears
=== PAUSE TestAccCloudWatchMetricStream_disappears
=== RUN TestAccCloudWatchMetricStream_nameGenerated
=== PAUSE TestAccCloudWatchMetricStream_nameGenerated
=== RUN TestAccCloudWatchMetricStream_namePrefix
=== PAUSE TestAccCloudWatchMetricStream_namePrefix
=== RUN TestAccCloudWatchMetricStream_includeFilters
=== PAUSE TestAccCloudWatchMetricStream_includeFilters
=== RUN TestAccCloudWatchMetricStream_excludeFilters
=== PAUSE TestAccCloudWatchMetricStream_excludeFilters
=== RUN TestAccCloudWatchMetricStream_update
=== PAUSE TestAccCloudWatchMetricStream_update
=== RUN TestAccCloudWatchMetricStream_tags
=== PAUSE TestAccCloudWatchMetricStream_tags
=== RUN TestAccCloudWatchMetricStream_additional_statistics
=== PAUSE TestAccCloudWatchMetricStream_additional_statistics
=== RUN TestAccCloudWatchMetricStream_includeLinkedAccountsMetrics
=== PAUSE TestAccCloudWatchMetricStream_includeLinkedAccountsMetrics
=== CONT TestAccCloudWatchMetricStream_basic
=== CONT TestAccCloudWatchMetricStream_excludeFilters
=== CONT TestAccCloudWatchMetricStream_additional_statistics
--- PASS: TestAccCloudWatchMetricStream_excludeFilters (18.84s)
=== CONT TestAccCloudWatchMetricStream_includeLinkedAccountsMetrics
--- PASS: TestAccCloudWatchMetricStream_additional_statistics (60.76s)
=== CONT TestAccCloudWatchMetricStream_tags
--- PASS: TestAccCloudWatchMetricStream_basic (376.44s)
=== CONT TestAccCloudWatchMetricStream_update
--- PASS: TestAccCloudWatchMetricStream_includeLinkedAccountsMetrics (217.60s)
=== CONT TestAccCloudWatchMetricStream_namePrefix
--- PASS: TestAccCloudWatchMetricStream_update (61.27s)
=== CONT TestAccCloudWatchMetricStream_includeFilters
=== CONT TestAccCloudWatchMetricStream_nameGenerated
--- PASS: TestAccCloudWatchMetricStream_tags (365.74s)
--- PASS: TestAccCloudWatchMetricStream_includeFilters (16.78s)
=== CONT TestAccCloudWatchMetricStream_disappears
--- PASS: TestAccCloudWatchMetricStream_namePrefix (1940.37s)
--- PASS: TestAccCloudWatchMetricStream_nameGenerated (1931.48s)
--- PASS: TestAccCloudWatchMetricStream_disappears (1967.63s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/cloudwatch 3867.035s
@mcgealy Thanks for the contribution 🎉 👏. |
This functionality has been released in v4.64.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. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
AWS CloudWatch recently launched support for cross account streams. This change adds support for the IncludLinkedAccountsMetrics which enables this feature.
Closes #30345.
References
Output from Acceptance Testing