-
Notifications
You must be signed in to change notification settings - Fork 26k
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
Update 10-open-telemetry.mdx, missing packages. #65885
base: canary
Are you sure you want to change the base?
Conversation
It seems that some package installs are missing from the documentation, which leads to the following error: Import trace for requested module: ./src/instrumentation.ts Error [ERR_MODULE_NOT_FOUND]: An error occurred while loading instrumentation hook: Cannot find package <PACKAGE NAME> imported from <ROOT>/node_modules/@vercel/otel/dist/node/index.js
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
Hi all! I noticed that following the docs didn't lead to a working version of the If this is not a documentation issue, but an issue with the package or setup, don't hesitate to close! |
@yurivangeffen Can you clarify what you had in your |
Added it to the original comment. |
@@ -27,10 +27,10 @@ That's why we prepared a package `@vercel/otel` that helps you get started quick | |||
|
|||
### Using `@vercel/otel` | |||
|
|||
To get started, you must install `@vercel/otel`: | |||
To get started, you must install `@vercel/otel`, `@opentelemetry/api`, and `@opentelemetry/api-logs`: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really need @opentelemetry/api-logs
?
It seems that some package installs are missing from the documentation, which leads to the following error:
My instrumentation file looks like this: