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

aws-sdk 3.363.0 breaks v3 instrumentation #1596

Closed
Fortiz2305 opened this issue Jul 17, 2023 · 2 comments
Closed

aws-sdk 3.363.0 breaks v3 instrumentation #1596

Fortiz2305 opened this issue Jul 17, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@Fortiz2305
Copy link

What version of OpenTelemetry are you using?

@opentelemetry/instrumentation-aws-sdk 0.35.0

What version of Node are you using?

v16.16.0

What did you do?

I upgraded some aws-sdk packages to the 3.363.0 version.

What did you expect to see?

I expected the instrumentation to work as usual, creating spans from the internal HTTP requests. For example, when making a request to S3, I used to see an S3.GetObject span, created by the aws-sdk instrumentation package.

What did you see instead?

I only see spans for the original HTTP requests. Now, when doing a request to S3, I only see an HTTP GET span from the http-instrumentation package, but the aws-sdk instrumentation package is not catching anything.

Additional context

It looks like AWS has renamed the smithy-client packages from @aws-sdk/smithy-client to @smithy/smithy-client in aws/aws-sdk-js-v3#4873. Checking the current code, it seems the smithy/smithy-client is not taken into account.

I already saw the same issue in some instrumentation providers like NewRelic or Elastic

@Fortiz2305 Fortiz2305 added the bug Something isn't working label Jul 17, 2023
@dyladan
Copy link
Member

dyladan commented Jul 19, 2023

@carolabadeer @blumamir can you look into this? Might be a duplicate of #1509. This can be closed if that is the case

@carolabadeer
Copy link
Contributor

Looks like both issues are reporting the same problem, thanks @Fortiz2305 for the additional context on the smithy-client packages. We can keep just one of those issues open for tracking a fix

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants