-
Notifications
You must be signed in to change notification settings - Fork 312
Update AgentTestRunner to use JUnit5 - Continuation #9413
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
* Support async servlet for RUM injection * Move isRumEnabled to instrumenter config
* chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * Rollback some dependencies to fix compilation. * Added dependency of `compilePlayRoutes` on `fixPlayRoutesDirectory`. * Rollback some dependencies to fix compilation. --------- Co-authored-by: dd-octo-sts[bot] <200755185+dd-octo-sts[bot]@users.noreply.github.com> Co-authored-by: Alexey Kuznetsov <[email protected]>
Bumps the gh-actions-packages group with 3 updates: [actions/checkout](https://github.com/actions/checkout), [actions/cache](https://github.com/actions/cache) and [github/codeql-action](https://github.com/github/codeql-action). Updates `actions/checkout` from 4.2.2 to 5.0.0 - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@11bd719...08c6903) Updates `actions/cache` from 4.2.3 to 4.2.4 - [Release notes](https://github.com/actions/cache/releases) - [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md) - [Commits](actions/cache@5a3ec84...0400d5f) Updates `github/codeql-action` from 3.29.7 to 3.29.8 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@51f7732...76621b6) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: 5.0.0 dependency-type: direct:production update-type: version-update:semver-major dependency-group: gh-actions-packages - dependency-name: actions/cache dependency-version: 4.2.4 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gh-actions-packages - dependency-name: github/codeql-action dependency-version: 3.29.8 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gh-actions-packages ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Avoid NPE on featureDiscovery creation * Review
What Does This Do Modifies WafModule#buildEvents to safely handle actionWithData.data being null. Motivation Fix #9346 Additional Notes The current version of libddwaf may return null in the data field of actionWithData. This was previously not handled and could cause unexpected logged exceptions. The method now explicitly checks for null to prevent this.
#9351) * use magic mirror depot * remove separate cache name used for testing
by eliminating the chance identity hash might collide even after 3 rehashes the easiest way to do this is to bump the size of the cache used in testing note this is only necessary because we want to keep both keys in the cache to test the 'visit' method - otherwise we'd need to relax that assertion
* Add dependency to OCI publish jobs * Override new publishing-gate job * Change stage order because publishing-gate depends on deploy_to_maven_central * Add verify_maven_central_deployment job * Test verify_maven_central_deployment job by temporarily removing dependency * Change to registry.ddbuild image * Test failing version * Clean * Change image for verify_maven_central_deployment job * Fix verify_maven_central_deployment script * Remove publishing-gate rules * Test verify job * Add manual trigger * Fix verify script * Add test-job to see what happens when verify job fails * Clean test code * Change retry rate * Try test-job again * Clean again * Create a separate job to skip the verify maven job --------- Co-authored-by: Laplie Anderson <[email protected]>
…Client5NamingV0Test`. (#9361)
* feat(env): Migrate internal apis to environment component * feat(env): Add new API to EnvironmentVariables and SystemProperties
* Refactor matchOperator to use method reference comparator for efficiency * Implement better exception handling and debug logging for StableConfig parsing errors * Add a debug log about no rules getting applied * Improve test file logic and cleanup in StableConfigSourceTest class * Github suggestions: Use static factory methods as constructors for Rule and Selector classes * Improve exception message: include value of object that caused the failure * Add more test cases to StableConfigSourceTest to improve codecov * Add test coverage for StableConfigMappingException * Update internal-api/src/main/java/datadog/trace/bootstrap/config/provider/stableconfig/Rule.java Co-authored-by: Alexey Kuznetsov <[email protected]> * Update internal-api/src/main/java/datadog/trace/bootstrap/config/provider/stableconfig/Rule.java Co-authored-by: Alexey Kuznetsov <[email protected]> * Use safeToString when printing stableconfig map * rearrange logic in selectors creation for better readability * delete superfluous else * Use constant to represent max length in safeToString * use throwStableConfigMappingException helper function * refactor exception catching in StableConfigSource * restore semicolon in calls to throwStableConfigMappingException * Use static import for throwStableConfigMappingException * test other kinds of exceptions in StableConfigSource * remove duplicate cfg assignments * fix missing operator test case * Update internal-api/src/main/java/datadog/trace/bootstrap/config/provider/stableconfig/Rule.java Co-authored-by: Alexey Kuznetsov <[email protected]> * Improve readability of StableConfigMappingException and its function invocations * modify safeToString logic to print first 50 and last 50 chars * remove extra placeholder in string message when printing exception * Update internal-api/src/main/java/datadog/trace/bootstrap/config/provider/StableConfigSource.java Co-authored-by: Alexey Kuznetsov <[email protected]> * refactor Rules.from to perform type assertions before heavyweight list manipulation * run spotless --------- Co-authored-by: Alexey Kuznetsov <[email protected]>
* Add stage for publishing-gate * Allow publishing-gate to run when populating cache
* chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * chore: Update Gradle dependencies * Rollback some dependencies to fix compilation. --------- Co-authored-by: dd-octo-sts[bot] <200755185+dd-octo-sts[bot]@users.noreply.github.com> Co-authored-by: Alexey Kuznetsov <[email protected]>
* feat(bootstrap): Migrate Spray instrumentation to Context API * feat(bootstrap): Migrate Tomcat instrumentation to Context API * feat(bootstrap): Migrate Liberty instrumentations to Context API * feat(bootstrap): Migrate instrumentations to Context API * fix(bootstrap): Fix unexpected call to IG callback * feat(play): Simplify testing * fix(bootstrap): Fix import
Do not stop on git log check Fix next patch release version compute
…generated keys (#9321) * Prevent crash in SQL Server JDBC when tracing execute methods with generated keys When tracing the JDBC statement execute methods, we prepend a comment to the SQL query used for DBM trace propagation. However, there is a bug in the SQL Server JDBC driver that prevents the generated keys from being returned when the SQL comment is prepended to the SQL query. I decided to only append the comment in this specific case to avoid the comment from being truncated. @see microsoft/mssql-jdbc#2729 * remove dynamic typing when packing all the JDBC arguments * move JDBC test on injection with generated keys to SQL Server file * remove useless constant in StatementAdvice
…#9382) Bumps the gh-actions-packages group with 1 update: [github/codeql-action](https://github.com/github/codeql-action). Updates `github/codeql-action` from 3.29.8 to 3.29.10 - [Release notes](https://github.com/github/codeql-action/releases) - [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md) - [Commits](github/codeql-action@76621b6...96f518a) --- updated-dependencies: - dependency-name: github/codeql-action dependency-version: 3.29.10 dependency-type: direct:production update-type: version-update:semver-patch dependency-group: gh-actions-packages ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat(dsm): Improve code style * feat(aws): Refactor AWS instrumentation around Context
* initial introduction of config_id * fix tests, fix NPE in ConfigSetting.equals, include configId in telemetryrequestbody when nonnull * report config id for stableconfigsources in get * Fix call to outdated ConfigSetting constructor in EventSourceTest * fix config_id report * fix where null config id was getting treated as string 'null' * report config id for all get methods * remove superfluous file * simplify configId resolution logic in configProvider * report sourceValue instead of value to ConfigCollector in get method * add tests for configid and no configid
🎯 Code Coverage 🔗 Commit SHA: 4955894 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 47 metrics, 12 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.065 s) : 0, 1064501
Total [baseline] (8.725 s) : 0, 8724685
Agent [candidate] (1.049 s) : 0, 1048920
Total [candidate] (8.647 s) : 0, 8647421
section iast
Agent [baseline] (1.187 s) : 0, 1186835
Total [baseline] (9.404 s) : 0, 9403689
Agent [candidate] (1.192 s) : 0, 1191865
Total [candidate] (9.358 s) : 0, 9357597
gantt
title insecure-bank - break down per module: candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.474 ms) : 0, 1474
crashtracking [candidate] (1.46 ms) : 0, 1460
BytebuddyAgent [baseline] (743.618 ms) : 0, 743618
BytebuddyAgent [candidate] (733.676 ms) : 0, 733676
GlobalTracer [baseline] (246.135 ms) : 0, 246135
GlobalTracer [candidate] (242.95 ms) : 0, 242950
AppSec [baseline] (30.761 ms) : 0, 30761
AppSec [candidate] (30.096 ms) : 0, 30096
Debugger [baseline] (6.132 ms) : 0, 6132
Debugger [candidate] (6.079 ms) : 0, 6079
Remote Config [baseline] (681.064 µs) : 0, 681
Remote Config [candidate] (665.059 µs) : 0, 665
Telemetry [baseline] (14.452 ms) : 0, 14452
Telemetry [candidate] (12.952 ms) : 0, 12952
section iast
crashtracking [baseline] (1.457 ms) : 0, 1457
crashtracking [candidate] (1.471 ms) : 0, 1471
BytebuddyAgent [baseline] (856.219 ms) : 0, 856219
BytebuddyAgent [candidate] (860.346 ms) : 0, 860346
GlobalTracer [baseline] (234.459 ms) : 0, 234459
GlobalTracer [candidate] (235.228 ms) : 0, 235228
IAST [baseline] (28.163 ms) : 0, 28163
IAST [candidate] (28.176 ms) : 0, 28176
AppSec [baseline] (30.508 ms) : 0, 30508
AppSec [candidate] (28.894 ms) : 0, 28894
Debugger [baseline] (5.806 ms) : 0, 5806
Debugger [candidate] (7.435 ms) : 0, 7435
Remote Config [baseline] (626.314 µs) : 0, 626
Remote Config [candidate] (604.932 µs) : 0, 605
Telemetry [baseline] (8.493 ms) : 0, 8493
Telemetry [candidate] (8.479 ms) : 0, 8479
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.059 s) : 0, 1058596
Total [baseline] (10.78 s) : 0, 10780464
Agent [candidate] (1.048 s) : 0, 1048357
Total [candidate] (10.769 s) : 0, 10768967
section appsec
Agent [baseline] (1.223 s) : 0, 1223167
Total [baseline] (10.789 s) : 0, 10788501
Agent [candidate] (1.224 s) : 0, 1224017
Total [candidate] (10.734 s) : 0, 10733976
section iast
Agent [baseline] (1.188 s) : 0, 1188482
Total [baseline] (10.952 s) : 0, 10951868
Agent [candidate] (1.18 s) : 0, 1180004
Total [candidate] (10.884 s) : 0, 10883755
section profiling
Agent [baseline] (1.207 s) : 0, 1206582
Total [baseline] (10.999 s) : 0, 10999255
Agent [candidate] (1.196 s) : 0, 1196431
Total [candidate] (10.912 s) : 0, 10912497
gantt
title petclinic - break down per module: candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.474 ms) : 0, 1474
crashtracking [candidate] (1.467 ms) : 0, 1467
BytebuddyAgent [baseline] (742.649 ms) : 0, 742649
BytebuddyAgent [candidate] (733.384 ms) : 0, 733384
GlobalTracer [baseline] (244.329 ms) : 0, 244329
GlobalTracer [candidate] (242.642 ms) : 0, 242642
AppSec [baseline] (30.431 ms) : 0, 30431
AppSec [candidate] (30.167 ms) : 0, 30167
Debugger [baseline] (6.081 ms) : 0, 6081
Debugger [candidate] (6.048 ms) : 0, 6048
Remote Config [baseline] (669.759 µs) : 0, 670
Remote Config [candidate] (665.454 µs) : 0, 665
Telemetry [baseline] (11.705 ms) : 0, 11705
Telemetry [candidate] (12.901 ms) : 0, 12901
section appsec
crashtracking [baseline] (1.45 ms) : 0, 1450
crashtracking [candidate] (1.458 ms) : 0, 1458
BytebuddyAgent [baseline] (755.269 ms) : 0, 755269
BytebuddyAgent [candidate] (755.736 ms) : 0, 755736
GlobalTracer [baseline] (235.633 ms) : 0, 235633
GlobalTracer [candidate] (235.757 ms) : 0, 235757
IAST [baseline] (23.468 ms) : 0, 23468
IAST [candidate] (23.74 ms) : 0, 23740
AppSec [baseline] (171.323 ms) : 0, 171323
AppSec [candidate] (171.388 ms) : 0, 171388
Debugger [baseline] (5.794 ms) : 0, 5794
Debugger [candidate] (5.732 ms) : 0, 5732
Remote Config [baseline] (638.294 µs) : 0, 638
Remote Config [candidate] (633.569 µs) : 0, 634
Telemetry [baseline] (8.512 ms) : 0, 8512
Telemetry [candidate] (8.407 ms) : 0, 8407
section iast
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.457 ms) : 0, 1457
BytebuddyAgent [baseline] (857.402 ms) : 0, 857402
BytebuddyAgent [candidate] (851.655 ms) : 0, 851655
GlobalTracer [baseline] (235.434 ms) : 0, 235434
GlobalTracer [candidate] (233.262 ms) : 0, 233262
IAST [baseline] (28.734 ms) : 0, 28734
IAST [candidate] (29.137 ms) : 0, 29137
AppSec [baseline] (28.569 ms) : 0, 28569
AppSec [candidate] (26.97 ms) : 0, 26970
Debugger [baseline] (6.613 ms) : 0, 6613
Debugger [candidate] (7.558 ms) : 0, 7558
Remote Config [baseline] (631.62 µs) : 0, 632
Remote Config [candidate] (602.67 µs) : 0, 603
Telemetry [baseline] (8.428 ms) : 0, 8428
Telemetry [candidate] (8.322 ms) : 0, 8322
section profiling
crashtracking [baseline] (1.457 ms) : 0, 1457
crashtracking [candidate] (1.449 ms) : 0, 1449
BytebuddyAgent [baseline] (768.695 ms) : 0, 768695
BytebuddyAgent [candidate] (761.941 ms) : 0, 761941
GlobalTracer [baseline] (224.453 ms) : 0, 224453
GlobalTracer [candidate] (222.794 ms) : 0, 222794
AppSec [baseline] (30.325 ms) : 0, 30325
AppSec [candidate] (29.932 ms) : 0, 29932
Debugger [baseline] (7.083 ms) : 0, 7083
Debugger [candidate] (7.751 ms) : 0, 7751
Remote Config [baseline] (724.72 µs) : 0, 725
Remote Config [candidate] (692.39 µs) : 0, 692
Telemetry [baseline] (15.721 ms) : 0, 15721
Telemetry [candidate] (14.709 ms) : 0, 14709
ProfilingAgent [baseline] (108.009 ms) : 0, 108009
ProfilingAgent [candidate] (107.458 ms) : 0, 107458
Profiling [baseline] (108.643 ms) : 0, 108643
Profiling [candidate] (108.103 ms) : 0, 108103
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section baseline
no_agent (37.193 ms) : 36890, 37495
. : milestone, 37193,
appsec (48.082 ms) : 47656, 48507
. : milestone, 48082,
code_origins (45.237 ms) : 44842, 45632
. : milestone, 45237,
iast (45.598 ms) : 45202, 45994
. : milestone, 45598,
profiling (48.094 ms) : 47595, 48593
. : milestone, 48094,
tracing (45.598 ms) : 45198, 45997
. : milestone, 45598,
section candidate
no_agent (37.225 ms) : 36916, 37533
. : milestone, 37225,
appsec (47.143 ms) : 46739, 47547
. : milestone, 47143,
code_origins (46.206 ms) : 45812, 46600
. : milestone, 46206,
iast (45.384 ms) : 44978, 45790
. : milestone, 45384,
profiling (49.156 ms) : 48737, 49574
. : milestone, 49156,
tracing (46.808 ms) : 46396, 47220
. : milestone, 46808,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section baseline
no_agent (4.446 ms) : 4396, 4496
. : milestone, 4446,
iast (9.327 ms) : 9173, 9481
. : milestone, 9327,
iast_FULL (14.175 ms) : 13893, 14456
. : milestone, 14175,
iast_GLOBAL (10.065 ms) : 9890, 10240
. : milestone, 10065,
profiling (9.277 ms) : 9126, 9429
. : milestone, 9277,
tracing (7.977 ms) : 7852, 8103
. : milestone, 7977,
section candidate
no_agent (4.428 ms) : 4378, 4478
. : milestone, 4428,
iast (9.531 ms) : 9378, 9685
. : milestone, 9531,
iast_FULL (14.261 ms) : 13973, 14550
. : milestone, 14261,
iast_GLOBAL (10.32 ms) : 10141, 10499
. : milestone, 10320,
profiling (8.526 ms) : 8396, 8655
. : milestone, 8526,
tracing (7.758 ms) : 7646, 7870
. : milestone, 7758,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section baseline
no_agent (1.478 ms) : 1466, 1489
. : milestone, 1478,
appsec (3.628 ms) : 3413, 3842
. : milestone, 3628,
iast (2.212 ms) : 2149, 2276
. : milestone, 2212,
iast_GLOBAL (2.252 ms) : 2189, 2315
. : milestone, 2252,
profiling (2.072 ms) : 2020, 2124
. : milestone, 2072,
tracing (2.033 ms) : 1984, 2082
. : milestone, 2033,
section candidate
no_agent (1.477 ms) : 1465, 1488
. : milestone, 1477,
appsec (3.622 ms) : 3409, 3835
. : milestone, 3622,
iast (2.203 ms) : 2140, 2266
. : milestone, 2203,
iast_GLOBAL (2.253 ms) : 2190, 2316
. : milestone, 2253,
profiling (2.062 ms) : 2011, 2114
. : milestone, 2062,
tracing (2.031 ms) : 1981, 2080
. : milestone, 2031,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.53.0-SNAPSHOT~49558946ff, baseline=1.53.0-SNAPSHOT~02a3250561
dateFormat X
axisFormat %s
section baseline
no_agent (15.05 s) : 15050000, 15050000
. : milestone, 15050000,
appsec (15.058 s) : 15058000, 15058000
. : milestone, 15058000,
iast (18.479 s) : 18479000, 18479000
. : milestone, 18479000,
iast_GLOBAL (18.211 s) : 18211000, 18211000
. : milestone, 18211000,
profiling (15.616 s) : 15616000, 15616000
. : milestone, 15616000,
tracing (14.965 s) : 14965000, 14965000
. : milestone, 14965000,
section candidate
no_agent (14.968 s) : 14968000, 14968000
. : milestone, 14968000,
appsec (14.95 s) : 14950000, 14950000
. : milestone, 14950000,
iast (18.307 s) : 18307000, 18307000
. : milestone, 18307000,
iast_GLOBAL (18.022 s) : 18022000, 18022000
. : milestone, 18022000,
profiling (15.416 s) : 15416000, 15416000
. : milestone, 15416000,
tracing (15.392 s) : 15392000, 15392000
. : milestone, 15392000,
|
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
|
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
|
…sting, and servlet tests
Closing in lieu of #9431 |
What Does This Do
Built on #9264 to address failing instrumentation tests
Motivation
Update AgentTestRunner to use JUnit5, necessary to support Java 25 and test JUnit6
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: [PROJ-IDENT]