-
Notifications
You must be signed in to change notification settings - Fork 303
Migrating all HttpClient Instrumentations to Inject Full Context #8826
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
base: master
Are you sure you want to change the base?
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 14 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.021 s) : 0, 1021434
Total [baseline] (8.723 s) : 0, 8723059
Agent [candidate] (1.018 s) : 0, 1017987
Total [candidate] (8.674 s) : 0, 8674382
section iast
Agent [baseline] (1.151 s) : 0, 1150846
Total [baseline] (9.225 s) : 0, 9225498
Agent [candidate] (1.149 s) : 0, 1149428
Total [candidate] (9.294 s) : 0, 9293753
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.153 s) : 0, 1152912
Total [baseline] (9.278 s) : 0, 9277592
Agent [candidate] (1.157 s) : 0, 1156965
Total [candidate] (9.251 s) : 0, 9251326
section iast_TELEMETRY_OFF
Agent [baseline] (1.146 s) : 0, 1146223
Total [baseline] (9.229 s) : 0, 9229229
Agent [candidate] (1.146 s) : 0, 1146251
Total [candidate] (9.284 s) : 0, 9284387
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.114 ms) : 0, 682114
BytebuddyAgent [candidate] (680.517 ms) : 0, 680517
GlobalTracer [baseline] (240.809 ms) : 0, 240809
GlobalTracer [candidate] (239.162 ms) : 0, 239162
AppSec [baseline] (55.35 ms) : 0, 55350
AppSec [candidate] (55.192 ms) : 0, 55192
Debugger [baseline] (8.425 ms) : 0, 8425
Debugger [candidate] (8.378 ms) : 0, 8378
Remote Config [baseline] (692.714 µs) : 0, 693
Remote Config [candidate] (695.88 µs) : 0, 696
Telemetry [baseline] (10.53 ms) : 0, 10530
Telemetry [candidate] (10.574 ms) : 0, 10574
section iast
BytebuddyAgent [baseline] (802.937 ms) : 0, 802937
BytebuddyAgent [candidate] (802.185 ms) : 0, 802185
GlobalTracer [baseline] (231.124 ms) : 0, 231124
GlobalTracer [candidate] (230.439 ms) : 0, 230439
IAST [baseline] (26.884 ms) : 0, 26884
IAST [candidate] (29.896 ms) : 0, 29896
AppSec [baseline] (51.32 ms) : 0, 51320
AppSec [candidate] (48.922 ms) : 0, 48922
Debugger [baseline] (5.907 ms) : 0, 5907
Debugger [candidate] (5.909 ms) : 0, 5909
Remote Config [baseline] (585.564 µs) : 0, 586
Remote Config [candidate] (628.581 µs) : 0, 629
Telemetry [baseline] (7.821 ms) : 0, 7821
Telemetry [candidate] (7.936 ms) : 0, 7936
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (803.784 ms) : 0, 803784
BytebuddyAgent [candidate] (807.379 ms) : 0, 807379
GlobalTracer [baseline] (231.873 ms) : 0, 231873
GlobalTracer [candidate] (231.893 ms) : 0, 231893
IAST [baseline] (28.312 ms) : 0, 28312
IAST [candidate] (29.766 ms) : 0, 29766
AppSec [baseline] (50.867 ms) : 0, 50867
AppSec [candidate] (49.794 ms) : 0, 49794
Debugger [baseline] (5.935 ms) : 0, 5935
Debugger [candidate] (5.925 ms) : 0, 5925
Remote Config [baseline] (592.504 µs) : 0, 593
Remote Config [candidate] (604.81 µs) : 0, 605
Telemetry [baseline] (7.973 ms) : 0, 7973
Telemetry [candidate] (7.958 ms) : 0, 7958
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.363 ms) : 0, 799363
BytebuddyAgent [candidate] (799.117 ms) : 0, 799117
GlobalTracer [baseline] (230.575 ms) : 0, 230575
GlobalTracer [candidate] (231.219 ms) : 0, 231219
IAST [baseline] (23.074 ms) : 0, 23074
IAST [candidate] (23.12 ms) : 0, 23120
AppSec [baseline] (55.41 ms) : 0, 55410
AppSec [candidate] (54.883 ms) : 0, 54883
Debugger [baseline] (5.942 ms) : 0, 5942
Debugger [candidate] (5.984 ms) : 0, 5984
Remote Config [baseline] (592.684 µs) : 0, 593
Remote Config [candidate] (596.163 µs) : 0, 596
Telemetry [baseline] (7.712 ms) : 0, 7712
Telemetry [candidate] (7.778 ms) : 0, 7778
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.02 s) : 0, 1020371
Total [baseline] (10.538 s) : 0, 10537922
Agent [candidate] (1.028 s) : 0, 1027603
Total [candidate] (10.507 s) : 0, 10506886
section appsec
Agent [baseline] (1.162 s) : 0, 1162411
Total [baseline] (10.673 s) : 0, 10672884
Agent [candidate] (1.16 s) : 0, 1160466
Total [candidate] (10.711 s) : 0, 10711330
section iast
Agent [baseline] (1.16 s) : 0, 1159919
Total [baseline] (10.939 s) : 0, 10938545
Agent [candidate] (1.154 s) : 0, 1154128
Total [candidate] (10.952 s) : 0, 10951965
section profiling
Agent [baseline] (1.293 s) : 0, 1293227
Total [baseline] (10.971 s) : 0, 10970879
Agent [candidate] (1.295 s) : 0, 1294713
Total [candidate] (10.99 s) : 0, 10989950
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.066 ms) : 0, 683066
BytebuddyAgent [candidate] (687.977 ms) : 0, 687977
GlobalTracer [baseline] (240.558 ms) : 0, 240558
GlobalTracer [candidate] (241.949 ms) : 0, 241949
AppSec [baseline] (54.394 ms) : 0, 54394
AppSec [candidate] (55.001 ms) : 0, 55001
Debugger [baseline] (8.344 ms) : 0, 8344
Debugger [candidate] (9.779 ms) : 0, 9779
Remote Config [baseline] (696.259 µs) : 0, 696
Remote Config [candidate] (712.819 µs) : 0, 713
Telemetry [baseline] (9.806 ms) : 0, 9806
Telemetry [candidate] (8.451 ms) : 0, 8451
section appsec
BytebuddyAgent [baseline] (701.039 ms) : 0, 701039
BytebuddyAgent [candidate] (699.906 ms) : 0, 699906
GlobalTracer [baseline] (237.074 ms) : 0, 237074
GlobalTracer [candidate] (236.99 ms) : 0, 236990
AppSec [baseline] (175.567 ms) : 0, 175567
AppSec [candidate] (175.317 ms) : 0, 175317
Debugger [baseline] (5.934 ms) : 0, 5934
Debugger [candidate] (5.938 ms) : 0, 5938
Remote Config [baseline] (620.032 µs) : 0, 620
Remote Config [candidate] (616.81 µs) : 0, 617
Telemetry [baseline] (7.748 ms) : 0, 7748
Telemetry [candidate] (7.379 ms) : 0, 7379
IAST [baseline] (21.795 ms) : 0, 21795
IAST [candidate] (21.673 ms) : 0, 21673
section iast
BytebuddyAgent [baseline] (810.457 ms) : 0, 810457
BytebuddyAgent [candidate] (806.563 ms) : 0, 806563
GlobalTracer [baseline] (232.481 ms) : 0, 232481
GlobalTracer [candidate] (230.686 ms) : 0, 230686
AppSec [baseline] (49.811 ms) : 0, 49811
AppSec [candidate] (51.921 ms) : 0, 51921
Debugger [baseline] (5.941 ms) : 0, 5941
Debugger [candidate] (5.92 ms) : 0, 5920
Remote Config [baseline] (596.171 µs) : 0, 596
Remote Config [candidate] (582.325 µs) : 0, 582
Telemetry [baseline] (7.942 ms) : 0, 7942
Telemetry [candidate] (7.933 ms) : 0, 7933
IAST [baseline] (29.223 ms) : 0, 29223
IAST [candidate] (26.949 ms) : 0, 26949
section profiling
ProfilingAgent [baseline] (104.902 ms) : 0, 104902
ProfilingAgent [candidate] (103.638 ms) : 0, 103638
BytebuddyAgent [baseline] (680.914 ms) : 0, 680914
BytebuddyAgent [candidate] (683.332 ms) : 0, 683332
GlobalTracer [baseline] (378.274 ms) : 0, 378274
GlobalTracer [candidate] (378.847 ms) : 0, 378847
AppSec [baseline] (62.598 ms) : 0, 62598
AppSec [candidate] (62.29 ms) : 0, 62290
Debugger [baseline] (6.362 ms) : 0, 6362
Debugger [candidate] (6.318 ms) : 0, 6318
Remote Config [baseline] (679.628 µs) : 0, 680
Remote Config [candidate] (652.537 µs) : 0, 653
Telemetry [baseline] (8.344 ms) : 0, 8344
Telemetry [candidate] (8.258 ms) : 0, 8258
Profiling [baseline] (104.927 ms) : 0, 104927
Profiling [candidate] (103.663 ms) : 0, 103663
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 18 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section baseline
no_agent (1.355 ms) : 1335, 1374
. : milestone, 1355,
appsec (1.731 ms) : 1709, 1754
. : milestone, 1731,
appsec_no_iast (1.737 ms) : 1714, 1760
. : milestone, 1737,
code_origins (1.698 ms) : 1669, 1726
. : milestone, 1698,
iast (1.52 ms) : 1496, 1544
. : milestone, 1520,
profiling (1.566 ms) : 1541, 1590
. : milestone, 1566,
tracing (1.476 ms) : 1451, 1501
. : milestone, 1476,
section candidate
no_agent (1.38 ms) : 1359, 1401
. : milestone, 1380,
appsec (1.735 ms) : 1711, 1758
. : milestone, 1735,
appsec_no_iast (1.734 ms) : 1710, 1758
. : milestone, 1734,
code_origins (1.676 ms) : 1649, 1703
. : milestone, 1676,
iast (1.523 ms) : 1498, 1548
. : milestone, 1523,
profiling (1.548 ms) : 1523, 1572
. : milestone, 1548,
tracing (1.498 ms) : 1473, 1523
. : milestone, 1498,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section baseline
no_agent (382.146 µs) : 361, 403
. : milestone, 382,
iast (516.477 µs) : 495, 538
. : milestone, 516,
iast_FULL (744.94 µs) : 723, 767
. : milestone, 745,
iast_GLOBAL (563.594 µs) : 542, 585
. : milestone, 564,
iast_HARDCODED_SECRET_DISABLED (515.819 µs) : 493, 538
. : milestone, 516,
iast_INACTIVE (466.493 µs) : 444, 489
. : milestone, 466,
iast_TELEMETRY_OFF (511.414 µs) : 489, 534
. : milestone, 511,
tracing (464.041 µs) : 442, 486
. : milestone, 464,
section candidate
no_agent (379.382 µs) : 359, 399
. : milestone, 379,
iast (522.17 µs) : 500, 544
. : milestone, 522,
iast_FULL (744.052 µs) : 722, 766
. : milestone, 744,
iast_GLOBAL (579.029 µs) : 556, 602
. : milestone, 579,
iast_HARDCODED_SECRET_DISABLED (527.254 µs) : 504, 551
. : milestone, 527,
iast_INACTIVE (470.288 µs) : 448, 493
. : milestone, 470,
iast_TELEMETRY_OFF (514.575 µs) : 492, 538
. : milestone, 515,
tracing (468.994 µs) : 446, 492
. : milestone, 469,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section baseline
no_agent (1.479 ms) : 1468, 1491
. : milestone, 1479,
appsec (2.421 ms) : 2371, 2471
. : milestone, 2421,
iast (2.191 ms) : 2129, 2254
. : milestone, 2191,
iast_GLOBAL (2.242 ms) : 2179, 2304
. : milestone, 2242,
profiling (2.51 ms) : 2272, 2747
. : milestone, 2510,
tracing (2.019 ms) : 1970, 2067
. : milestone, 2019,
section candidate
no_agent (1.481 ms) : 1470, 1493
. : milestone, 1481,
appsec (2.414 ms) : 2365, 2464
. : milestone, 2414,
iast (2.201 ms) : 2138, 2263
. : milestone, 2201,
iast_GLOBAL (2.234 ms) : 2171, 2296
. : milestone, 2234,
profiling (2.05 ms) : 1999, 2101
. : milestone, 2050,
tracing (2.013 ms) : 1965, 2061
. : milestone, 2013,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~ad0c438cbd, baseline=1.50.0-SNAPSHOT~45c7667bff
dateFormat X
axisFormat %s
section baseline
no_agent (15.485 s) : 15485000, 15485000
. : milestone, 15485000,
appsec (14.91 s) : 14910000, 14910000
. : milestone, 14910000,
iast (18.214 s) : 18214000, 18214000
. : milestone, 18214000,
iast_GLOBAL (18.25 s) : 18250000, 18250000
. : milestone, 18250000,
profiling (14.817 s) : 14817000, 14817000
. : milestone, 14817000,
tracing (15.003 s) : 15003000, 15003000
. : milestone, 15003000,
section candidate
no_agent (15.273 s) : 15273000, 15273000
. : milestone, 15273000,
appsec (15.032 s) : 15032000, 15032000
. : milestone, 15032000,
iast (18.766 s) : 18766000, 18766000
. : milestone, 18766000,
iast_GLOBAL (18.044 s) : 18044000, 18044000
. : milestone, 18044000,
profiling (15.054 s) : 15054000, 15054000
. : milestone, 15054000,
tracing (14.864 s) : 14864000, 14864000
. : milestone, 14864000,
|
Context.current()
during injection
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the refactoring!
...jax-rs-client-2.0/src/main/java/datadog/trace/instrumentation/jaxrs/ClientTracingFilter.java
Outdated
Show resolved
Hide resolved
I would recommend using a "meaningful" title for the PR if it is not hidden from the release notes. |
Context.current()
during injection
What Does This Do
Migrates all Http Client Instrumentations to inject full context through the use of
Context.current()
rather than only injecting span contexts. This also leads to support of W3C Baggage injection for all Http Client instrumentations.Motivation
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]