Skip to content

Correct Import for Yet Another Cloudwatch Exporter#3880

Open
BoweFlex wants to merge 4 commits intografana:mainfrom
BoweFlex:cloudwatch-exporter-update
Open

Correct Import for Yet Another Cloudwatch Exporter#3880
BoweFlex wants to merge 4 commits intografana:mainfrom
BoweFlex:cloudwatch-exporter-update

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

@github-actions
Copy link
Contributor

github-actions bot commented Oct 3, 2025

This PR has not had any activity in the past 90 days, so the needs-attention label has been added to it.
If you do not have enough time to follow up on this PR or you think it's no longer relevant, consider closing it.
The needs-attention label signals to maintainers that something has fallen through the cracks. No action is needed by you; your PR will be kept open and you do not have to respond to this comment. The label will be removed the next time this job runs if there is new activity.
Thank you for your contributions!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants