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

Elasticsearch exporter lack status description in traces. #27645

Closed
pengweiqhca opened this issue Oct 12, 2023 · 2 comments · Fixed by #27800
Closed

Elasticsearch exporter lack status description in traces. #27645

pengweiqhca opened this issue Oct 12, 2023 · 2 comments · Fixed by #27800
Assignees
@pengweiqhca
Labels
enhancement New feature or request exporter/elasticsearch

Comments

@pengweiqhca
Copy link
Contributor

Component(s)

exporter/elasticsearch

Is your feature request related to a problem? Please describe.

Trace status has description but Elasticsearch exporter not export.

trace api
trace.proto

Describe the solution you'd like

Add TraceStatusDescription.

Describe alternatives you've considered

No response

Additional context

No response
@pengweiqhca pengweiqhca added enhancement New feature or request needs triage New item requiring triage labels Oct 12, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@crobert-1
Copy link
Member

This makes sense to me. Also, the status code is already being encoded here, so the message would fit right next to it.

@crobert-1 crobert-1 removed the needs triage New item requiring triage label Oct 13, 2023
@github-actions github-actions bot mentioned this issue Oct 17, 2023
Closed
@pengweiqhca pengweiqhca mentioned this issue Oct 17, 2023
Merged
pengweiqhca added a commit to pengweiqhca/opentelemetry-collector-contrib that referenced this issue Oct 18, 2023
@pengweiqhca
Fix open-telemetry#27645
d4a16c7
codeboten pushed a commit that referenced this issue Dec 15, 2023
@pengweiqhca
[exporter/elasticsearch] Add TraceStatusDescription in span (#27800)
d24f4ea 
Fix
#27645
cparkins pushed a commit to AmadeusITGroup/opentelemetry-collector-contrib that referenced this issue Jan 10, 2024
@pengweiqhca @cparkins
[exporter/elasticsearch] Add TraceStatusDescription in span (open-tel…
88684c3 
…emetry#27800)

Fix
open-telemetry#27645
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pengweiqhca pengweiqhca

Labels
enhancement New feature or request exporter/elasticsearch
Projects
None yet
Milestone
No milestone
2 participants
@pengweiqhca @crobert-1