Skip to content

[9.2] Various tracing fixes (#137908)#138109

Merged
elasticsearchmachine merged 1 commit intoelastic:9.2from
mosche:backport/9.2/pr-137908
Nov 14, 2025
Merged

[9.2] Various tracing fixes (#137908)#138109
elasticsearchmachine merged 1 commit intoelastic:9.2from
mosche:backport/9.2/pr-137908

Conversation

@mosche
Copy link
Copy Markdown
Contributor

@mosche mosche commented Nov 14, 2025

Backports the following commits to 9.2:

- Only continue tracing in TaskManager if a parent APM trace context exists. Trace headers might
be set by external transactions as well. If present, it caused APMTracer to report a transaction
for every span when in fact being sampled out.

- Correctly report the duration of transactions if not recording by not discarding the root span
immediately. These transactions might still get reported, but without spans.

- Discard transient trace start time in `newTraceContext` when a parent APM trace context
already exists. If propagated, all spans would start at the same time.

Relates to ES-13389
@mosche mosche requested a review from a team as a code owner November 14, 2025 16:02
@mosche mosche added :Core/Infra/Metrics Metrics and metering infrastructure >refactoring auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport serverless-linked Added by automation, don't add manually Team:Core/Infra Meta label for core/infra team labels Nov 14, 2025
@elasticsearchmachine elasticsearchmachine merged commit b189842 into elastic:9.2 Nov 14, 2025
34 checks passed
@mosche mosche deleted the backport/9.2/pr-137908 branch November 14, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Metrics Metrics and metering infrastructure >refactoring serverless-linked Added by automation, don't add manually Team:Core/Infra Meta label for core/infra team v9.2.2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants