-
Notifications
You must be signed in to change notification settings - Fork 322
Reorganise internal instrumentation to comply with the renaming policy #10280
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
Conversation
| return@childLoop | ||
| } | ||
| // skip the special datadog top level instrumentation directory | ||
| if (parentName == null && moduleName == "datadog") { |
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.
suggestion: Maybe declare the excluded module name in the plugin extension. This can be done in a follow-up PR.
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.
the exclusion list works for a single module but not for all its childs. It can be expanded to support this. For this reason I kept it simple and hardcoded in this way. That said this should be the only exception of this kind
Kafka / producer-benchmarkParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 3 metrics, 0 unstable metrics. See unchanged results
|
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 12 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.087 s) : 0, 1087008
Total [baseline] (8.732 s) : 0, 8731930
Agent [candidate] (1.092 s) : 0, 1091757
Total [candidate] (8.772 s) : 0, 8772152
section iast
Agent [baseline] (1.227 s) : 0, 1226738
Total [baseline] (9.339 s) : 0, 9339383
Agent [candidate] (1.225 s) : 0, 1224671
Total [candidate] (9.308 s) : 0, 9308245
gantt
title insecure-bank - break down per module: candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.203 ms) : 0, 1203
crashtracking [candidate] (1.215 ms) : 0, 1215
BytebuddyAgent [baseline] (653.23 ms) : 0, 653230
BytebuddyAgent [candidate] (656.076 ms) : 0, 656076
GlobalTracer [baseline] (283.37 ms) : 0, 283370
GlobalTracer [candidate] (284.222 ms) : 0, 284222
AppSec [baseline] (32.555 ms) : 0, 32555
AppSec [candidate] (32.886 ms) : 0, 32886
Debugger [baseline] (67.651 ms) : 0, 67651
Debugger [candidate] (67.967 ms) : 0, 67967
Remote Config [baseline] (633.242 µs) : 0, 633
Remote Config [candidate] (627.39 µs) : 0, 627
Telemetry [baseline] (9.017 ms) : 0, 9017
Telemetry [candidate] (9.089 ms) : 0, 9089
Flare Poller [baseline] (3.731 ms) : 0, 3731
Flare Poller [candidate] (3.803 ms) : 0, 3803
section iast
crashtracking [baseline] (1.2 ms) : 0, 1200
crashtracking [candidate] (1.194 ms) : 0, 1194
BytebuddyAgent [baseline] (794.302 ms) : 0, 794302
BytebuddyAgent [candidate] (792.714 ms) : 0, 792714
GlobalTracer [baseline] (255.623 ms) : 0, 255623
GlobalTracer [candidate] (255.819 ms) : 0, 255819
IAST [baseline] (27.182 ms) : 0, 27182
IAST [candidate] (27.039 ms) : 0, 27039
AppSec [baseline] (32.655 ms) : 0, 32655
AppSec [candidate] (33.614 ms) : 0, 33614
Debugger [baseline] (67.483 ms) : 0, 67483
Debugger [candidate] (66.171 ms) : 0, 66171
Remote Config [baseline] (575.971 µs) : 0, 576
Remote Config [candidate] (571.907 µs) : 0, 572
Telemetry [baseline] (8.609 ms) : 0, 8609
Telemetry [candidate] (8.53 ms) : 0, 8530
Flare Poller [baseline] (3.543 ms) : 0, 3543
Flare Poller [candidate] (3.493 ms) : 0, 3493
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.093 s) : 0, 1092808
Total [baseline] (10.836 s) : 0, 10835887
Agent [candidate] (1.091 s) : 0, 1091210
Total [candidate] (10.789 s) : 0, 10789107
section appsec
Agent [baseline] (1.272 s) : 0, 1271520
Total [baseline] (10.982 s) : 0, 10981633
Agent [candidate] (1.276 s) : 0, 1275589
Total [candidate] (10.901 s) : 0, 10901405
section iast
Agent [baseline] (1.224 s) : 0, 1224205
Total [baseline] (11.11 s) : 0, 11109948
Agent [candidate] (1.226 s) : 0, 1226265
Total [candidate] (11.182 s) : 0, 11181522
section profiling
Agent [baseline] (1.206 s) : 0, 1206242
Total [baseline] (10.917 s) : 0, 10917015
Agent [candidate] (1.203 s) : 0, 1202853
Total [candidate] (10.872 s) : 0, 10872447
gantt
title petclinic - break down per module: candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.193 ms) : 0, 1193
crashtracking [candidate] (1.195 ms) : 0, 1195
BytebuddyAgent [baseline] (656.178 ms) : 0, 656178
BytebuddyAgent [candidate] (654.539 ms) : 0, 654539
GlobalTracer [baseline] (284.597 ms) : 0, 284597
GlobalTracer [candidate] (284.159 ms) : 0, 284159
AppSec [baseline] (32.893 ms) : 0, 32893
AppSec [candidate] (32.927 ms) : 0, 32927
Debugger [baseline] (68.076 ms) : 0, 68076
Debugger [candidate] (69.187 ms) : 0, 69187
Remote Config [baseline] (634.025 µs) : 0, 634
Remote Config [candidate] (639.269 µs) : 0, 639
Telemetry [baseline] (9.695 ms) : 0, 9695
Telemetry [candidate] (9.022 ms) : 0, 9022
Flare Poller [baseline] (3.767 ms) : 0, 3767
Flare Poller [candidate] (3.785 ms) : 0, 3785
section appsec
crashtracking [baseline] (1.189 ms) : 0, 1189
crashtracking [candidate] (1.224 ms) : 0, 1224
BytebuddyAgent [baseline] (693.477 ms) : 0, 693477
BytebuddyAgent [candidate] (698.196 ms) : 0, 698196
GlobalTracer [baseline] (259.963 ms) : 0, 259963
GlobalTracer [candidate] (260.563 ms) : 0, 260563
IAST [baseline] (24.936 ms) : 0, 24936
IAST [candidate] (24.812 ms) : 0, 24812
AppSec [baseline] (175.76 ms) : 0, 175760
AppSec [candidate] (175.218 ms) : 0, 175218
Debugger [baseline] (66.67 ms) : 0, 66670
Debugger [candidate] (66.391 ms) : 0, 66391
Remote Config [baseline] (795.48 µs) : 0, 795
Remote Config [candidate] (748.647 µs) : 0, 749
Telemetry [baseline] (9.547 ms) : 0, 9547
Telemetry [candidate] (9.236 ms) : 0, 9236
Flare Poller [baseline] (3.735 ms) : 0, 3735
Flare Poller [candidate] (3.599 ms) : 0, 3599
section iast
crashtracking [baseline] (1.18 ms) : 0, 1180
crashtracking [candidate] (1.187 ms) : 0, 1187
BytebuddyAgent [baseline] (791.667 ms) : 0, 791667
BytebuddyAgent [candidate] (792.886 ms) : 0, 792886
GlobalTracer [baseline] (255.5 ms) : 0, 255500
GlobalTracer [candidate] (255.919 ms) : 0, 255919
IAST [baseline] (27.2 ms) : 0, 27200
IAST [candidate] (26.839 ms) : 0, 26839
AppSec [baseline] (32.998 ms) : 0, 32998
AppSec [candidate] (33.977 ms) : 0, 33977
Debugger [baseline] (67.489 ms) : 0, 67489
Debugger [candidate] (67.177 ms) : 0, 67177
Remote Config [baseline] (577.574 µs) : 0, 578
Remote Config [candidate] (580.336 µs) : 0, 580
Telemetry [baseline] (8.549 ms) : 0, 8549
Telemetry [candidate] (8.584 ms) : 0, 8584
Flare Poller [baseline] (3.504 ms) : 0, 3504
Flare Poller [candidate] (3.52 ms) : 0, 3520
section profiling
crashtracking [baseline] (1.225 ms) : 0, 1225
crashtracking [candidate] (1.216 ms) : 0, 1216
BytebuddyAgent [baseline] (704.577 ms) : 0, 704577
BytebuddyAgent [candidate] (702.139 ms) : 0, 702139
GlobalTracer [baseline] (220.679 ms) : 0, 220679
GlobalTracer [candidate] (219.846 ms) : 0, 219846
AppSec [baseline] (32.138 ms) : 0, 32138
AppSec [candidate] (32.002 ms) : 0, 32002
Debugger [baseline] (67.887 ms) : 0, 67887
Debugger [candidate] (67.786 ms) : 0, 67786
Remote Config [baseline] (662.146 µs) : 0, 662
Remote Config [candidate] (622.948 µs) : 0, 623
Telemetry [baseline] (8.666 ms) : 0, 8666
Telemetry [candidate] (8.827 ms) : 0, 8827
Flare Poller [baseline] (3.666 ms) : 0, 3666
Flare Poller [candidate] (3.666 ms) : 0, 3666
ProfilingAgent [baseline] (96.871 ms) : 0, 96871
ProfilingAgent [candidate] (96.978 ms) : 0, 96978
Profiling [baseline] (97.45 ms) : 0, 97450
Profiling [candidate] (97.546 ms) : 0, 97546
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 20 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section baseline
no_agent (1.209 ms) : 1196, 1221
. : milestone, 1209,
iast (3.195 ms) : 3154, 3237
. : milestone, 3195,
iast_FULL (5.839 ms) : 5781, 5897
. : milestone, 5839,
iast_GLOBAL (3.552 ms) : 3496, 3609
. : milestone, 3552,
profiling (1.985 ms) : 1967, 2002
. : milestone, 1985,
tracing (1.852 ms) : 1836, 1868
. : milestone, 1852,
section candidate
no_agent (1.187 ms) : 1176, 1199
. : milestone, 1187,
iast (3.107 ms) : 3072, 3142
. : milestone, 3107,
iast_FULL (5.614 ms) : 5559, 5669
. : milestone, 5614,
iast_GLOBAL (3.62 ms) : 3563, 3678
. : milestone, 3620,
profiling (1.989 ms) : 1971, 2006
. : milestone, 1989,
tracing (1.818 ms) : 1803, 1833
. : milestone, 1818,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section baseline
no_agent (17.992 ms) : 17808, 18176
. : milestone, 17992,
appsec (18.708 ms) : 18514, 18902
. : milestone, 18708,
code_origins (17.974 ms) : 17793, 18154
. : milestone, 17974,
iast (17.951 ms) : 17769, 18133
. : milestone, 17951,
profiling (18.873 ms) : 18687, 19060
. : milestone, 18873,
tracing (17.735 ms) : 17558, 17911
. : milestone, 17735,
section candidate
no_agent (17.76 ms) : 17582, 17939
. : milestone, 17760,
appsec (18.486 ms) : 18298, 18673
. : milestone, 18486,
code_origins (17.768 ms) : 17593, 17944
. : milestone, 17768,
iast (17.763 ms) : 17585, 17940
. : milestone, 17763,
profiling (18.903 ms) : 18710, 19095
. : milestone, 18903,
tracing (17.858 ms) : 17678, 18039
. : milestone, 17858,
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.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section baseline
no_agent (14.993 s) : 14993000, 14993000
. : milestone, 14993000,
appsec (14.756 s) : 14756000, 14756000
. : milestone, 14756000,
iast (18.73 s) : 18730000, 18730000
. : milestone, 18730000,
iast_GLOBAL (18.006 s) : 18006000, 18006000
. : milestone, 18006000,
profiling (15.212 s) : 15212000, 15212000
. : milestone, 15212000,
tracing (14.46 s) : 14460000, 14460000
. : milestone, 14460000,
section candidate
no_agent (15.7 s) : 15700000, 15700000
. : milestone, 15700000,
appsec (14.769 s) : 14769000, 14769000
. : milestone, 14769000,
iast (18.289 s) : 18289000, 18289000
. : milestone, 18289000,
iast_GLOBAL (17.793 s) : 17793000, 17793000
. : milestone, 17793000,
profiling (14.846 s) : 14846000, 14846000
. : milestone, 14846000,
tracing (14.714 s) : 14714000, 14714000
. : milestone, 14714000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.58.0-SNAPSHOT~efb10497fc, baseline=1.58.0-SNAPSHOT~2a419dbac3
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1457, 1480
. : milestone, 1469,
appsec (2.506 ms) : 2451, 2561
. : milestone, 2506,
iast (2.197 ms) : 2133, 2261
. : milestone, 2197,
iast_GLOBAL (2.243 ms) : 2179, 2308
. : milestone, 2243,
profiling (2.045 ms) : 1993, 2098
. : milestone, 2045,
tracing (2.038 ms) : 1987, 2089
. : milestone, 2038,
section candidate
no_agent (1.47 ms) : 1458, 1481
. : milestone, 1470,
appsec (3.714 ms) : 3494, 3933
. : milestone, 3714,
iast (2.201 ms) : 2137, 2266
. : milestone, 2201,
iast_GLOBAL (2.238 ms) : 2174, 2302
. : milestone, 2238,
profiling (2.059 ms) : 2007, 2112
. : milestone, 2059,
tracing (2.031 ms) : 1981, 2082
. : milestone, 2031,
|
Kafka / consumer-benchmarkParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 3 metrics, 0 unstable metrics. See unchanged results
|
What Does This Do
Finish the refactoring of instrumentation modules by putting internal modules under datadog top folder.
This folder is not checked by the
instrumentationNamingPluginThe instrumentation under this folder are classified by product:
The codeowner has been updated
Motivation
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]