Skip to content

[processor/resourcedetection] update to aws-sdk-go-v2 #39368

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

Merged
merged 1 commit into from
Apr 14, 2025

Conversation

mjlshen
Copy link
Contributor

@mjlshen mjlshen commented Apr 13, 2025

Description

aws-sdk-go is being deprecated on July 31, 2025. This PR updates processor/resourcedetection to aws-sdk-go-v2.

Link to tracking issue

Fixes #37730

Testing

I converted the existing unit tests to aws-sdk-go-v2 as well, keeping the original intent of the tests.

@mjlshen mjlshen requested review from dashpole and a team as code owners April 13, 2025 05:45
@github-actions github-actions bot added the processor/resourcedetection Resource detection processor label Apr 13, 2025
@github-actions github-actions bot requested a review from Aneurysm9 April 13, 2025 05:45
Copy link
Member

@bogdandrutu bogdandrutu left a comment

Choose a reason for hiding this comment

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

Please add a changelog

@mjlshen
Copy link
Contributor Author

mjlshen commented Apr 13, 2025

Please add a changelog

@bogdandrutu Thanks for the review! I'm a relatively new contributor, so I'm just making sure here. I've made a couple aws-sdk-go-v2 migration PRs recently and was previously told that there's no need for a changelog since there isn't a user/API change (#39097 (comment)). Would you like me to write a changelog indicating a user-facing change?

@atoulme atoulme added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Apr 14, 2025
Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

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

LGTM - since no change to user experience or API, we typically skip changelog for those.

@bogdandrutu bogdandrutu merged commit ddddb51 into open-telemetry:main Apr 14, 2025
180 of 181 checks passed
@github-actions github-actions bot added this to the next release milestone Apr 14, 2025
@mjlshen mjlshen deleted the 37730 branch April 14, 2025 03:37
akshays-19 pushed a commit to akshays-19/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2025
…#39368)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
aws-sdk-go is being [deprecated on July 31,
2025](https://aws.amazon.com/blogs/developer/announcing-end-of-support-for-aws-sdk-for-go-v1-on-july-31-2025/).
This PR updates `processor/resourcedetection` to aws-sdk-go-v2.

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

<!--Describe what testing was performed and which tests were added.-->
#### Testing
I converted the existing unit tests to aws-sdk-go-v2 as well, keeping
the original intent of the tests.

Signed-off-by: Michael Shen <[email protected]>
Fiery-Fenix pushed a commit to Fiery-Fenix/opentelemetry-collector-contrib that referenced this pull request Apr 24, 2025
…#39368)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
aws-sdk-go is being [deprecated on July 31,
2025](https://aws.amazon.com/blogs/developer/announcing-end-of-support-for-aws-sdk-for-go-v1-on-july-31-2025/).
This PR updates `processor/resourcedetection` to aws-sdk-go-v2.

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

<!--Describe what testing was performed and which tests were added.-->
#### Testing
I converted the existing unit tests to aws-sdk-go-v2 as well, keeping
the original intent of the tests.

Signed-off-by: Michael Shen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
processor/resourcedetection Resource detection processor Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[processor/resourcedetection] migrate to AWS v2 SDK
3 participants