Track server context in netty4 instrumentation#10556
Track server context in netty4 instrumentation#10556gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit intomasterfrom
Conversation
PerfectSlayer
left a comment
There was a problem hiding this comment.
Make sense 👍 Thanks for the fix!
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 62 metrics, 9 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.072 s) : 0, 1071712
Total [baseline] (10.932 s) : 0, 10931696
Agent [candidate] (1.073 s) : 0, 1072863
Total [candidate] (10.925 s) : 0, 10924735
section appsec
Agent [baseline] (1.248 s) : 0, 1247751
Total [baseline] (11.067 s) : 0, 11067062
Agent [candidate] (1.242 s) : 0, 1241589
Total [candidate] (11.134 s) : 0, 11134153
section iast
Agent [baseline] (1.233 s) : 0, 1233309
Total [baseline] (11.262 s) : 0, 11262038
Agent [candidate] (1.236 s) : 0, 1236274
Total [candidate] (11.18 s) : 0, 11179678
section profiling
Agent [baseline] (1.196 s) : 0, 1195708
Total [baseline] (10.973 s) : 0, 10972765
Agent [candidate] (1.191 s) : 0, 1190750
Total [candidate] (10.98 s) : 0, 10979937
gantt
title petclinic - break down per module: candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.189 ms) : 0, 1189
crashtracking [candidate] (1.184 ms) : 0, 1184
BytebuddyAgent [baseline] (633.863 ms) : 0, 633863
BytebuddyAgent [candidate] (633.979 ms) : 0, 633979
AgentMeter [baseline] (29.115 ms) : 0, 29115
AgentMeter [candidate] (29.089 ms) : 0, 29089
GlobalTracer [baseline] (259.547 ms) : 0, 259547
GlobalTracer [candidate] (259.875 ms) : 0, 259875
AppSec [baseline] (33.053 ms) : 0, 33053
AppSec [candidate] (33.065 ms) : 0, 33065
Debugger [baseline] (62.005 ms) : 0, 62005
Debugger [candidate] (63.066 ms) : 0, 63066
Remote Config [baseline] (624.336 µs) : 0, 624
Remote Config [candidate] (634.451 µs) : 0, 634
Telemetry [baseline] (11.569 ms) : 0, 11569
Telemetry [candidate] (12.102 ms) : 0, 12102
Flare Poller [baseline] (5.335 ms) : 0, 5335
Flare Poller [candidate] (4.489 ms) : 0, 4489
section appsec
crashtracking [baseline] (1.2 ms) : 0, 1200
crashtracking [candidate] (1.178 ms) : 0, 1178
BytebuddyAgent [baseline] (663.309 ms) : 0, 663309
BytebuddyAgent [candidate] (659.804 ms) : 0, 659804
AgentMeter [baseline] (11.996 ms) : 0, 11996
AgentMeter [candidate] (11.971 ms) : 0, 11971
GlobalTracer [baseline] (260.14 ms) : 0, 260140
GlobalTracer [candidate] (259.41 ms) : 0, 259410
IAST [baseline] (25.544 ms) : 0, 25544
IAST [candidate] (25.306 ms) : 0, 25306
AppSec [baseline] (169.965 ms) : 0, 169965
AppSec [candidate] (167.732 ms) : 0, 167732
Debugger [baseline] (66.744 ms) : 0, 66744
Debugger [candidate] (67.381 ms) : 0, 67381
Remote Config [baseline] (664.222 µs) : 0, 664
Remote Config [candidate] (667.082 µs) : 0, 667
Telemetry [baseline] (9.081 ms) : 0, 9081
Telemetry [candidate] (9.13 ms) : 0, 9130
Flare Poller [baseline] (3.707 ms) : 0, 3707
Flare Poller [candidate] (3.667 ms) : 0, 3667
section iast
crashtracking [baseline] (1.181 ms) : 0, 1181
crashtracking [candidate] (1.192 ms) : 0, 1192
BytebuddyAgent [baseline] (796.253 ms) : 0, 796253
BytebuddyAgent [candidate] (798.646 ms) : 0, 798646
AgentMeter [baseline] (11.235 ms) : 0, 11235
AgentMeter [candidate] (11.275 ms) : 0, 11275
GlobalTracer [baseline] (248.696 ms) : 0, 248696
GlobalTracer [candidate] (249.544 ms) : 0, 249544
IAST [baseline] (27.072 ms) : 0, 27072
IAST [candidate] (26.959 ms) : 0, 26959
AppSec [baseline] (33.281 ms) : 0, 33281
AppSec [candidate] (34.021 ms) : 0, 34021
Debugger [baseline] (67.655 ms) : 0, 67655
Debugger [candidate] (66.583 ms) : 0, 66583
Remote Config [baseline] (548.013 µs) : 0, 548
Remote Config [candidate] (547.794 µs) : 0, 548
Telemetry [baseline] (8.649 ms) : 0, 8649
Telemetry [candidate] (8.726 ms) : 0, 8726
Flare Poller [baseline] (3.515 ms) : 0, 3515
Flare Poller [candidate] (3.486 ms) : 0, 3486
section profiling
crashtracking [baseline] (1.225 ms) : 0, 1225
crashtracking [candidate] (1.209 ms) : 0, 1209
BytebuddyAgent [baseline] (685.66 ms) : 0, 685660
BytebuddyAgent [candidate] (683.117 ms) : 0, 683117
AgentMeter [baseline] (8.604 ms) : 0, 8604
AgentMeter [candidate] (8.538 ms) : 0, 8538
GlobalTracer [baseline] (216.768 ms) : 0, 216768
GlobalTracer [candidate] (216.067 ms) : 0, 216067
AppSec [baseline] (32.769 ms) : 0, 32769
AppSec [candidate] (32.287 ms) : 0, 32287
Debugger [baseline] (67.933 ms) : 0, 67933
Debugger [candidate] (67.41 ms) : 0, 67410
Remote Config [baseline] (609.239 µs) : 0, 609
Remote Config [candidate] (597.702 µs) : 0, 598
Telemetry [baseline] (8.883 ms) : 0, 8883
Telemetry [candidate] (8.777 ms) : 0, 8777
Flare Poller [baseline] (3.821 ms) : 0, 3821
Flare Poller [candidate] (3.778 ms) : 0, 3778
ProfilingAgent [baseline] (99.362 ms) : 0, 99362
ProfilingAgent [candidate] (99.305 ms) : 0, 99305
Profiling [baseline] (99.928 ms) : 0, 99928
Profiling [candidate] (99.871 ms) : 0, 99871
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.082 s) : 0, 1081980
Total [baseline] (8.83 s) : 0, 8830492
Agent [candidate] (1.068 s) : 0, 1068083
Total [candidate] (8.784 s) : 0, 8783646
section iast
Agent [baseline] (1.241 s) : 0, 1241236
Total [baseline] (9.397 s) : 0, 9397195
Agent [candidate] (1.237 s) : 0, 1237014
Total [candidate] (9.427 s) : 0, 9427000
gantt
title insecure-bank - break down per module: candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.207 ms) : 0, 1207
crashtracking [candidate] (1.175 ms) : 0, 1175
BytebuddyAgent [baseline] (639.665 ms) : 0, 639665
BytebuddyAgent [candidate] (633.086 ms) : 0, 633086
AgentMeter [baseline] (29.369 ms) : 0, 29369
AgentMeter [candidate] (28.994 ms) : 0, 28994
GlobalTracer [baseline] (261.141 ms) : 0, 261141
GlobalTracer [candidate] (258.371 ms) : 0, 258371
AppSec [baseline] (33.428 ms) : 0, 33428
AppSec [candidate] (32.89 ms) : 0, 32890
Debugger [baseline] (60.576 ms) : 0, 60576
Debugger [candidate] (60.879 ms) : 0, 60879
Remote Config [baseline] (701.769 µs) : 0, 702
Remote Config [candidate] (642.148 µs) : 0, 642
Telemetry [baseline] (12.52 ms) : 0, 12520
Telemetry [candidate] (9.957 ms) : 0, 9957
Flare Poller [baseline] (7.718 ms) : 0, 7718
Flare Poller [candidate] (6.858 ms) : 0, 6858
section iast
crashtracking [baseline] (1.206 ms) : 0, 1206
crashtracking [candidate] (1.184 ms) : 0, 1184
BytebuddyAgent [baseline] (802.333 ms) : 0, 802333
BytebuddyAgent [candidate] (800.954 ms) : 0, 800954
AgentMeter [baseline] (11.488 ms) : 0, 11488
AgentMeter [candidate] (11.263 ms) : 0, 11263
GlobalTracer [baseline] (250.253 ms) : 0, 250253
GlobalTracer [candidate] (249.308 ms) : 0, 249308
IAST [baseline] (27.089 ms) : 0, 27089
IAST [candidate] (26.834 ms) : 0, 26834
AppSec [baseline] (35.017 ms) : 0, 35017
AppSec [candidate] (33.042 ms) : 0, 33042
Debugger [baseline] (65.591 ms) : 0, 65591
Debugger [candidate] (66.325 ms) : 0, 66325
Remote Config [baseline] (554.062 µs) : 0, 554
Remote Config [candidate] (540.37 µs) : 0, 540
Telemetry [baseline] (8.729 ms) : 0, 8729
Telemetry [candidate] (8.654 ms) : 0, 8654
Flare Poller [baseline] (3.513 ms) : 0, 3513
Flare Poller [candidate] (3.476 ms) : 0, 3476
LoadParameters
See matching parameters
SummaryFound 4 performance improvements and 4 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section baseline
no_agent (1.181 ms) : 1170, 1193
. : milestone, 1181,
iast (3.247 ms) : 3203, 3292
. : milestone, 3247,
iast_FULL (5.806 ms) : 5748, 5864
. : milestone, 5806,
iast_GLOBAL (3.721 ms) : 3661, 3780
. : milestone, 3721,
profiling (1.963 ms) : 1947, 1979
. : milestone, 1963,
tracing (1.783 ms) : 1768, 1798
. : milestone, 1783,
section candidate
no_agent (1.172 ms) : 1161, 1183
. : milestone, 1172,
iast (3.14 ms) : 3101, 3178
. : milestone, 3140,
iast_FULL (5.806 ms) : 5747, 5864
. : milestone, 5806,
iast_GLOBAL (3.706 ms) : 3636, 3775
. : milestone, 3706,
profiling (2.082 ms) : 2063, 2100
. : milestone, 2082,
tracing (1.805 ms) : 1790, 1820
. : milestone, 1805,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section baseline
no_agent (19.207 ms) : 19010, 19404
. : milestone, 19207,
appsec (18.644 ms) : 18454, 18833
. : milestone, 18644,
code_origins (17.816 ms) : 17643, 17990
. : milestone, 17816,
iast (18.709 ms) : 18522, 18897
. : milestone, 18709,
profiling (18.734 ms) : 18544, 18923
. : milestone, 18734,
tracing (17.565 ms) : 17390, 17739
. : milestone, 17565,
section candidate
no_agent (17.127 ms) : 16954, 17299
. : milestone, 17127,
appsec (19.42 ms) : 19218, 19622
. : milestone, 19420,
code_origins (17.798 ms) : 17622, 17974
. : milestone, 17798,
iast (17.827 ms) : 17650, 18003
. : milestone, 17827,
profiling (19.635 ms) : 19433, 19837
. : milestone, 19635,
tracing (18.624 ms) : 18438, 18811
. : milestone, 18624,
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.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section baseline
no_agent (15.109 s) : 15109000, 15109000
. : milestone, 15109000,
appsec (14.554 s) : 14554000, 14554000
. : milestone, 14554000,
iast (18.344 s) : 18344000, 18344000
. : milestone, 18344000,
iast_GLOBAL (18.717 s) : 18717000, 18717000
. : milestone, 18717000,
profiling (15.673 s) : 15673000, 15673000
. : milestone, 15673000,
tracing (14.628 s) : 14628000, 14628000
. : milestone, 14628000,
section candidate
no_agent (15.67 s) : 15670000, 15670000
. : milestone, 15670000,
appsec (15.023 s) : 15023000, 15023000
. : milestone, 15023000,
iast (17.569 s) : 17569000, 17569000
. : milestone, 17569000,
iast_GLOBAL (17.774 s) : 17774000, 17774000
. : milestone, 17774000,
profiling (15.149 s) : 15149000, 15149000
. : milestone, 15149000,
tracing (15.094 s) : 15094000, 15094000
. : milestone, 15094000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.60.0-SNAPSHOT~6ccc620cd3, baseline=1.60.0-SNAPSHOT~d10055d2a1
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1461, 1484
. : milestone, 1473,
appsec (3.747 ms) : 3527, 3967
. : milestone, 3747,
iast (2.255 ms) : 2185, 2325
. : milestone, 2255,
iast_GLOBAL (2.3 ms) : 2230, 2370
. : milestone, 2300,
profiling (2.083 ms) : 2028, 2138
. : milestone, 2083,
tracing (2.072 ms) : 2019, 2126
. : milestone, 2072,
section candidate
no_agent (1.475 ms) : 1464, 1487
. : milestone, 1475,
appsec (3.764 ms) : 3542, 3987
. : milestone, 3764,
iast (2.254 ms) : 2185, 2324
. : milestone, 2254,
iast_GLOBAL (2.301 ms) : 2231, 2371
. : milestone, 2301,
profiling (2.094 ms) : 2038, 2151
. : milestone, 2094,
tracing (2.06 ms) : 2006, 2114
. : milestone, 2060,
|
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
mergequeue build completed successfully, but the github api returned an error while merging the pr DetailsError: PUT https://api.github.com/repos/DataDog/dd-trace-java/pulls/10556/merge: 405 Repository rule violations found Changes must be made through the merge queue [] (Request ID: 8D9C:1EBEA3:75915:1FA1A9:698A05EF) FullStacktrace: Changes must be made through the merge queue [] (Request ID: 8D9C:1EBEA3:75915:1FA1A9:698A05EF) (type: GitFailure, retryable: false): PUT https://api.github.com/repos/DataDog/dd-trace-java/pulls/10556/merge: 405 Repository rule violations found Changes must be made through the merge queue [] (type: ErrorResponse, retryable: true) |
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
73b2611
into
master
Motivation
We should track the full server context and not just the span, to avoid potentially losing custom context when re-attaching the server context.
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.