Skip to content
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

[receiver/awsecscontainermetrics] migrate to AWS v2 SDK #38479

Closed
petern48 opened this issue Mar 9, 2025 · 2 comments · Fixed by #38480
Closed

[receiver/awsecscontainermetrics] migrate to AWS v2 SDK #38479

petern48 opened this issue Mar 9, 2025 · 2 comments · Fixed by #38480
Labels

Comments

@petern48
Copy link
Contributor

petern48 commented Mar 9, 2025

Component(s)

receiver/awsecscontainermetrics

Describe the issue you're reporting

Component(s)

exporter/awsecscontainermetrics

Describe the issue you're reporting

AWS has annouced that it will be discontinuing support for go AWS SDK v1 and we should be updating the collector to migrate away from it to ensure any issues with the package are dealt with.

https://aws.amazon.com/blogs/developer/announcing-end-of-support-for-aws-sdk-for-go-v1-on-july-31-2025/

@petern48 petern48 added the needs triage New item requiring triage label Mar 9, 2025
Copy link
Contributor

github-actions bot commented Mar 9, 2025

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@petern48
Copy link
Contributor Author

petern48 commented Mar 9, 2025

This issue wasn't created along with the others in #36699, so I created one to link to.

andrzej-stencel pushed a commit that referenced this issue Mar 11, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Upgrade awsecscontainermetrics receiver to use aws sdk 2

<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes #38479

<!--Describe what testing was performed and which tests were added.-->
#### Testing
No changes needed.

<!--Describe the documentation added.-->
#### Documentation
Added a changelog

<!--Please delete paragraphs that you did not use before submitting.-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant