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

Elasticsearch exporter: "otel.library.version" and "otel.library.name" attributes missing #27282

Closed
minhntt1 opened this issue Sep 30, 2023 · 2 comments
Assignees

Comments

@minhntt1
Copy link

Component(s)

exporter/elasticsearch

Describe the issue you're reporting

Trying to export traces to both Jaeger and Elasticsearch but there are no otel related attributes in Elasticsearch exported data. I'm using an Opentelemetry Java agent.
Exported trace in Jaeger:
... "tags": [ { "key": "otel.library.name", "type": "string", "value": "io.opentelemetry.rabbitmq-2.7" }, { "key": "otel.library.version", "type": "string", "value": "1.30.0-alpha" }, { "key": "thread.name", "type": "string", "value": "pool-6-thread-4" }, { "key": "messaging.destination.name", "type": "string", "value": "<default>" }, { "key": "messaging.operation", "type": "string", "value": "process" }, { "key": "messaging.system", "type": "string", "value": "rabbitmq" }, { "key": "messaging.message.payload_size_bytes", "type": "int64", "value": 89 }, { "key": "messaging.rabbitmq.destination.routing_key", "type": "string", "value": "..." }, { "key": "thread.id", "type": "int64", "value": 62 }, { "key": "span.kind", "type": "string", "value": "consumer" }, { "key": "internal.span.format", "type": "string", "value": "proto" } ] ...
Elasticsearch:
{ "_index": "trace_index", "_id": "y0Es5ooBnMZJW47qYVws", "_version": 1, "_seq_no": 1545, "_primary_term": 13, "found": true, "_source": { "@timestamp": "2023-09-30T12:59:28.252146600Z", "Attributes": { "messaging": { "destination": { "name": "<default>" }, "message": { "payload_size_bytes": 89 }, "operation": "process", "rabbitmq": { "destination": { "routing_key": "..." } }, "system": "rabbitmq" }, "thread": { "id": 77, "name": "pool-6-thread-10" } }, "Duration": 67, "EndTimestamp": "2023-09-30T12:59:28.252213700Z", "Kind": "SPAN_KIND_CONSUMER", "Link": "[]", "Name": "...", "ParentSpanId": "f9e140fe07c276e3", "Resource": { ... }, "SpanId": "5a29c7977f3b1532", "TraceId": "96e6b94f282db97ae089ab79771d0a5a", "TraceStatus": 0 } }
Here is Otel collector (ver: 0.85.0) config:
receivers:
otlp:
protocols:
http:

processors:
batch:

exporters:
otlp:
endpoint: "jaeger:4317"
elasticsearch/trace:
endpoints: "http://elasticsearch:9200"
traces_index: trace_index
mapping:
dedup: false

service:
pipelines:
traces:
receivers: [otlp]
exporters: [otlp,elasticsearch/trace]

@minhntt1 minhntt1 added the needs triage New item requiring triage label Sep 30, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

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

@JaredTan95 JaredTan95 added enhancement New feature or request and removed needs triage New item requiring triage labels Oct 1, 2023
mx-psi pushed a commit that referenced this issue Oct 23, 2023
#27288)

**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

**Link to tracking Issue:**
#27282

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

---------

Signed-off-by: Jared Tan <[email protected]>
Signed-off-by: Dominik Rosiek <[email protected]>
Signed-off-by: Raphael Silva <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
Co-authored-by: Antoine Toulme <[email protected]>
Co-authored-by: Raj Nishtala <[email protected]>
Co-authored-by: Dominik Rosiek <[email protected]>
Co-authored-by: Povilas Versockas <[email protected]>
Co-authored-by: Priyanshu Raj <[email protected]>
Co-authored-by: sakulali <[email protected]>
Co-authored-by: Raphael Philipe Mendes da Silva <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>
Co-authored-by: Yotam loewenbach <[email protected]>
Co-authored-by: Alex Boten <[email protected]>
Co-authored-by: bryan-aguilar <[email protected]>
Co-authored-by: Daniel Jaglowski <[email protected]>
Co-authored-by: Jina Jain <[email protected]>
Co-authored-by: Yang Song <[email protected]>
Co-authored-by: Dmitrii Anoshin <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Abhishek Saharn <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Ziqi Zhao <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Faith Chikwekwe <[email protected]>
Co-authored-by: Tyler Helmuth <[email protected]>
Co-authored-by: Daniel Kuiper <[email protected]>
Co-authored-by: Carlos Castro <[email protected]>
Co-authored-by: Christian <[email protected]>
Co-authored-by: ArchangelSDY <[email protected]>
Co-authored-by: Pavol Loffay <[email protected]>
Co-authored-by: Paulo Janotti <[email protected]>
Co-authored-by: Nathan Burke <[email protected]>
Co-authored-by: VihasMakwana <[email protected]>
Co-authored-by: shalper2 <[email protected]>
Co-authored-by: OpenTelemetry Bot <[email protected]>
Co-authored-by: Martin Majlis <[email protected]>
Co-authored-by: hovavza <[email protected]>
sigilioso added a commit to carlossscastro/opentelemetry-collector-contrib that referenced this issue Oct 27, 2023
open-telemetry#27288)

