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

Migrate to AWS v2 SDK #36699

Open
2 of 19 tasks
MovieStoreGuy opened this issue Dec 6, 2024 · 6 comments
Open
2 of 19 tasks

Migrate to AWS v2 SDK #36699

MovieStoreGuy opened this issue Dec 6, 2024 · 6 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@MovieStoreGuy
Copy link
Contributor

MovieStoreGuy commented Dec 6, 2024

Component(s)

No response

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/

Components using github.com/aws/aws-sdk-go

  • confmap/provider
  • exporter/awscloudwatchlogsexporter
  • exporter/awsemfexporter
  • exporter/awskinesisexporter
  • exporter/awss3exporter
  • exporter/awsxrayexporter
  • exporter/datadogexporter
  • exporter/loadbalancingexporter
  • extension/observer
  • extension/sigv4authextension
  • internal/aws
  • internal/kafka
  • internal/metadataproviders
  • processor/resourcedetectionprocessor
  • receiver/awscloudwatchreceiver
  • receiver/awscontainerinsightreceiver
  • receiver/awsecscontainermetricsreceiver
  • receiver/awss3receiver
  • receiver/awsxrayreceiver

(generated with rg 'github.com/aws/aws-sdk-go' -l -tgo | cut -d'/' -f1-2 | sort | uniq | sed 's/^/- [ ] /')

@MovieStoreGuy MovieStoreGuy added the needs triage New item requiring triage label Dec 6, 2024
@MovieStoreGuy MovieStoreGuy added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers and removed needs triage New item requiring triage labels Dec 6, 2024
@MovieStoreGuy
Copy link
Contributor Author

Some of these may already be migrated off, just confirming as we go.

@MovieStoreGuy
Copy link
Contributor Author

For those wanting to pick up this issue,

The migration guides https://aws.github.io/aws-sdk-go-v2/docs/migrating/ is a good starting place to understand what needs to be done.

@LZiHaN
Copy link
Contributor

LZiHaN commented Dec 10, 2024

hi @MovieStoreGuy , is this issue still available? i'd like to working on it, assign please

@mx-psi
Copy link
Member

mx-psi commented Dec 10, 2024

Duplicate of #33208?

@david-sevcik
Copy link

david-sevcik commented Dec 22, 2024

Hello @MovieStoreGuy and @mx-psi, is it still available? I would like to take it as my first issue

@MovieStoreGuy
Copy link
Contributor Author

Duplicate of #33208?

Yes, sorry

mx-psi pushed a commit that referenced this issue Jan 10, 2025
…v2 (#36797)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Migrating datadog exporter to use aws sdk v2

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

<!--Describe what testing was performed and which tests were added.-->
#### Testing
passes tests.
updated mock client implementations to support the new SDK interfaces.

<!--Describe the documentation added.-->
#### Documentation
no need to update.

<!--Please delete paragraphs that you did not use before submitting.-->
AkhigbeEromo pushed a commit to sematext/opentelemetry-collector-contrib that referenced this issue Jan 13, 2025
…v2 (open-telemetry#36797)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Migrating datadog exporter to use aws sdk v2

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

<!--Describe what testing was performed and which tests were added.-->
#### Testing
passes tests.
updated mock client implementations to support the new SDK interfaces.

<!--Describe the documentation added.-->
#### Documentation
no need to update.

<!--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
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants