-
Notifications
You must be signed in to change notification settings - Fork 279
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
Fix Gradle v8.9 instrumentation #7319
Merged
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/fix-gradle-8.9-instrumentation
Jul 12, 2024
Merged
Fix Gradle v8.9 instrumentation #7319
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/fix-gradle-8.9-instrumentation
Jul 12, 2024
Conversation
This file contains 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
nikita-tkachenko-datadog
added
type: bug
comp: ci visibility
Continuous Integration Visibility
labels
Jul 12, 2024
PerfectSlayer
approved these changes
Jul 12, 2024
juan-fernandez
approved these changes
Jul 12, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 10 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.065 s) : 0, 1065109
Total [baseline] (8.513 s) : 0, 8512536
Agent [candidate] (1.065 s) : 0, 1064878
Total [candidate] (8.509 s) : 0, 8509453
section iast
Agent [baseline] (1.172 s) : 0, 1171859
Total [baseline] (8.971 s) : 0, 8970753
Agent [candidate] (1.191 s) : 0, 1190823
Total [candidate] (9.023 s) : 0, 9022645
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.173 s) : 0, 1172676
Total [baseline] (8.916 s) : 0, 8916087
Agent [candidate] (1.17 s) : 0, 1169711
Total [candidate] (8.96 s) : 0, 8960472
section iast_TELEMETRY_OFF
Agent [baseline] (1.176 s) : 0, 1176160
Total [baseline] (8.98 s) : 0, 8979806
Agent [candidate] (1.168 s) : 0, 1168387
Total [candidate] (8.964 s) : 0, 8964478
gantt
title insecure-bank - break down per module: candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.418 ms) : 0, 666418
BytebuddyAgent [candidate] (666.148 ms) : 0, 666148
GlobalTracer [baseline] (305.475 ms) : 0, 305475
GlobalTracer [candidate] (305.727 ms) : 0, 305727
AppSec [baseline] (50.338 ms) : 0, 50338
AppSec [candidate] (50.09 ms) : 0, 50090
Remote Config [baseline] (670.83 µs) : 0, 671
Remote Config [candidate] (675.15 µs) : 0, 675
Telemetry [baseline] (7.652 ms) : 0, 7652
Telemetry [candidate] (7.628 ms) : 0, 7628
section iast
BytebuddyAgent [baseline] (781.103 ms) : 0, 781103
BytebuddyAgent [candidate] (796.588 ms) : 0, 796588
GlobalTracer [baseline] (295.38 ms) : 0, 295380
GlobalTracer [candidate] (299.5 ms) : 0, 299500
AppSec [baseline] (48.065 ms) : 0, 48065
AppSec [candidate] (49.376 ms) : 0, 49376
IAST [baseline] (26.247 ms) : 0, 26247
IAST [candidate] (24.005 ms) : 0, 24005
Remote Config [baseline] (586.284 µs) : 0, 586
Remote Config [candidate] (578.87 µs) : 0, 579
Telemetry [baseline] (7.009 ms) : 0, 7009
Telemetry [candidate] (7.032 ms) : 0, 7032
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (780.702 ms) : 0, 780702
BytebuddyAgent [candidate] (779.884 ms) : 0, 779884
GlobalTracer [baseline] (295.616 ms) : 0, 295616
GlobalTracer [candidate] (295.197 ms) : 0, 295197
AppSec [baseline] (48.998 ms) : 0, 48998
AppSec [candidate] (48.027 ms) : 0, 48027
IAST [baseline] (26.317 ms) : 0, 26317
IAST [candidate] (25.497 ms) : 0, 25497
Remote Config [baseline] (559.481 µs) : 0, 559
Remote Config [candidate] (568.301 µs) : 0, 568
Telemetry [baseline] (6.977 ms) : 0, 6977
Telemetry [candidate] (7.013 ms) : 0, 7013
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (782.905 ms) : 0, 782905
BytebuddyAgent [candidate] (778.425 ms) : 0, 778425
GlobalTracer [baseline] (296.823 ms) : 0, 296823
GlobalTracer [candidate] (294.808 ms) : 0, 294808
AppSec [baseline] (47.189 ms) : 0, 47189
AppSec [candidate] (47.159 ms) : 0, 47159
IAST [baseline] (27.504 ms) : 0, 27504
IAST [candidate] (27.064 ms) : 0, 27064
Remote Config [baseline] (561.376 µs) : 0, 561
Remote Config [candidate] (572.19 µs) : 0, 572
Telemetry [baseline] (7.558 ms) : 0, 7558
Telemetry [candidate] (6.803 ms) : 0, 6803
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.073 s) : 0, 1072517
Total [baseline] (10.35 s) : 0, 10350170
Agent [candidate] (1.071 s) : 0, 1070819
Total [candidate] (10.327 s) : 0, 10327440
section appsec
Agent [baseline] (1.184 s) : 0, 1184491
Total [baseline] (10.487 s) : 0, 10486838
Agent [candidate] (1.192 s) : 0, 1192371
Total [candidate] (10.511 s) : 0, 10511040
section iast
Agent [baseline] (1.172 s) : 0, 1171837
Total [baseline] (10.698 s) : 0, 10697554
Agent [candidate] (1.183 s) : 0, 1183281
Total [candidate] (10.722 s) : 0, 10721586
section profiling
Agent [baseline] (1.281 s) : 0, 1281351
Total [baseline] (10.594 s) : 0, 10593798
Agent [candidate] (1.266 s) : 0, 1265851
Total [candidate] (10.584 s) : 0, 10584357
gantt
title petclinic - break down per module: candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (671.451 ms) : 0, 671451
BytebuddyAgent [candidate] (671.67 ms) : 0, 671670
GlobalTracer [baseline] (307.508 ms) : 0, 307508
GlobalTracer [candidate] (305.935 ms) : 0, 305935
AppSec [baseline] (50.344 ms) : 0, 50344
AppSec [candidate] (49.923 ms) : 0, 49923
Remote Config [baseline] (683.408 µs) : 0, 683
Remote Config [candidate] (669.186 µs) : 0, 669
Telemetry [baseline] (7.651 ms) : 0, 7651
Telemetry [candidate] (7.607 ms) : 0, 7607
section appsec
BytebuddyAgent [baseline] (677.516 ms) : 0, 677516
BytebuddyAgent [candidate] (682.038 ms) : 0, 682038
GlobalTracer [baseline] (299.81 ms) : 0, 299810
GlobalTracer [candidate] (300.916 ms) : 0, 300916
AppSec [baseline] (153.633 ms) : 0, 153633
AppSec [candidate] (154.191 ms) : 0, 154191
IAST [baseline] (21.645 ms) : 0, 21645
IAST [candidate] (22.904 ms) : 0, 22904
Remote Config [baseline] (621.487 µs) : 0, 621
Remote Config [candidate] (622.863 µs) : 0, 623
Telemetry [baseline] (7.365 ms) : 0, 7365
Telemetry [candidate] (8.273 ms) : 0, 8273
section iast
BytebuddyAgent [baseline] (780.58 ms) : 0, 780580
BytebuddyAgent [candidate] (787.407 ms) : 0, 787407
GlobalTracer [baseline] (295.247 ms) : 0, 295247
GlobalTracer [candidate] (297.89 ms) : 0, 297890
AppSec [baseline] (48.655 ms) : 0, 48655
AppSec [candidate] (47.686 ms) : 0, 47686
IAST [baseline] (26.239 ms) : 0, 26239
IAST [candidate] (26.621 ms) : 0, 26621
Remote Config [baseline] (579.288 µs) : 0, 579
Remote Config [candidate] (582.792 µs) : 0, 583
Telemetry [baseline] (7.007 ms) : 0, 7007
Telemetry [candidate] (9.411 ms) : 0, 9411
section profiling
BytebuddyAgent [baseline] (672.324 ms) : 0, 672324
BytebuddyAgent [candidate] (663.721 ms) : 0, 663721
GlobalTracer [baseline] (392.974 ms) : 0, 392974
GlobalTracer [candidate] (389.047 ms) : 0, 389047
AppSec [baseline] (52.68 ms) : 0, 52680
AppSec [candidate] (51.943 ms) : 0, 51943
Remote Config [baseline] (668.847 µs) : 0, 669
Remote Config [candidate] (669.66 µs) : 0, 670
Telemetry [baseline] (7.39 ms) : 0, 7390
Telemetry [candidate] (7.325 ms) : 0, 7325
ProfilingAgent [baseline] (97.318 ms) : 0, 97318
ProfilingAgent [candidate] (95.873 ms) : 0, 95873
Profiling [baseline] (97.343 ms) : 0, 97343
Profiling [candidate] (95.898 ms) : 0, 95898
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.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section baseline
no_agent (1.343 ms) : 1324, 1362
. : milestone, 1343,
appsec (1.692 ms) : 1669, 1714
. : milestone, 1692,
appsec_no_iast (1.722 ms) : 1699, 1746
. : milestone, 1722,
iast (1.472 ms) : 1450, 1494
. : milestone, 1472,
profiling (1.483 ms) : 1458, 1508
. : milestone, 1483,
tracing (1.448 ms) : 1424, 1473
. : milestone, 1448,
section candidate
no_agent (1.34 ms) : 1320, 1360
. : milestone, 1340,
appsec (1.706 ms) : 1683, 1730
. : milestone, 1706,
appsec_no_iast (1.727 ms) : 1703, 1750
. : milestone, 1727,
iast (1.475 ms) : 1453, 1498
. : milestone, 1475,
profiling (1.492 ms) : 1467, 1516
. : milestone, 1492,
tracing (1.458 ms) : 1434, 1483
. : milestone, 1458,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section baseline
no_agent (363.643 µs) : 344, 383
. : milestone, 364,
iast (486.197 µs) : 464, 508
. : milestone, 486,
iast_FULL (545.073 µs) : 524, 566
. : milestone, 545,
iast_GLOBAL (504.285 µs) : 483, 526
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (473.398 µs) : 452, 494
. : milestone, 473,
iast_INACTIVE (450.801 µs) : 429, 472
. : milestone, 451,
iast_TELEMETRY_OFF (465.001 µs) : 444, 486
. : milestone, 465,
tracing (441.197 µs) : 420, 463
. : milestone, 441,
section candidate
no_agent (371.902 µs) : 352, 392
. : milestone, 372,
iast (476.326 µs) : 455, 497
. : milestone, 476,
iast_FULL (542.473 µs) : 521, 564
. : milestone, 542,
iast_GLOBAL (506.265 µs) : 485, 527
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (486.181 µs) : 464, 508
. : milestone, 486,
iast_INACTIVE (448.911 µs) : 428, 470
. : milestone, 449,
iast_TELEMETRY_OFF (470.028 µs) : 449, 491
. : milestone, 470,
tracing (441.909 µs) : 421, 463
. : milestone, 442,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section baseline
no_agent (15.277 s) : 15277000, 15277000
. : milestone, 15277000,
appsec (15.197 s) : 15197000, 15197000
. : milestone, 15197000,
iast (18.774 s) : 18774000, 18774000
. : milestone, 18774000,
iast_GLOBAL (18.035 s) : 18035000, 18035000
. : milestone, 18035000,
profiling (15.416 s) : 15416000, 15416000
. : milestone, 15416000,
tracing (14.994 s) : 14994000, 14994000
. : milestone, 14994000,
section candidate
no_agent (15.149 s) : 15149000, 15149000
. : milestone, 15149000,
appsec (15.285 s) : 15285000, 15285000
. : milestone, 15285000,
iast (18.561 s) : 18561000, 18561000
. : milestone, 18561000,
iast_GLOBAL (17.665 s) : 17665000, 17665000
. : milestone, 17665000,
profiling (15.551 s) : 15551000, 15551000
. : milestone, 15551000,
tracing (15.261 s) : 15261000, 15261000
. : milestone, 15261000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.38.0-SNAPSHOT~352f20da9c, baseline=1.38.0-SNAPSHOT~63f16d7b20
dateFormat X
axisFormat %s
section baseline
no_agent (1.451 ms) : 1440, 1463
. : milestone, 1451,
appsec (2.236 ms) : 2200, 2272
. : milestone, 2236,
iast (1.965 ms) : 1923, 2007
. : milestone, 1965,
iast_GLOBAL (2.006 ms) : 1962, 2049
. : milestone, 2006,
profiling (1.844 ms) : 1810, 1879
. : milestone, 1844,
tracing (1.827 ms) : 1795, 1860
. : milestone, 1827,
section candidate
no_agent (1.463 ms) : 1451, 1474
. : milestone, 1463,
appsec (2.205 ms) : 2171, 2239
. : milestone, 2205,
iast (1.973 ms) : 1931, 2015
. : milestone, 1973,
iast_GLOBAL (2.002 ms) : 1961, 2044
. : milestone, 2002,
profiling (1.84 ms) : 1806, 1875
. : milestone, 1840,
tracing (1.836 ms) : 1803, 1869
. : milestone, 1836,
|
nikita-tkachenko-datadog
deleted the
nikita-tkachenko/fix-gradle-8.9-instrumentation
branch
July 12, 2024 13:37
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Fixes Gradle instrumentation following release v8.9
Additional Notes
Previous version of the code used
org.gradle.internal.service.DefaultServiceRegistry#addProvider(Object)
method to register CI Visibility plugin.In Gradle v8.9 this method was replaced with
org.gradle.internal.service.DefaultServiceRegistry#addProvider(org.gradle.internal.service.ServiceRegistrationProvider)
.The new
ServiceRegistrationProvider
interface is introduced in Gradle v8.9.The fixed version of the code uses alternative service registration method which works both in v8.9 and in older versions.
Jira ticket: SDTEST-566