Fix lettuce tests#8664
Merged
randomanderson merged 4 commits intomasterfrom Apr 3, 2025
Merged
Conversation
# Conflicts: # dd-java-agent/instrumentation/lettuce-5/src/test/groovy/Lettuce5AsyncClientTest.groovy # dd-java-agent/instrumentation/lettuce-5/src/test/groovy/Lettuce5ReactiveClientTest.groovy # dd-java-agent/instrumentation/lettuce-5/src/test/groovy/Lettuce5SyncClientTest.groovy
Contributor
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
randomanderson
commented
Apr 2, 2025
Comment on lines
+162
to
+165
| "$Tags.PEER_HOSTNAME" redisServer.getHost() | ||
| "$Tags.PEER_PORT" port | ||
| "$Tags.DB_TYPE" "redis" | ||
| "db.redis.dbIndex" 0 |
Contributor
Author
There was a problem hiding this comment.
These tests were never run so the fact that these tags weren't verified was hidden
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 66 metrics, 5 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.058 s) : 0, 1058037
Total [baseline] (10.519 s) : 0, 10519056
Agent [candidate] (1.065 s) : 0, 1064856
Total [candidate] (10.456 s) : 0, 10456214
section appsec
Agent [baseline] (1.197 s) : 0, 1196568
Total [baseline] (10.764 s) : 0, 10763736
Agent [candidate] (1.203 s) : 0, 1202696
Total [candidate] (10.904 s) : 0, 10903921
section iast
Agent [baseline] (1.188 s) : 0, 1188444
Total [baseline] (11.1 s) : 0, 11099600
Agent [candidate] (1.181 s) : 0, 1181009
Total [candidate] (11.018 s) : 0, 11018307
section profiling
Agent [baseline] (1.276 s) : 0, 1276099
Total [baseline] (10.926 s) : 0, 10926445
Agent [candidate] (1.279 s) : 0, 1279170
Total [candidate] (10.951 s) : 0, 10951069
gantt
title petclinic - break down per module: candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (723.833 ms) : 0, 723833
BytebuddyAgent [candidate] (727.958 ms) : 0, 727958
GlobalTracer [baseline] (241.283 ms) : 0, 241283
GlobalTracer [candidate] (242.055 ms) : 0, 242055
AppSec [baseline] (54.778 ms) : 0, 54778
AppSec [candidate] (55.369 ms) : 0, 55369
Debugger [baseline] (4.428 ms) : 0, 4428
Debugger [candidate] (4.456 ms) : 0, 4456
Remote Config [baseline] (706.47 µs) : 0, 706
Remote Config [candidate] (710.741 µs) : 0, 711
Telemetry [baseline] (12.388 ms) : 0, 12388
Telemetry [candidate] (13.573 ms) : 0, 13573
section appsec
BytebuddyAgent [baseline] (739.79 ms) : 0, 739790
BytebuddyAgent [candidate] (743.039 ms) : 0, 743039
GlobalTracer [baseline] (236.459 ms) : 0, 236459
GlobalTracer [candidate] (238.007 ms) : 0, 238007
IAST [baseline] (21.541 ms) : 0, 21541
IAST [candidate] (22.141 ms) : 0, 22141
AppSec [baseline] (175.993 ms) : 0, 175993
AppSec [candidate] (176.757 ms) : 0, 176757
Debugger [baseline] (4.261 ms) : 0, 4261
Debugger [candidate] (4.301 ms) : 0, 4301
Remote Config [baseline] (633.365 µs) : 0, 633
Remote Config [candidate] (650.163 µs) : 0, 650
Telemetry [baseline] (8.235 ms) : 0, 8235
Telemetry [candidate] (8.284 ms) : 0, 8284
section iast
BytebuddyAgent [baseline] (844.624 ms) : 0, 844624
BytebuddyAgent [candidate] (839.003 ms) : 0, 839003
GlobalTracer [baseline] (230.717 ms) : 0, 230717
GlobalTracer [candidate] (229.715 ms) : 0, 229715
IAST [baseline] (24.405 ms) : 0, 24405
IAST [candidate] (22.717 ms) : 0, 22717
AppSec [baseline] (54.58 ms) : 0, 54580
AppSec [candidate] (55.84 ms) : 0, 55840
Debugger [baseline] (4.139 ms) : 0, 4139
Debugger [candidate] (4.075 ms) : 0, 4075
Remote Config [baseline] (607.654 µs) : 0, 608
Remote Config [candidate] (597.205 µs) : 0, 597
Telemetry [baseline] (8.793 ms) : 0, 8793
Telemetry [candidate] (8.652 ms) : 0, 8652
section profiling
BytebuddyAgent [baseline] (711.326 ms) : 0, 711326
BytebuddyAgent [candidate] (712.793 ms) : 0, 712793
GlobalTracer [baseline] (348.884 ms) : 0, 348884
GlobalTracer [candidate] (350.117 ms) : 0, 350117
AppSec [baseline] (54.145 ms) : 0, 54145
AppSec [candidate] (53.201 ms) : 0, 53201
Debugger [baseline] (4.293 ms) : 0, 4293
Debugger [candidate] (4.303 ms) : 0, 4303
Remote Config [baseline] (687.793 µs) : 0, 688
Remote Config [candidate] (699.868 µs) : 0, 700
Telemetry [baseline] (8.948 ms) : 0, 8948
Telemetry [candidate] (8.961 ms) : 0, 8961
ProfilingAgent [baseline] (102.167 ms) : 0, 102167
ProfilingAgent [candidate] (103.337 ms) : 0, 103337
Profiling [baseline] (102.192 ms) : 0, 102192
Profiling [candidate] (103.364 ms) : 0, 103364
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.051 s) : 0, 1050862
Total [baseline] (8.67 s) : 0, 8669515
Agent [candidate] (1.059 s) : 0, 1058822
Total [candidate] (8.71 s) : 0, 8709992
section iast
Agent [baseline] (1.18 s) : 0, 1180403
Total [baseline] (9.287 s) : 0, 9287352
Agent [candidate] (1.197 s) : 0, 1197309
Total [candidate] (9.301 s) : 0, 9300656
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.18 s) : 0, 1179797
Total [baseline] (9.212 s) : 0, 9211536
Agent [candidate] (1.19 s) : 0, 1189817
Total [candidate] (9.263 s) : 0, 9263496
section iast_TELEMETRY_OFF
Agent [baseline] (1.185 s) : 0, 1185397
Total [baseline] (9.307 s) : 0, 9307191
Agent [candidate] (1.188 s) : 0, 1187959
Total [candidate] (9.316 s) : 0, 9316032
gantt
title insecure-bank - break down per module: candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (719.26 ms) : 0, 719260
BytebuddyAgent [candidate] (720.905 ms) : 0, 720905
GlobalTracer [baseline] (239.505 ms) : 0, 239505
GlobalTracer [candidate] (241.631 ms) : 0, 241631
AppSec [baseline] (54.385 ms) : 0, 54385
AppSec [candidate] (55.542 ms) : 0, 55542
Debugger [baseline] (4.392 ms) : 0, 4392
Debugger [candidate] (4.415 ms) : 0, 4415
Remote Config [baseline] (696.755 µs) : 0, 697
Remote Config [candidate] (715.722 µs) : 0, 716
Telemetry [baseline] (12.177 ms) : 0, 12177
Telemetry [candidate] (15.131 ms) : 0, 15131
section iast
BytebuddyAgent [baseline] (838.226 ms) : 0, 838226
BytebuddyAgent [candidate] (851.83 ms) : 0, 851830
GlobalTracer [baseline] (229.536 ms) : 0, 229536
GlobalTracer [candidate] (231.721 ms) : 0, 231721
AppSec [baseline] (55.929 ms) : 0, 55929
AppSec [candidate] (56.378 ms) : 0, 56378
Debugger [baseline] (4.162 ms) : 0, 4162
Debugger [candidate] (4.196 ms) : 0, 4196
Remote Config [baseline] (609.883 µs) : 0, 610
Remote Config [candidate] (607.29 µs) : 0, 607
Telemetry [baseline] (8.785 ms) : 0, 8785
Telemetry [candidate] (8.838 ms) : 0, 8838
IAST [baseline] (22.728 ms) : 0, 22728
IAST [candidate] (23.119 ms) : 0, 23119
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (837.648 ms) : 0, 837648
BytebuddyAgent [candidate] (845.076 ms) : 0, 845076
GlobalTracer [baseline] (229.897 ms) : 0, 229897
GlobalTracer [candidate] (231.173 ms) : 0, 231173
AppSec [baseline] (55.632 ms) : 0, 55632
AppSec [candidate] (56.119 ms) : 0, 56119
Debugger [baseline] (4.15 ms) : 0, 4150
Debugger [candidate] (4.179 ms) : 0, 4179
Remote Config [baseline] (614.89 µs) : 0, 615
Remote Config [candidate] (601.096 µs) : 0, 601
Telemetry [baseline] (8.739 ms) : 0, 8739
Telemetry [candidate] (8.843 ms) : 0, 8843
IAST [baseline] (22.674 ms) : 0, 22674
IAST [candidate] (23.203 ms) : 0, 23203
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (841.525 ms) : 0, 841525
BytebuddyAgent [candidate] (843.768 ms) : 0, 843768
GlobalTracer [baseline] (230.997 ms) : 0, 230997
GlobalTracer [candidate] (231.383 ms) : 0, 231383
AppSec [baseline] (56.397 ms) : 0, 56397
AppSec [candidate] (56.129 ms) : 0, 56129
Debugger [baseline] (4.183 ms) : 0, 4183
Debugger [candidate] (4.21 ms) : 0, 4210
Remote Config [baseline] (597.859 µs) : 0, 598
Remote Config [candidate] (599.714 µs) : 0, 600
Telemetry [baseline] (8.648 ms) : 0, 8648
Telemetry [candidate] (8.679 ms) : 0, 8679
IAST [baseline] (22.503 ms) : 0, 22503
IAST [candidate] (22.623 ms) : 0, 22623
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 14 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section baseline
no_agent (1.36 ms) : 1340, 1381
. : milestone, 1360,
appsec (1.746 ms) : 1722, 1769
. : milestone, 1746,
appsec_no_iast (1.738 ms) : 1716, 1761
. : milestone, 1738,
code_origins (1.65 ms) : 1623, 1678
. : milestone, 1650,
iast (1.506 ms) : 1481, 1531
. : milestone, 1506,
profiling (1.514 ms) : 1491, 1537
. : milestone, 1514,
tracing (1.51 ms) : 1486, 1534
. : milestone, 1510,
section candidate
no_agent (1.358 ms) : 1340, 1377
. : milestone, 1358,
appsec (1.731 ms) : 1708, 1755
. : milestone, 1731,
appsec_no_iast (1.717 ms) : 1694, 1741
. : milestone, 1717,
code_origins (1.693 ms) : 1667, 1720
. : milestone, 1693,
iast (1.511 ms) : 1486, 1537
. : milestone, 1511,
profiling (1.522 ms) : 1498, 1546
. : milestone, 1522,
tracing (1.495 ms) : 1470, 1520
. : milestone, 1495,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section baseline
no_agent (373.5 µs) : 354, 393
. : milestone, 373,
iast (508.165 µs) : 486, 530
. : milestone, 508,
iast_FULL (733.821 µs) : 712, 756
. : milestone, 734,
iast_GLOBAL (563.457 µs) : 540, 587
. : milestone, 563,
iast_HARDCODED_SECRET_DISABLED (507.132 µs) : 486, 529
. : milestone, 507,
iast_INACTIVE (457.386 µs) : 436, 478
. : milestone, 457,
iast_TELEMETRY_OFF (504.079 µs) : 482, 526
. : milestone, 504,
tracing (455.247 µs) : 435, 476
. : milestone, 455,
section candidate
no_agent (381.079 µs) : 360, 402
. : milestone, 381,
iast (513.864 µs) : 492, 536
. : milestone, 514,
iast_FULL (728.771 µs) : 707, 751
. : milestone, 729,
iast_GLOBAL (561.917 µs) : 539, 584
. : milestone, 562,
iast_HARDCODED_SECRET_DISABLED (514.917 µs) : 493, 537
. : milestone, 515,
iast_INACTIVE (461.359 µs) : 440, 483
. : milestone, 461,
iast_TELEMETRY_OFF (506.061 µs) : 484, 529
. : milestone, 506,
tracing (456.513 µs) : 436, 477
. : milestone, 457,
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.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section baseline
no_agent (1.484 ms) : 1472, 1495
. : milestone, 1484,
appsec (2.385 ms) : 2339, 2432
. : milestone, 2385,
iast (2.177 ms) : 2118, 2236
. : milestone, 2177,
iast_GLOBAL (2.213 ms) : 2154, 2272
. : milestone, 2213,
profiling (2.013 ms) : 1966, 2060
. : milestone, 2013,
tracing (1.992 ms) : 1947, 2038
. : milestone, 1992,
section candidate
no_agent (1.486 ms) : 1475, 1498
. : milestone, 1486,
appsec (2.401 ms) : 2353, 2449
. : milestone, 2401,
iast (2.163 ms) : 2104, 2222
. : milestone, 2163,
iast_GLOBAL (2.217 ms) : 2157, 2277
. : milestone, 2217,
profiling (2.034 ms) : 1985, 2083
. : milestone, 2034,
tracing (2.037 ms) : 1989, 2086
. : milestone, 2037,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.48.0-SNAPSHOT~305b97e86b, baseline=1.48.0-SNAPSHOT~09b7c65d37
dateFormat X
axisFormat %s
section baseline
no_agent (14.853 s) : 14853000, 14853000
. : milestone, 14853000,
appsec (15.148 s) : 15148000, 15148000
. : milestone, 15148000,
iast (18.702 s) : 18702000, 18702000
. : milestone, 18702000,
iast_GLOBAL (17.712 s) : 17712000, 17712000
. : milestone, 17712000,
profiling (14.943 s) : 14943000, 14943000
. : milestone, 14943000,
tracing (14.685 s) : 14685000, 14685000
. : milestone, 14685000,
section candidate
no_agent (15.422 s) : 15422000, 15422000
. : milestone, 15422000,
appsec (14.727 s) : 14727000, 14727000
. : milestone, 14727000,
iast (18.859 s) : 18859000, 18859000
. : milestone, 18859000,
iast_GLOBAL (18.066 s) : 18066000, 18066000
. : milestone, 18066000,
profiling (15.19 s) : 15190000, 15190000
. : milestone, 15190000,
tracing (15.137 s) : 15137000, 15137000
. : milestone, 15137000,
|
mcculls
approved these changes
Apr 2, 2025
This file contains hidden or 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
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.
Lettuce tests were flawed in a few ways:
shutdown()test which shuts down the server and breaks all tests after itSynctests were not run at all. The inner classes inLettuce5SyncClientTestwere extending theAsyncflavorThis PR fixes all of those issues. The setup code has been moved into a base class and the server is restarted between tests to prevent order dependence.
Motivation
I stumbled upon the broken tests while migrating to Gitlab. Gitlab has a different test order so it always broke.
Contributor 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]