-
Notifications
You must be signed in to change notification settings - Fork 292
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
avoid emission of endpoint events for client and producer root spans #7732
Conversation
c5c1bd2
to
63319bf
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 12 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.071 s) : 0, 1070975
Total [baseline] (10.424 s) : 0, 10423551
Agent [candidate] (1.083 s) : 0, 1082737
Total [candidate] (10.446 s) : 0, 10446126
section appsec
Agent [baseline] (1.207 s) : 0, 1207059
Total [baseline] (10.615 s) : 0, 10614992
Agent [candidate] (1.208 s) : 0, 1208120
Total [candidate] (10.715 s) : 0, 10714811
section iast
Agent [baseline] (1.198 s) : 0, 1198420
Total [baseline] (10.919 s) : 0, 10919032
Agent [candidate] (1.205 s) : 0, 1204939
Total [candidate] (10.873 s) : 0, 10872812
section profiling
Agent [baseline] (1.269 s) : 0, 1269265
Total [baseline] (10.621 s) : 0, 10621402
Agent [candidate] (1.27 s) : 0, 1269684
Total [candidate] (10.617 s) : 0, 10616863
gantt
title petclinic - break down per module: candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.063 ms) : 0, 683063
BytebuddyAgent [candidate] (691.588 ms) : 0, 691588
GlobalTracer [baseline] (311.814 ms) : 0, 311814
GlobalTracer [candidate] (314.512 ms) : 0, 314512
AppSec [baseline] (53.994 ms) : 0, 53994
AppSec [candidate] (54.333 ms) : 0, 54333
Remote Config [baseline] (654.461 µs) : 0, 654
Remote Config [candidate] (659.524 µs) : 0, 660
Telemetry [baseline] (7.683 ms) : 0, 7683
Telemetry [candidate] (7.702 ms) : 0, 7702
section appsec
BytebuddyAgent [baseline] (701.223 ms) : 0, 701223
BytebuddyAgent [candidate] (702.342 ms) : 0, 702342
GlobalTracer [baseline] (309.046 ms) : 0, 309046
GlobalTracer [candidate] (309.302 ms) : 0, 309302
AppSec [baseline] (162.984 ms) : 0, 162984
AppSec [candidate] (162.585 ms) : 0, 162585
Remote Config [baseline] (641.213 µs) : 0, 641
Remote Config [candidate] (647.37 µs) : 0, 647
Telemetry [baseline] (9.633 ms) : 0, 9633
Telemetry [candidate] (8.933 ms) : 0, 8933
IAST [baseline] (19.474 ms) : 0, 19474
IAST [candidate] (20.959 ms) : 0, 20959
section iast
BytebuddyAgent [baseline] (797.272 ms) : 0, 797272
BytebuddyAgent [candidate] (802.417 ms) : 0, 802417
GlobalTracer [baseline] (300.994 ms) : 0, 300994
GlobalTracer [candidate] (302.265 ms) : 0, 302265
AppSec [baseline] (53.385 ms) : 0, 53385
AppSec [candidate] (55.703 ms) : 0, 55703
Remote Config [baseline] (610.435 µs) : 0, 610
Remote Config [candidate] (609.853 µs) : 0, 610
Telemetry [baseline] (7.112 ms) : 0, 7112
Telemetry [candidate] (7.988 ms) : 0, 7988
IAST [baseline] (25.282 ms) : 0, 25282
IAST [candidate] (22.08 ms) : 0, 22080
section profiling
BytebuddyAgent [baseline] (675.329 ms) : 0, 675329
BytebuddyAgent [candidate] (676.601 ms) : 0, 676601
GlobalTracer [baseline] (394.38 ms) : 0, 394380
GlobalTracer [candidate] (394.877 ms) : 0, 394877
AppSec [baseline] (55.193 ms) : 0, 55193
AppSec [candidate] (54.636 ms) : 0, 54636
Remote Config [baseline] (666.574 µs) : 0, 667
Remote Config [candidate] (658.281 µs) : 0, 658
Telemetry [baseline] (7.522 ms) : 0, 7522
Telemetry [candidate] (7.445 ms) : 0, 7445
ProfilingAgent [baseline] (97.488 ms) : 0, 97488
ProfilingAgent [candidate] (96.601 ms) : 0, 96601
Profiling [baseline] (97.512 ms) : 0, 97512
Profiling [candidate] (96.626 ms) : 0, 96626
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1084355
Total [baseline] (8.593 s) : 0, 8592929
Agent [candidate] (1.071 s) : 0, 1070567
Total [candidate] (8.555 s) : 0, 8555094
section iast
Agent [baseline] (1.205 s) : 0, 1205099
Total [baseline] (9.118 s) : 0, 9117607
Agent [candidate] (1.199 s) : 0, 1198531
Total [candidate] (9.108 s) : 0, 9107965
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.207 s) : 0, 1206848
Total [baseline] (9.096 s) : 0, 9096258
Agent [candidate] (1.198 s) : 0, 1198383
Total [candidate] (9.12 s) : 0, 9120364
section iast_TELEMETRY_OFF
Agent [baseline] (1.203 s) : 0, 1202686
Total [baseline] (9.133 s) : 0, 9133267
Agent [candidate] (1.196 s) : 0, 1195730
Total [candidate] (9.106 s) : 0, 9105976
gantt
title insecure-bank - break down per module: candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (693.204 ms) : 0, 693204
BytebuddyAgent [candidate] (682.459 ms) : 0, 682459
GlobalTracer [baseline] (314.555 ms) : 0, 314555
GlobalTracer [candidate] (311.803 ms) : 0, 311803
AppSec [baseline] (54.173 ms) : 0, 54173
AppSec [candidate] (54.187 ms) : 0, 54187
Remote Config [baseline] (669.852 µs) : 0, 670
Remote Config [candidate] (663.328 µs) : 0, 663
Telemetry [baseline] (7.771 ms) : 0, 7771
Telemetry [candidate] (7.656 ms) : 0, 7656
section iast
BytebuddyAgent [baseline] (802.533 ms) : 0, 802533
BytebuddyAgent [candidate] (797.761 ms) : 0, 797761
GlobalTracer [baseline] (302.554 ms) : 0, 302554
GlobalTracer [candidate] (300.491 ms) : 0, 300491
AppSec [baseline] (54.922 ms) : 0, 54922
AppSec [candidate] (53.493 ms) : 0, 53493
Remote Config [baseline] (597.603 µs) : 0, 598
Remote Config [candidate] (612.982 µs) : 0, 613
Telemetry [baseline] (7.064 ms) : 0, 7064
Telemetry [candidate] (7.083 ms) : 0, 7083
IAST [baseline] (23.529 ms) : 0, 23529
IAST [candidate] (25.222 ms) : 0, 25222
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (803.518 ms) : 0, 803518
BytebuddyAgent [candidate] (797.052 ms) : 0, 797052
GlobalTracer [baseline] (302.418 ms) : 0, 302418
GlobalTracer [candidate] (300.887 ms) : 0, 300887
AppSec [baseline] (56.088 ms) : 0, 56088
AppSec [candidate] (54.968 ms) : 0, 54968
Remote Config [baseline] (613.694 µs) : 0, 614
Remote Config [candidate] (619.397 µs) : 0, 619
Telemetry [baseline] (7.178 ms) : 0, 7178
Telemetry [candidate] (7.109 ms) : 0, 7109
IAST [baseline] (23.077 ms) : 0, 23077
IAST [candidate] (23.868 ms) : 0, 23868
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.941 ms) : 0, 799941
BytebuddyAgent [candidate] (795.297 ms) : 0, 795297
GlobalTracer [baseline] (302.022 ms) : 0, 302022
GlobalTracer [candidate] (300.616 ms) : 0, 300616
AppSec [baseline] (53.746 ms) : 0, 53746
AppSec [candidate] (55.711 ms) : 0, 55711
Remote Config [baseline] (692.646 µs) : 0, 693
Remote Config [candidate] (658.921 µs) : 0, 659
Telemetry [baseline] (7.064 ms) : 0, 7064
Telemetry [candidate] (7.009 ms) : 0, 7009
IAST [baseline] (25.276 ms) : 0, 25276
IAST [candidate] (22.56 ms) : 0, 22560
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section baseline
no_agent (1.32 ms) : 1301, 1339
. : milestone, 1320,
appsec (1.703 ms) : 1679, 1728
. : milestone, 1703,
appsec_no_iast (1.711 ms) : 1687, 1735
. : milestone, 1711,
iast (1.483 ms) : 1461, 1505
. : milestone, 1483,
profiling (1.469 ms) : 1444, 1493
. : milestone, 1469,
tracing (1.482 ms) : 1458, 1505
. : milestone, 1482,
section candidate
no_agent (1.341 ms) : 1321, 1360
. : milestone, 1341,
appsec (1.716 ms) : 1693, 1739
. : milestone, 1716,
appsec_no_iast (1.722 ms) : 1697, 1746
. : milestone, 1722,
iast (1.472 ms) : 1450, 1494
. : milestone, 1472,
profiling (1.477 ms) : 1453, 1502
. : milestone, 1477,
tracing (1.476 ms) : 1451, 1500
. : milestone, 1476,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section baseline
no_agent (372.888 µs) : 353, 393
. : milestone, 373,
iast (480.338 µs) : 459, 502
. : milestone, 480,
iast_FULL (550.644 µs) : 529, 572
. : milestone, 551,
iast_GLOBAL (506.65 µs) : 486, 528
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (476.127 µs) : 455, 497
. : milestone, 476,
iast_INACTIVE (455.718 µs) : 434, 477
. : milestone, 456,
iast_TELEMETRY_OFF (471.749 µs) : 450, 493
. : milestone, 472,
tracing (444.019 µs) : 423, 465
. : milestone, 444,
section candidate
no_agent (373.03 µs) : 354, 393
. : milestone, 373,
iast (479.563 µs) : 458, 501
. : milestone, 480,
iast_FULL (547.637 µs) : 526, 569
. : milestone, 548,
iast_GLOBAL (523.493 µs) : 500, 547
. : milestone, 523,
iast_HARDCODED_SECRET_DISABLED (479.924 µs) : 459, 501
. : milestone, 480,
iast_INACTIVE (445.508 µs) : 425, 466
. : milestone, 446,
iast_TELEMETRY_OFF (471.56 µs) : 450, 493
. : milestone, 472,
tracing (440.357 µs) : 420, 461
. : milestone, 440,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1458, 1482
. : milestone, 1470,
appsec (2.322 ms) : 2282, 2363
. : milestone, 2322,
iast (2.078 ms) : 2026, 2129
. : milestone, 2078,
iast_GLOBAL (2.109 ms) : 2058, 2160
. : milestone, 2109,
profiling (1.95 ms) : 1908, 1992
. : milestone, 1950,
tracing (1.913 ms) : 1874, 1952
. : milestone, 1913,
section candidate
no_agent (1.472 ms) : 1460, 1483
. : milestone, 1472,
appsec (2.333 ms) : 2292, 2373
. : milestone, 2333,
iast (2.067 ms) : 2015, 2118
. : milestone, 2067,
iast_GLOBAL (2.122 ms) : 2071, 2174
. : milestone, 2122,
profiling (1.948 ms) : 1907, 1990
. : milestone, 1948,
tracing (1.917 ms) : 1878, 1957
. : milestone, 1917,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~63319bfc15, baseline=1.41.0-SNAPSHOT~9db260fd16
dateFormat X
axisFormat %s
section baseline
no_agent (15.134 s) : 15134000, 15134000
. : milestone, 15134000,
appsec (15.398 s) : 15398000, 15398000
. : milestone, 15398000,
iast (18.673 s) : 18673000, 18673000
. : milestone, 18673000,
iast_GLOBAL (18.129 s) : 18129000, 18129000
. : milestone, 18129000,
profiling (15.247 s) : 15247000, 15247000
. : milestone, 15247000,
tracing (15.275 s) : 15275000, 15275000
. : milestone, 15275000,
section candidate
no_agent (15.187 s) : 15187000, 15187000
. : milestone, 15187000,
appsec (15.333 s) : 15333000, 15333000
. : milestone, 15333000,
iast (18.871 s) : 18871000, 18871000
. : milestone, 18871000,
iast_GLOBAL (17.805 s) : 17805000, 17805000
. : milestone, 17805000,
profiling (15.84 s) : 15840000, 15840000
. : milestone, 15840000,
tracing (15.408 s) : 15408000, 15408000
. : milestone, 15408000,
|
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.
LGTM
What Does This Do
We emit endpoint events for any local root span, but the local root span isn't necessarily associated with an endpoint. If tracing context propagation breaks for some reason, or if instrumented client code is otherwise invoked in the absence of a trace, we can end up with the profiler reporting things like SQL queries as "endpoints." These aren't very useful for analysis, and can also mean using a lot of native memory to store them in a constant pool, so this change explicitly prevents endpoint profiling when the root span is a client or producer span.
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: PROF-10666