-
Notifications
You must be signed in to change notification settings - Fork 12k
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
CloudWatch: Make it possible to specify custom api endpoint #31402
Conversation
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.
Added minor copy-edit suggestions.
public/app/plugins/datasource/cloudwatch/components/ConfigEditor.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: achatterjee-grafana <70489351+achatterjee-grafana@users.noreply.github.com>
…or.tsx Co-authored-by: achatterjee-grafana <70489351+achatterjee-grafana@users.noreply.github.com>
Co-authored-by: Arve Knudsen <arve.knudsen@gmail.com>
Co-authored-by: Ursula Kallio <73951760+osg-grafana@users.noreply.github.com>
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.
Unblocking (if I was blocking this PR) with an Approve
Good catch, "documentation" word was missing and it is needed here. |
I actually commented on the wrong PR, but I don't think this will work for both metrics and logs or there should be a documented caveat on that. Cloudwatch Metrics and Cloudwatch Logs both use different endpoints |
We ran into this problem as well but adding to different data sources still leads to an error and the data is not fetched. |
Tested using
monitoring.us-east-2.amazonaws.com
and it works fine. Info about api endpoints and why to use them can be found hereAll the other AWS data source plugins that we have, i.e IOT Sitewise, Timestream and X-Ray, already have support for specifying a custom endpoint.
Fixes #26041