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.

Sign up for GitHub

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

Jump to bottom

Jaeger: Handle InstrumentationLibrary #768

Closed
lalitb opened this issue May 19, 2021 · 2 comments · Fixed by #864
Closed

Jaeger: Handle InstrumentationLibrary #768

lalitb opened this issue May 19, 2021 · 2 comments · Fixed by #864
Assignees
@anshulkumar19
Labels
area:exporter bug Something isn't working release:required-for-ga To be resolved before GA release
Milestone
1.0.0

Comments

@lalitb
Copy link
Member

lalitb commented May 19, 2021

As per the specs, InstrumentationLibrary should be reported as Jaeger span tags:
https://github.com/open-telemetry/opentelemetry-specification/blob/v1.0.0/specification/trace/sdk_exporters/jaeger.md#instrumentationlibrary
@lalitb lalitb added the bug Something isn't working label May 19, 2021
@lalitb lalitb mentioned this issue May 19, 2021
Closed
22 tasks
@lalitb lalitb added area:exporter release:required-for-ga To be resolved before GA release labels May 19, 2021
@lalitb lalitb added this to the 1.0.0 GA milestone May 19, 2021
@anshulkumar19
Copy link
Contributor

I would like to pick this up, can I get it assigned?

@lalitb
Copy link
Member Author

lalitb commented Jun 7, 2021

Sure. @ThomsonTan can help you if needed as he knows this component better.

This was referenced Jun 16, 2021
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anshulkumar19 anshulkumar19

Labels
area:exporter bug Something isn't working release:required-for-ga To be resolved before GA release
Projects
None yet
Milestone
1.0.0
2 participants
@lalitb @anshulkumar19