Skip to content

Correct Import for Yet Another Cloudwatch Exporter #3880

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

BoweFlex
Copy link

PR Description

Correcting the import for Yet Another Cloudwatch Exporter.

Which issue(s) this PR fixes

No Issues related AFAIK, attempting to resolve the rebase errors in #3602.

Notes to the Reviewer

PR Checklist

  • CHANGELOG.md updated

@BoweFlex BoweFlex requested a review from a team as a code owner June 21, 2025 02:03
@tristanburgess tristanburgess self-requested a review June 30, 2025 12:51
Copy link

@tristanburgess tristanburgess left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

needs go mod tidy + resolve merge conflicts

@dehaansa
Copy link
Contributor

It appears from the release notes that there are several new services that should be listed in the docs

Add ContainerInsights service by @JetSquirrel
Add AWS/Scheduler and AWS/ECR services by @andriikushch
Add AWS/VpcLattice service by @greymd
Add AWS/QuickSight service by @choppedpork
Add AWS/Timestream service by @andriikushch
Add Network Manager / Cloud WAN support by @kylehodgetts
RDS: include RDS Proxy metrics within the RDS namespace by @vitaliyf
Mediapackage: include mediapackagev2 namespace by @henrylaiBrightcove

@BoweFlex
Copy link
Author

It appears from the release notes that there are several new services that should be listed in the docs

Add ContainerInsights service by @JetSquirrel
Add AWS/Scheduler and AWS/ECR services by @andriikushch
Add AWS/VpcLattice service by @greymd
Add AWS/QuickSight service by @choppedpork
Add AWS/Timestream service by @andriikushch
Add Network Manager / Cloud WAN support by @kylehodgetts
RDS: include RDS Proxy metrics within the RDS namespace by @vitaliyf
Mediapackage: include mediapackagev2 namespace by @henrylaiBrightcove

Happy to add this, can you point me to where specifically this should be added?

@dehaansa
Copy link
Contributor

dehaansa commented Jul 3, 2025

It appears from the release notes that there are several new services that should be listed in the docs

Add ContainerInsights service by @JetSquirrel
Add AWS/Scheduler and AWS/ECR services by @andriikushch
Add AWS/VpcLattice service by @greymd
Add AWS/QuickSight service by @choppedpork
Add AWS/Timestream service by @andriikushch
Add Network Manager / Cloud WAN support by @kylehodgetts
RDS: include RDS Proxy metrics within the RDS namespace by @vitaliyf
Mediapackage: include mediapackagev2 namespace by @henrylaiBrightcove

Happy to add this, can you point me to where specifically this should be added?

They would be listed in docs/sources/reference/components/prometheus/prometheus.exporter.cloudwatch.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants