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

Importing @opentelemetry/core fails with no other dependencies #471

Closed
dyladan opened this issue Oct 31, 2019 · 5 comments
Closed

Importing @opentelemetry/core fails with no other dependencies #471

dyladan opened this issue Oct 31, 2019 · 5 comments
Labels
bug Something isn't working
Milestone

Comments

@dyladan
Copy link
Member

dyladan commented Oct 31, 2019

Please answer these questions before submitting a bug report.

What version of OpenTelemetry are you using?

0.1.1

What version of Node are you using?

10.15.0

What did you do?

If possible, provide a recipe for reproducing the error.

  1. create a new node.js project
  2. install @opentelemetry/core
  3. import @opentelemetry/core

What did you expect to see?

Useable OpenTelemetry core api

What did you see instead?

Module fails to import and crashes the program.

@dyladan dyladan added the bug Something isn't working label Oct 31, 2019
@mayurkale22 mayurkale22 added this to the Alpha v0.2 milestone Oct 31, 2019
@mayurkale22
Copy link
Member

@dyladan Can we close this one via #472?

@dyladan
Copy link
Member Author

dyladan commented Nov 1, 2019

yes

@dyladan dyladan closed this as completed Nov 1, 2019
@dyladan
Copy link
Member Author

dyladan commented Nov 1, 2019

@mayurkale22 I was just thinking about this. The program crashed. Failing to load a plugin should not cause a crash right? The actual crash was not during plugin load but was in the line that logs the traceId. Something along the lines of cannot access traceId of undefined. Should I open another issue for that?

@mayurkale22
Copy link
Member

Sure, please provide a recipe for reproducing the error.

@dyladan
Copy link
Member Author

dyladan commented Nov 1, 2019

I was creating the bug and realized I was mixing up 2 different bugs. Current state is fine :)

pichlermarc pushed a commit to dynatrace-oss-contrib/opentelemetry-js that referenced this issue Dec 15, 2023
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

2 participants