-
Notifications
You must be signed in to change notification settings - Fork 486
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
Add Cloudwatch metrics integration #2740
Conversation
14ac66f
to
f5acc7d
Compare
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
Co-authored-by: MattNolf <86960799+matthewnolf@users.noreply.github.com>
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
@marctc all comments have been resolved. Can you take another look? |
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, will merge after @karengermond reviews the docs
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 -- just a few comments.
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
docs/sources/configuration/integrations/cloudwatch-exporter-config.md
Outdated
Show resolved
Hide resolved
…nfig.md Co-authored-by: Karen Germond <110922559+karengermond@users.noreply.github.com>
Co-authored-by: Karen Germond <110922559+karengermond@users.noreply.github.com>
@marctc @karengermond all docs comments fixed 😃 |
@@ -25,6 +25,8 @@ Main (unreleased) | |||
|
|||
- Integrations: Introduce the `mssql` integration. (@binaryfissiongames) | |||
|
|||
- Integrations: Introduce `cloudwatch metrics` integration. (@thepalbi) |
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.
@marctc just fixed it. Terrible mistake trying to merge conflicts with the gh ui 🤦
PR Description
Add support for scraping AWS CloudWatch metrics, by embedding YACE as an integration.
Which issue(s) this PR fixes
Resolves https://github.com/grafana/cloud-onboarding/issues/2976
Notes to the Reviewer
PR Checklist