open-telemetry / opentelemetry-lambda

Create your own Lambda Layer in each OTel language using this starter code. Add the Lambda Layer to your Lamdba Function to get tracing with OpenTelemetry.
https://opentelemetry.io
Apache License 2.0
260 stars 156 forks source link

Improve nodejs layer by auto-instrumentations-node #448

Open wangzlei opened 1 year ago

wangzlei commented 1 year ago

Describe the bug onboard auto-instrumentations-node instead of managing a library instrumentation list in https://github.com/open-telemetry/opentelemetry-lambda/blob/main/nodejs/packages/layer/src/wrapper.ts

Refer to python wrapper improvement https://github.com/open-telemetry/opentelemetry-lambda/pull/164

Steps to reproduce If possible, provide a recipe for reproducing the error.

What did you expect to see? No need manage a lib instrumentation list in wrapper.ts Load lib instrumentation by auto-instrumentations-node

What did you see instead?

What version of collector/language SDK version did you use? Version: (e.g., v0.58.0, v1.11.0, etc)

What language layer did you use? Config: (e.g., Java, Python, etc)

Additional context Add any other context about the problem here.

mat-rumian commented 1 year ago

@wangzlei I could take care of it :)

github-actions[bot] commented 1 month ago

This issue was marked stale. It will be closed in 30 days without additional activity.

tylerbenson commented 1 month ago

@mat-rumian do you still want to take care of this? Feel free.