-
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
aws_cloudwatch_metric
alarm validate return_data
flag on metric_query
blocks
#29925
base: main
Are you sure you want to change the base?
aws_cloudwatch_metric
alarm validate return_data
flag on metric_query
blocks
#29925
Conversation
Community NoteVoting for Prioritization
For Submitters
|
aws_cloudwatch_metric
alarm validate return_data
flag on metric_query
blocksaws_cloudwatch_metric
alarm validate return_data
flag on metric_query
blocks
@gdavison Updated to resolve merge conflict. Looks like all checks are passing except one that I believe is failing on master. If there is anything else I should change here please let me know or feel free to update PR. Thanks for review in advance! |
@gdavison Updated PR again with main branch it it looks like all CI checks pass now. Please let me know if there's anything else I should, or feel free to update branch. |
Hi @teddylear, I'm finally getting back to this. I noticed that
However, when I removed the second
which is returned by the AWS API. It looks like the logic for how many |
Description
aws_cloudwatch_metric
alarm validatereturn_data
flag onmetric_query
blocks.Please let me know if you want me to squash commits or change anything here.
Closes #29894
Output from Acceptance Testing
Please ignore shell line numbers