Update Spock to 2.4-M6#9516
Conversation
… compatible with Spock 2.4
There was a problem hiding this comment.
This class used to be a Spy created with Spock. Due to the changes in mocking in Spock 2.4 it is no longer possible to spy on DDSpan (it has final methods and no 0-arg constructor, so Spock's new ByteBuddy and Mockito mockers cannot handle it), which is why it was replaced with a regular class replicating the same logic.
There was a problem hiding this comment.
This class used to be a Spy created with Spock. Due to the changes in mocking in Spock 2.4 it is no longer possible to spy on DDSpan (it has final methods and no 0-arg constructor, so Spock's new ByteBuddy and Mockito mockers cannot handle it), which is why it was replaced with a regular class replicating the same logic.
|
🎯 Code Coverage 🔗 Commit SHA: 559008b | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 8 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.07 s) : 0, 1069860
Total [baseline] (10.737 s) : 0, 10737419
Agent [candidate] (1.063 s) : 0, 1062670
Total [candidate] (10.625 s) : 0, 10624705
section appsec
Agent [baseline] (1.237 s) : 0, 1236599
Total [baseline] (11.02 s) : 0, 11020456
Agent [candidate] (1.234 s) : 0, 1234486
Total [candidate] (10.837 s) : 0, 10837404
section iast
Agent [baseline] (1.193 s) : 0, 1192997
Total [baseline] (11.095 s) : 0, 11094506
Agent [candidate] (1.191 s) : 0, 1191099
Total [candidate] (11.108 s) : 0, 11107820
section profiling
Agent [baseline] (1.209 s) : 0, 1208762
Total [baseline] (10.867 s) : 0, 10867039
Agent [candidate] (1.213 s) : 0, 1212557
Total [candidate] (10.879 s) : 0, 10879335
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.442 ms) : 0, 1442
BytebuddyAgent [baseline] (738.424 ms) : 0, 738424
BytebuddyAgent [candidate] (733.307 ms) : 0, 733307
GlobalTracer [baseline] (254.361 ms) : 0, 254361
GlobalTracer [candidate] (252.901 ms) : 0, 252901
AppSec [baseline] (30.792 ms) : 0, 30792
AppSec [candidate] (30.575 ms) : 0, 30575
Debugger [baseline] (6.443 ms) : 0, 6443
Debugger [candidate] (6.411 ms) : 0, 6411
Remote Config [baseline] (706.884 µs) : 0, 707
Remote Config [candidate] (677.755 µs) : 0, 678
Telemetry [baseline] (16.431 ms) : 0, 16431
Telemetry [candidate] (16.259 ms) : 0, 16259
section appsec
crashtracking [baseline] (1.454 ms) : 0, 1454
crashtracking [candidate] (1.452 ms) : 0, 1452
BytebuddyAgent [baseline] (756.912 ms) : 0, 756912
BytebuddyAgent [candidate] (755.707 ms) : 0, 755707
GlobalTracer [baseline] (246.046 ms) : 0, 246046
GlobalTracer [candidate] (245.733 ms) : 0, 245733
AppSec [baseline] (171.85 ms) : 0, 171850
AppSec [candidate] (170.685 ms) : 0, 170685
Debugger [baseline] (6.104 ms) : 0, 6104
Debugger [candidate] (6.036 ms) : 0, 6036
Remote Config [baseline] (650.82 µs) : 0, 651
Remote Config [candidate] (629.549 µs) : 0, 630
Telemetry [baseline] (8.519 ms) : 0, 8519
Telemetry [candidate] (9.279 ms) : 0, 9279
IAST [baseline] (23.855 ms) : 0, 23855
IAST [candidate] (23.76 ms) : 0, 23760
section iast
crashtracking [baseline] (1.463 ms) : 0, 1463
crashtracking [candidate] (1.442 ms) : 0, 1442
BytebuddyAgent [baseline] (854.088 ms) : 0, 854088
BytebuddyAgent [candidate] (852.001 ms) : 0, 852001
GlobalTracer [baseline] (248.161 ms) : 0, 248161
GlobalTracer [candidate] (246.285 ms) : 0, 246285
AppSec [baseline] (25.491 ms) : 0, 25491
AppSec [candidate] (25.603 ms) : 0, 25603
Debugger [baseline] (6.05 ms) : 0, 6050
Debugger [candidate] (6.131 ms) : 0, 6131
Remote Config [baseline] (630.067 µs) : 0, 630
Remote Config [candidate] (606.878 µs) : 0, 607
Telemetry [baseline] (8.185 ms) : 0, 8185
Telemetry [candidate] (8.219 ms) : 0, 8219
IAST [baseline] (27.896 ms) : 0, 27896
IAST [candidate] (29.783 ms) : 0, 29783
section profiling
ProfilingAgent [baseline] (107.593 ms) : 0, 107593
ProfilingAgent [candidate] (107.247 ms) : 0, 107247
crashtracking [baseline] (1.436 ms) : 0, 1436
crashtracking [candidate] (1.428 ms) : 0, 1428
BytebuddyAgent [baseline] (761.683 ms) : 0, 761683
BytebuddyAgent [candidate] (764.673 ms) : 0, 764673
GlobalTracer [baseline] (233.002 ms) : 0, 233002
GlobalTracer [candidate] (233.662 ms) : 0, 233662
AppSec [baseline] (31.105 ms) : 0, 31105
AppSec [candidate] (31.331 ms) : 0, 31331
Debugger [baseline] (9.073 ms) : 0, 9073
Debugger [candidate] (8.205 ms) : 0, 8205
Remote Config [baseline] (735.173 µs) : 0, 735
Remote Config [candidate] (726.44 µs) : 0, 726
Telemetry [baseline] (13.238 ms) : 0, 13238
Telemetry [candidate] (14.066 ms) : 0, 14066
Profiling [baseline] (108.245 ms) : 0, 108245
Profiling [candidate] (107.914 ms) : 0, 107914
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.062 s) : 0, 1062135
Total [baseline] (8.614 s) : 0, 8614263
Agent [candidate] (1.06 s) : 0, 1059790
Total [candidate] (8.611 s) : 0, 8611176
section iast
Agent [baseline] (1.198 s) : 0, 1198019
Total [baseline] (9.278 s) : 0, 9277964
Agent [candidate] (1.194 s) : 0, 1193658
Total [candidate] (9.341 s) : 0, 9341340
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.459 ms) : 0, 1459
crashtracking [candidate] (1.447 ms) : 0, 1447
BytebuddyAgent [baseline] (733.248 ms) : 0, 733248
BytebuddyAgent [candidate] (731.341 ms) : 0, 731341
GlobalTracer [baseline] (252.326 ms) : 0, 252326
GlobalTracer [candidate] (252.13 ms) : 0, 252130
AppSec [baseline] (30.678 ms) : 0, 30678
AppSec [candidate] (30.545 ms) : 0, 30545
Debugger [baseline] (6.411 ms) : 0, 6411
Debugger [candidate] (6.427 ms) : 0, 6427
Remote Config [baseline] (693.127 µs) : 0, 693
Remote Config [candidate] (692.595 µs) : 0, 693
Telemetry [baseline] (16.306 ms) : 0, 16306
Telemetry [candidate] (16.201 ms) : 0, 16201
section iast
crashtracking [baseline] (1.463 ms) : 0, 1463
crashtracking [candidate] (1.466 ms) : 0, 1466
BytebuddyAgent [baseline] (858.931 ms) : 0, 858931
BytebuddyAgent [candidate] (855.972 ms) : 0, 855972
GlobalTracer [baseline] (247.468 ms) : 0, 247468
GlobalTracer [candidate] (247.301 ms) : 0, 247301
IAST [baseline] (27.926 ms) : 0, 27926
IAST [candidate] (28.496 ms) : 0, 28496
AppSec [baseline] (26.376 ms) : 0, 26376
AppSec [candidate] (24.727 ms) : 0, 24727
Debugger [baseline] (6.057 ms) : 0, 6057
Debugger [candidate] (6.031 ms) : 0, 6031
Remote Config [baseline] (606.413 µs) : 0, 606
Remote Config [candidate] (611.961 µs) : 0, 612
Telemetry [baseline] (8.131 ms) : 0, 8131
Telemetry [candidate] (8.07 ms) : 0, 8070
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 1 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section baseline
no_agent (4.348 ms) : 4299, 4397
. : milestone, 4348,
iast (9.227 ms) : 9069, 9385
. : milestone, 9227,
iast_FULL (13.856 ms) : 13582, 14129
. : milestone, 13856,
iast_GLOBAL (10.882 ms) : 10690, 11074
. : milestone, 10882,
profiling (8.727 ms) : 8590, 8864
. : milestone, 8727,
tracing (7.548 ms) : 7443, 7653
. : milestone, 7548,
section candidate
no_agent (4.475 ms) : 4421, 4528
. : milestone, 4475,
iast (9.442 ms) : 9288, 9596
. : milestone, 9442,
iast_FULL (14.682 ms) : 14382, 14981
. : milestone, 14682,
iast_GLOBAL (10.514 ms) : 10325, 10704
. : milestone, 10514,
profiling (8.862 ms) : 8724, 8999
. : milestone, 8862,
tracing (7.585 ms) : 7479, 7690
. : milestone, 7585,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section baseline
no_agent (38.912 ms) : 38601, 39224
. : milestone, 38912,
appsec (49.327 ms) : 48908, 49746
. : milestone, 49327,
code_origins (46.756 ms) : 46358, 47154
. : milestone, 46756,
iast (44.559 ms) : 44172, 44945
. : milestone, 44559,
profiling (48.302 ms) : 47836, 48768
. : milestone, 48302,
tracing (44.037 ms) : 43672, 44402
. : milestone, 44037,
section candidate
no_agent (36.394 ms) : 36102, 36686
. : milestone, 36394,
appsec (47.96 ms) : 47557, 48364
. : milestone, 47960,
code_origins (45.087 ms) : 44714, 45460
. : milestone, 45087,
iast (45.568 ms) : 45180, 45956
. : milestone, 45568,
profiling (48.813 ms) : 48362, 49265
. : milestone, 48813,
tracing (45.089 ms) : 44730, 45448
. : milestone, 45089,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section baseline
no_agent (15.529 s) : 15529000, 15529000
. : milestone, 15529000,
appsec (15.149 s) : 15149000, 15149000
. : milestone, 15149000,
iast (18.368 s) : 18368000, 18368000
. : milestone, 18368000,
iast_GLOBAL (18.1 s) : 18100000, 18100000
. : milestone, 18100000,
profiling (15.895 s) : 15895000, 15895000
. : milestone, 15895000,
tracing (14.904 s) : 14904000, 14904000
. : milestone, 14904000,
section candidate
no_agent (15.525 s) : 15525000, 15525000
. : milestone, 15525000,
appsec (15.04 s) : 15040000, 15040000
. : milestone, 15040000,
iast (18.625 s) : 18625000, 18625000
. : milestone, 18625000,
iast_GLOBAL (17.958 s) : 17958000, 17958000
. : milestone, 17958000,
profiling (15.335 s) : 15335000, 15335000
. : milestone, 15335000,
tracing (15.146 s) : 15146000, 15146000
. : milestone, 15146000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~559008b07e, baseline=1.54.0-SNAPSHOT~f7ec25f082
dateFormat X
axisFormat %s
section baseline
no_agent (1.481 ms) : 1470, 1493
. : milestone, 1481,
appsec (3.732 ms) : 3513, 3952
. : milestone, 3732,
iast (2.216 ms) : 2153, 2278
. : milestone, 2216,
iast_GLOBAL (2.259 ms) : 2196, 2323
. : milestone, 2259,
profiling (2.064 ms) : 2013, 2114
. : milestone, 2064,
tracing (2.034 ms) : 1985, 2083
. : milestone, 2034,
section candidate
no_agent (1.481 ms) : 1469, 1492
. : milestone, 1481,
appsec (2.467 ms) : 2417, 2517
. : milestone, 2467,
iast (2.213 ms) : 2151, 2276
. : milestone, 2213,
iast_GLOBAL (2.263 ms) : 2199, 2326
. : milestone, 2263,
profiling (2.069 ms) : 2017, 2121
. : milestone, 2069,
tracing (2.031 ms) : 1983, 2080
. : milestone, 2031,
|
What Does This Do
Updates Spock version used by the tracer from 2.3 to 2.4-M6.
Motivation
Improve CI stability: Spock 2.3 suffers from a deadlock problem fixed in 2.4 (PR in Spock).
Additional Notes
Kudos to @AlexeyKuznetsov-DD for laying the groundwork.
While migrating, 4 problems had to be solved:
InstrumentationSpecificationwas broken due to the changes in how mocking works in Spock 2.4Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]