-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Change max query size for GetMetricData API to 500 and add RecentlyActive config #33105
Change max query size for GetMetricData API to 500 and add RecentlyActive config #33105
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
This pull request is now in conflicts. Could you fix it? 🙏
|
CI error:
This is caused by
|
…tive config (#33105) (#33301) (cherry picked from commit 35cab79) Co-authored-by: kaiyan-sheng <[email protected]>
… 500 and add RecentlyActive config (#33300) * Change max query size for GetMetricData API to 500 and add RecentlyActive config (#33105) (cherry picked from commit 35cab79) * Update CHANGELOG.next.asciidoc Co-authored-by: kaiyan-sheng <[email protected]>
|
What does this PR do?
This PR is to change max query size for GetMetricData API from 100 to 500.
500 is the maximum metrics for a single API request:
A GetMetricData operation that does not include a query can retrieve as many as 500 different metrics in a single request, with a total of as many as 100,800 data points.
AWS documentation link: https://docs.aws.amazon.com/AmazonCloudWatch/latest/APIReference/API_GetMetricData.html
This PR also adds
RecentlyActive
config parameter intoListMetrics
API call to get recent metrics instead of all.This is the first step to help improve performance for the CloudWatch metricset, please see #33195 for more details.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.