**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

**Link to tracking Issue:**
open-telemetry#27282

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

---------

Signed-off-by: Jared Tan <[email protected]>
Signed-off-by: Dominik Rosiek <[email protected]>
Signed-off-by: Raphael Silva <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
Co-authored-by: Antoine Toulme <[email protected]>
Co-authored-by: Raj Nishtala <[email protected]>
Co-authored-by: Dominik Rosiek <[email protected]>
Co-authored-by: Povilas Versockas <[email protected]>
Co-authored-by: Priyanshu Raj <[email protected]>
Co-authored-by: sakulali <[email protected]>
Co-authored-by: Raphael Philipe Mendes da Silva <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>
Co-authored-by: Yotam loewenbach <[email protected]>
Co-authored-by: Alex Boten <[email protected]>
Co-authored-by: bryan-aguilar <[email protected]>
Co-authored-by: Daniel Jaglowski <[email protected]>
Co-authored-by: Jina Jain <[email protected]>
Co-authored-by: Yang Song <[email protected]>
Co-authored-by: Dmitrii Anoshin <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Abhishek Saharn <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Ziqi Zhao <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Faith Chikwekwe <[email protected]>
Co-authored-by: Tyler Helmuth <[email protected]>
Co-authored-by: Daniel Kuiper <[email protected]>
Co-authored-by: Carlos Castro <[email protected]>
Co-authored-by: Christian <[email protected]>
Co-authored-by: ArchangelSDY <[email protected]>
Co-authored-by: Pavol Loffay <[email protected]>
Co-authored-by: Paulo Janotti <[email protected]>
Co-authored-by: Nathan Burke <[email protected]>
Co-authored-by: VihasMakwana <[email protected]>
Co-authored-by: shalper2 <[email protected]>
Co-authored-by: OpenTelemetry Bot <[email protected]>
Co-authored-by: Martin Majlis <[email protected]>
Co-authored-by: hovavza <[email protected]>
jmsnll pushed a commit to jmsnll/opentelemetry-collector-contrib that referenced this issue Nov 12, 2023
open-telemetry#27288)

**Description:** <Describe what has changed.>
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->

**Link to tracking Issue:**
open-telemetry#27282

**Testing:** <Describe what testing was performed and which tests were
added.>

**Documentation:** <Describe the documentation added.>

---------

Signed-off-by: Jared Tan <[email protected]>
Signed-off-by: Dominik Rosiek <[email protected]>
Signed-off-by: Raphael Silva <[email protected]>
Signed-off-by: Alex Boten <[email protected]>
Signed-off-by: Pavol Loffay <[email protected]>
Co-authored-by: Antoine Toulme <[email protected]>
Co-authored-by: Raj Nishtala <[email protected]>
Co-authored-by: Dominik Rosiek <[email protected]>
Co-authored-by: Povilas Versockas <[email protected]>
Co-authored-by: Priyanshu Raj <[email protected]>
Co-authored-by: sakulali <[email protected]>
Co-authored-by: Raphael Philipe Mendes da Silva <[email protected]>
Co-authored-by: Anthony Mirabella <[email protected]>
Co-authored-by: Yotam loewenbach <[email protected]>
Co-authored-by: Alex Boten <[email protected]>
Co-authored-by: bryan-aguilar <[email protected]>
Co-authored-by: Daniel Jaglowski <[email protected]>
Co-authored-by: Jina Jain <[email protected]>
Co-authored-by: Yang Song <[email protected]>
Co-authored-by: Dmitrii Anoshin <[email protected]>
Co-authored-by: Curtis Robert <[email protected]>
Co-authored-by: Abhishek Saharn <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Ziqi Zhao <[email protected]>
Co-authored-by: Ramachandran A G <[email protected]>
Co-authored-by: Faith Chikwekwe <[email protected]>
Co-authored-by: Tyler Helmuth <[email protected]>
Co-authored-by: Daniel Kuiper <[email protected]>
Co-authored-by: Carlos Castro <[email protected]>
Co-authored-by: Christian <[email protected]>
Co-authored-by: ArchangelSDY <[email protected]>
Co-authored-by: Pavol Loffay <[email protected]>
Co-authored-by: Paulo Janotti <[email protected]>
Co-authored-by: Nathan Burke <[email protected]>
Co-authored-by: VihasMakwana <[email protected]>
Co-authored-by: shalper2 <[email protected]>
Co-authored-by: OpenTelemetry Bot <[email protected]>
Co-authored-by: Martin Majlis <[email protected]>
Co-authored-by: hovavza <[email protected]>
Copy link
Contributor

github-actions bot commented Dec 4, 2023

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

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

@github-actions github-actions bot added the Stale label Dec 4, 2023
@minhntt1 minhntt1 closed this as completed Dec 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants