Skip to content

WO-1582 Add tracing getInvocationSpan - #7112

Draft
jmorrell-cloudflare wants to merge 5 commits into
jmorrell/add-get-active-spanfrom
jmorrell/add-get-invocation-span
Draft

WO-1582 Add tracing getInvocationSpan#7112
jmorrell-cloudflare wants to merge 5 commits into
jmorrell/add-get-active-spanfrom
jmorrell/add-get-invocation-span

Conversation

@jmorrell-cloudflare

Copy link
Copy Markdown
Contributor

Stack created with GitHub Stacks CLIGive Feedback 💬

@jmorrell-cloudflare
jmorrell-cloudflare requested review from a team as code owners August 25, 2026 00:06
@jmorrell-cloudflare
jmorrell-cloudflare requested review from edmundhung and removed request for a team August 25, 2026 00:06
@jmorrell-cloudflare jmorrell-cloudflare changed the title jmorrell/add get invocation span WO-1582 Add tracing getInvocationSpan Aug 25, 2026
@ask-bonk

ask-bonk Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

LGTM!

github run

@jmorrell-cloudflare
jmorrell-cloudflare marked this pull request as draft August 25, 2026 00:10
@codecov-commenter

codecov-commenter commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 30.23256% with 60 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.66%. Comparing base (562fea1) to head (99aa5ed).

Files with missing lines Patch % Lines
src/workerd/api/tracing.c++ 0.00% 44 Missing ⚠️
src/workerd/io/io-context.c++ 58.97% 10 Missing and 6 partials ⚠️
Additional details and impacted files
@@                       Coverage Diff                        @@
##           jmorrell/add-get-active-span    #7112      +/-   ##
================================================================
- Coverage                         67.68%   67.66%   -0.03%     
================================================================
  Files                               459      459              
  Lines                            129561   129647      +86     
  Branches                          21165    21184      +19     
================================================================
+ Hits                              87695    87720      +25     
- Misses                            29036    29089      +53     
- Partials                          12830    12838       +8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jmorrell-cloudflare
jmorrell-cloudflare force-pushed the jmorrell/add-get-invocation-span branch from 96e93b6 to 99aa5ed Compare August 25, 2026 00:37
@jmorrell-cloudflare
jmorrell-cloudflare force-pushed the jmorrell/add-get-active-span branch from 165a055 to 73eb9f4 Compare August 25, 2026 02:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants