-
Notifications
You must be signed in to change notification settings - Fork 312
Fix pekko instrumentation in a rare edge case #9676
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
Draft
vandonr
wants to merge
1
commit into
master
Choose a base branch
from
vandonr/fix
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 8 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.026 s) : 0, 1025613
Total [baseline] (8.626 s) : 0, 8625552
Agent [candidate] (1.021 s) : 0, 1020754
Total [candidate] (8.656 s) : 0, 8655927
section iast
Agent [baseline] (1.161 s) : 0, 1160634
Total [baseline] (9.261 s) : 0, 9261225
Agent [candidate] (1.152 s) : 0, 1152412
Total [candidate] (9.215 s) : 0, 9215059
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.468 ms) : 0, 1468
crashtracking [candidate] (1.455 ms) : 0, 1455
BytebuddyAgent [baseline] (699.543 ms) : 0, 699543
BytebuddyAgent [candidate] (694.767 ms) : 0, 694767
GlobalTracer [baseline] (243.086 ms) : 0, 243086
GlobalTracer [candidate] (243.064 ms) : 0, 243064
AppSec [baseline] (33.059 ms) : 0, 33059
AppSec [candidate] (32.928 ms) : 0, 32928
Debugger [baseline] (6.457 ms) : 0, 6457
Debugger [candidate] (6.421 ms) : 0, 6421
Remote Config [baseline] (695.333 µs) : 0, 695
Remote Config [candidate] (698.614 µs) : 0, 699
Telemetry [baseline] (9.416 ms) : 0, 9416
Telemetry [candidate] (9.237 ms) : 0, 9237
Flare Poller [baseline] (10.753 ms) : 0, 10753
Flare Poller [candidate] (11.104 ms) : 0, 11104
section iast
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.459 ms) : 0, 1459
BytebuddyAgent [baseline] (820.593 ms) : 0, 820593
BytebuddyAgent [candidate] (816.247 ms) : 0, 816247
GlobalTracer [baseline] (234.52 ms) : 0, 234520
GlobalTracer [candidate] (232.227 ms) : 0, 232227
AppSec [baseline] (35.826 ms) : 0, 35826
AppSec [candidate] (35.107 ms) : 0, 35107
Debugger [baseline] (6.173 ms) : 0, 6173
Debugger [candidate] (6.141 ms) : 0, 6141
Remote Config [baseline] (632.52 µs) : 0, 633
Remote Config [candidate] (606.239 µs) : 0, 606
Telemetry [baseline] (8.696 ms) : 0, 8696
Telemetry [candidate] (8.461 ms) : 0, 8461
Flare Poller [baseline] (4.193 ms) : 0, 4193
Flare Poller [candidate] (4.164 ms) : 0, 4164
IAST [baseline] (27.349 ms) : 0, 27349
IAST [candidate] (26.858 ms) : 0, 26858
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.023 s) : 0, 1022669
Total [baseline] (10.735 s) : 0, 10734776
Agent [candidate] (1.02 s) : 0, 1020304
Total [candidate] (10.654 s) : 0, 10654023
section appsec
Agent [baseline] (1.194 s) : 0, 1194188
Total [baseline] (11.065 s) : 0, 11064767
Agent [candidate] (1.192 s) : 0, 1191747
Total [candidate] (10.922 s) : 0, 10921980
section iast
Agent [baseline] (1.161 s) : 0, 1160747
Total [baseline] (10.945 s) : 0, 10944780
Agent [candidate] (1.15 s) : 0, 1150035
Total [candidate] (10.874 s) : 0, 10873543
section profiling
Agent [baseline] (1.174 s) : 0, 1173930
Total [baseline] (11.139 s) : 0, 11139363
Agent [candidate] (1.165 s) : 0, 1164896
Total [candidate] (10.999 s) : 0, 10999251
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.487 ms) : 0, 1487
crashtracking [candidate] (1.459 ms) : 0, 1459
BytebuddyAgent [baseline] (697.451 ms) : 0, 697451
BytebuddyAgent [candidate] (694.838 ms) : 0, 694838
GlobalTracer [baseline] (241.711 ms) : 0, 241711
GlobalTracer [candidate] (242.214 ms) : 0, 242214
AppSec [baseline] (32.779 ms) : 0, 32779
AppSec [candidate] (32.911 ms) : 0, 32911
Debugger [baseline] (6.429 ms) : 0, 6429
Debugger [candidate] (6.465 ms) : 0, 6465
Remote Config [baseline] (694.796 µs) : 0, 695
Remote Config [candidate] (683.706 µs) : 0, 684
Telemetry [baseline] (9.249 ms) : 0, 9249
Telemetry [candidate] (9.337 ms) : 0, 9337
Flare Poller [baseline] (11.787 ms) : 0, 11787
Flare Poller [candidate] (11.305 ms) : 0, 11305
section appsec
crashtracking [baseline] (1.464 ms) : 0, 1464
crashtracking [candidate] (1.466 ms) : 0, 1466
BytebuddyAgent [baseline] (717.61 ms) : 0, 717610
BytebuddyAgent [candidate] (716.003 ms) : 0, 716003
GlobalTracer [baseline] (234.487 ms) : 0, 234487
GlobalTracer [candidate] (233.867 ms) : 0, 233867
AppSec [baseline] (173.055 ms) : 0, 173055
AppSec [candidate] (174.463 ms) : 0, 174463
Debugger [baseline] (6.192 ms) : 0, 6192
Debugger [candidate] (6.19 ms) : 0, 6190
Remote Config [baseline] (645.382 µs) : 0, 645
Remote Config [candidate] (647.332 µs) : 0, 647
Telemetry [baseline] (10.071 ms) : 0, 10071
Telemetry [candidate] (9.361 ms) : 0, 9361
Flare Poller [baseline] (4.777 ms) : 0, 4777
Flare Poller [candidate] (4.024 ms) : 0, 4024
IAST [baseline] (24.843 ms) : 0, 24843
IAST [candidate] (24.703 ms) : 0, 24703
section iast
crashtracking [baseline] (1.468 ms) : 0, 1468
crashtracking [candidate] (1.455 ms) : 0, 1455
BytebuddyAgent [baseline] (822.307 ms) : 0, 822307
BytebuddyAgent [candidate] (813.849 ms) : 0, 813849
GlobalTracer [baseline] (233.576 ms) : 0, 233576
GlobalTracer [candidate] (232.348 ms) : 0, 232348
AppSec [baseline] (35.614 ms) : 0, 35614
AppSec [candidate] (35.168 ms) : 0, 35168
Debugger [baseline] (6.078 ms) : 0, 6078
Debugger [candidate] (6.051 ms) : 0, 6051
Remote Config [baseline] (623.702 µs) : 0, 624
Remote Config [candidate] (599.493 µs) : 0, 599
Telemetry [baseline] (8.489 ms) : 0, 8489
Telemetry [candidate] (8.406 ms) : 0, 8406
Flare Poller [baseline] (4.205 ms) : 0, 4205
Flare Poller [candidate] (4.184 ms) : 0, 4184
IAST [baseline] (27.09 ms) : 0, 27090
IAST [candidate] (26.746 ms) : 0, 26746
section profiling
ProfilingAgent [baseline] (108.5 ms) : 0, 108500
ProfilingAgent [candidate] (107.505 ms) : 0, 107505
crashtracking [baseline] (1.433 ms) : 0, 1433
crashtracking [candidate] (1.434 ms) : 0, 1434
BytebuddyAgent [baseline] (729.115 ms) : 0, 729115
BytebuddyAgent [candidate] (724.948 ms) : 0, 724948
GlobalTracer [baseline] (219.894 ms) : 0, 219894
GlobalTracer [candidate] (217.145 ms) : 0, 217145
AppSec [baseline] (33.469 ms) : 0, 33469
AppSec [candidate] (33.101 ms) : 0, 33101
Debugger [baseline] (7.361 ms) : 0, 7361
Debugger [candidate] (6.461 ms) : 0, 6461
Remote Config [baseline] (720.244 µs) : 0, 720
Remote Config [candidate] (694.539 µs) : 0, 695
Telemetry [baseline] (15.978 ms) : 0, 15978
Telemetry [candidate] (15.823 ms) : 0, 15823
Flare Poller [baseline] (4.337 ms) : 0, 4337
Flare Poller [candidate] (4.947 ms) : 0, 4947
Profiling [baseline] (109.235 ms) : 0, 109235
Profiling [candidate] (108.13 ms) : 0, 108130
LoadParameters
See matching parameters
SummaryFound 6 performance improvements and 0 performance regressions! Performance is the same for 6 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section baseline
no_agent (4.493 ms) : 4438, 4548
. : milestone, 4493,
iast (10.043 ms) : 9872, 10214
. : milestone, 10043,
iast_FULL (13.943 ms) : 13666, 14220
. : milestone, 13943,
iast_GLOBAL (11.129 ms) : 10931, 11328
. : milestone, 11129,
profiling (9.372 ms) : 9214, 9530
. : milestone, 9372,
tracing (8.218 ms) : 8098, 8338
. : milestone, 8218,
section candidate
no_agent (4.395 ms) : 4341, 4449
. : milestone, 4395,
iast (9.989 ms) : 9814, 10165
. : milestone, 9989,
iast_FULL (14.448 ms) : 14159, 14738
. : milestone, 14448,
iast_GLOBAL (11.208 ms) : 11000, 11416
. : milestone, 11208,
profiling (8.76 ms) : 8620, 8900
. : milestone, 8760,
tracing (7.71 ms) : 7602, 7819
. : milestone, 7710,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section baseline
no_agent (37.967 ms) : 37652, 38283
. : milestone, 37967,
appsec (46.738 ms) : 46325, 47152
. : milestone, 46738,
code_origins (43.665 ms) : 43298, 44032
. : milestone, 43665,
iast (45.44 ms) : 45058, 45822
. : milestone, 45440,
profiling (47.972 ms) : 47485, 48458
. : milestone, 47972,
tracing (45.777 ms) : 45391, 46164
. : milestone, 45777,
section candidate
no_agent (35.617 ms) : 35335, 35899
. : milestone, 35617,
appsec (47.779 ms) : 47368, 48191
. : milestone, 47779,
code_origins (42.382 ms) : 42015, 42749
. : milestone, 42382,
iast (43.062 ms) : 42695, 43429
. : milestone, 43062,
profiling (46.252 ms) : 45817, 46688
. : milestone, 46252,
tracing (45.53 ms) : 45130, 45930
. : milestone, 45530,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section baseline
no_agent (15.5 s) : 15500000, 15500000
. : milestone, 15500000,
appsec (15.193 s) : 15193000, 15193000
. : milestone, 15193000,
iast (18.69 s) : 18690000, 18690000
. : milestone, 18690000,
iast_GLOBAL (18.182 s) : 18182000, 18182000
. : milestone, 18182000,
profiling (15.458 s) : 15458000, 15458000
. : milestone, 15458000,
tracing (15.077 s) : 15077000, 15077000
. : milestone, 15077000,
section candidate
no_agent (15.65 s) : 15650000, 15650000
. : milestone, 15650000,
appsec (15.039 s) : 15039000, 15039000
. : milestone, 15039000,
iast (18.421 s) : 18421000, 18421000
. : milestone, 18421000,
iast_GLOBAL (17.634 s) : 17634000, 17634000
. : milestone, 17634000,
profiling (15.422 s) : 15422000, 15422000
. : milestone, 15422000,
tracing (15.207 s) : 15207000, 15207000
. : milestone, 15207000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~f99014d8ad, baseline=1.54.0-SNAPSHOT~20dac8a306
dateFormat X
axisFormat %s
section baseline
no_agent (1.475 ms) : 1463, 1486
. : milestone, 1475,
appsec (3.724 ms) : 3504, 3945
. : milestone, 3724,
iast (2.207 ms) : 2144, 2270
. : milestone, 2207,
iast_GLOBAL (2.253 ms) : 2189, 2317
. : milestone, 2253,
profiling (2.04 ms) : 1989, 2091
. : milestone, 2040,
tracing (2.026 ms) : 1977, 2075
. : milestone, 2026,
section candidate
no_agent (1.479 ms) : 1467, 1490
. : milestone, 1479,
appsec (3.74 ms) : 3522, 3959
. : milestone, 3740,
iast (2.199 ms) : 2136, 2262
. : milestone, 2199,
iast_GLOBAL (2.252 ms) : 2188, 2315
. : milestone, 2252,
profiling (2.047 ms) : 1996, 2098
. : milestone, 2047,
tracing (2.036 ms) : 1987, 2086
. : milestone, 2036,
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What Does This Do
pekko tests on error handling are flaky, and Andrea identified the issue in the logs as (probably) a span that has a continuation that is left open. We see this from this log:
PendingTrace enqueued but won't be reported.
Thus, the span is not written immediately, and since the test that is failing is waiting for the trace with a timeout, we don't see the trace we expect within the set timeframe.
Since I have no experience with Pekko or that instrumentation, I called an AI for help, which highlighted this comment:
dd-trace-java/dd-java-agent/instrumentation/pekko/pekko-http-1.0/src/main/java/datadog/trace/instrumentation/pekkohttp/DatadogServerRequestResponseFlowWrapper.java
Lines 70 to 74 in f99014d
as a potential source for the lost continuation. It then suggested fixing the
onUpstreamFailure
handler, which is what I'm pushing here.Motivation
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any useful labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]