[connector/spanmetrics] Incorrect starttimestamps for subspans. #35994
Labels
Comments
shivanthzen
added
bug
shivanthzen
changed the title
shivanthzen
changed the title
|
Reopening |
shivanthzen
added a commit
to shivanthzen/opentelemetry-collector-contrib
that referenced
this issue
Nov 4, 2024
… of a metric is currently tied to the root span. When a "new" child span appears for trace (eg : unhappy path on an api call which results in a new subspan or an async process that was triggered much later), the time starttimestamp for the new metric for the new child span is that of the parent span(which can be well in the past). This MR moves the start timestamp from resource level (root span) to metric level(child span) . (Doesn't consider delta-temporality as of now). References: open-telemetry#35994 Upstream Fix: open-telemetry#36019
shivanthzen
added a commit
to shivanthzen/opentelemetry-collector-contrib
that referenced
this issue
Nov 4, 2024
Currently while generating metrics out of traces, the start timestamp of a metric is currently tied to the root span. When a "new" child span appears for trace (eg : unhappy path on an api call which results in a new subspan or an async process that was triggered much later), the time starttimestamp for the new metric for the new child span is that of the parent span(which can be well in the past). This MR moves the start timestamp from resource level (root span) to metric level(child span) . (Doesn't consider delta-temporality as of now). References: open-telemetry#35994 Upstream Fix: open-telemetry#36019
|
Ping @Frapschen @portertech |
This was referenced Nov 5, 2024
This was referenced Dec 17, 2024
|
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 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component(s)
connector/spanmetrics
What happened?
Description
Currently while generating metrics out of traces, the start timestamp of a metric is currently tied to the root span. When a "new" subspan appears for trace (eg : unhappy path on an api call which results in a new subspan), the time starttimestamp for the new metric for the new subspan is that of the root span(which can be well in the past).
Steps to Reproduce
We store start-timestamp at resource level rawmetrics in the spanconnector. This starttimestamp is used regardless of when the "new" subspan appeared.
Expected Result
The metric generated for the new span has a timestamp of the first occurance of the "new" span.
Actual Result
The metric generated for the new span has the time stamp corresponding start of the periodic trace
Collector version
v0.112.0
Environment information
Environment
OS: (e.g., "Ubuntu 20.04")
Compiler(if manually compiled): (e.g., "go 14.2")
OpenTelemetry Collector configuration
No response
Log output
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: