Update TestNG6 DTD resolution during tests#11206
Update TestNG6 DTD resolution during tests#11206gh-worker-dd-mergequeue-cf854d[bot] merged 2 commits intomasterfrom
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 62 metrics, 9 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.058 s) : 0, 1058399
Total [baseline] (8.861 s) : 0, 8861305
Agent [candidate] (1.057 s) : 0, 1057339
Total [candidate] (8.863 s) : 0, 8863158
section iast
Agent [baseline] (1.237 s) : 0, 1236634
Total [baseline] (9.594 s) : 0, 9593530
Agent [candidate] (1.234 s) : 0, 1234488
Total [candidate] (9.545 s) : 0, 9544771
gantt
title insecure-bank - break down per module: candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.229 ms) : 0, 1229
crashtracking [candidate] (1.231 ms) : 0, 1231
BytebuddyAgent [baseline] (637.051 ms) : 0, 637051
BytebuddyAgent [candidate] (635.814 ms) : 0, 635814
AgentMeter [baseline] (29.647 ms) : 0, 29647
AgentMeter [candidate] (29.786 ms) : 0, 29786
GlobalTracer [baseline] (248.879 ms) : 0, 248879
GlobalTracer [candidate] (249.758 ms) : 0, 249758
AppSec [baseline] (32.369 ms) : 0, 32369
AppSec [candidate] (32.491 ms) : 0, 32491
Debugger [baseline] (59.805 ms) : 0, 59805
Debugger [candidate] (59.74 ms) : 0, 59740
Remote Config [baseline] (615.862 µs) : 0, 616
Remote Config [candidate] (604.239 µs) : 0, 604
Telemetry [baseline] (8.069 ms) : 0, 8069
Telemetry [candidate] (8.1 ms) : 0, 8100
Flare Poller [baseline] (4.424 ms) : 0, 4424
Flare Poller [candidate] (3.572 ms) : 0, 3572
section iast
crashtracking [baseline] (1.237 ms) : 0, 1237
crashtracking [candidate] (1.223 ms) : 0, 1223
BytebuddyAgent [baseline] (813.995 ms) : 0, 813995
BytebuddyAgent [candidate] (811.857 ms) : 0, 811857
AgentMeter [baseline] (11.395 ms) : 0, 11395
AgentMeter [candidate] (11.389 ms) : 0, 11389
GlobalTracer [baseline] (239.658 ms) : 0, 239658
GlobalTracer [candidate] (239.665 ms) : 0, 239665
IAST [baseline] (29.37 ms) : 0, 29370
IAST [candidate] (30.091 ms) : 0, 30091
AppSec [baseline] (27.735 ms) : 0, 27735
AppSec [candidate] (28.585 ms) : 0, 28585
Debugger [baseline] (65.313 ms) : 0, 65313
Debugger [candidate] (63.892 ms) : 0, 63892
Remote Config [baseline] (547.377 µs) : 0, 547
Remote Config [candidate] (533.538 µs) : 0, 534
Telemetry [baseline] (7.746 ms) : 0, 7746
Telemetry [candidate] (7.816 ms) : 0, 7816
Flare Poller [baseline] (3.458 ms) : 0, 3458
Flare Poller [candidate] (3.397 ms) : 0, 3397
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.072 s) : 0, 1072026
Total [baseline] (11.167 s) : 0, 11166784
Agent [candidate] (1.057 s) : 0, 1057076
Total [candidate] (11.193 s) : 0, 11193016
section appsec
Agent [baseline] (1.273 s) : 0, 1273106
Total [baseline] (11.181 s) : 0, 11181494
Agent [candidate] (1.276 s) : 0, 1276222
Total [candidate] (11.135 s) : 0, 11134690
section iast
Agent [baseline] (1.235 s) : 0, 1235014
Total [baseline] (11.311 s) : 0, 11311266
Agent [candidate] (1.234 s) : 0, 1234317
Total [candidate] (11.319 s) : 0, 11319157
section profiling
Agent [baseline] (1.194 s) : 0, 1194046
Total [baseline] (11.085 s) : 0, 11085141
Agent [candidate] (1.187 s) : 0, 1186884
Total [candidate] (11.035 s) : 0, 11034538
gantt
title petclinic - break down per module: candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.26 ms) : 0, 1260
crashtracking [candidate] (1.236 ms) : 0, 1236
BytebuddyAgent [baseline] (644.718 ms) : 0, 644718
BytebuddyAgent [candidate] (636.306 ms) : 0, 636306
AgentMeter [baseline] (29.994 ms) : 0, 29994
AgentMeter [candidate] (29.526 ms) : 0, 29526
GlobalTracer [baseline] (251.366 ms) : 0, 251366
GlobalTracer [candidate] (249.144 ms) : 0, 249144
AppSec [baseline] (32.817 ms) : 0, 32817
AppSec [candidate] (32.276 ms) : 0, 32276
Debugger [baseline] (61.352 ms) : 0, 61352
Debugger [candidate] (60.147 ms) : 0, 60147
Remote Config [baseline] (617.947 µs) : 0, 618
Remote Config [candidate] (605.539 µs) : 0, 606
Telemetry [baseline] (8.18 ms) : 0, 8180
Telemetry [candidate] (8.091 ms) : 0, 8091
Flare Poller [baseline] (5.235 ms) : 0, 5235
Flare Poller [candidate] (3.584 ms) : 0, 3584
section appsec
crashtracking [baseline] (1.241 ms) : 0, 1241
crashtracking [candidate] (1.248 ms) : 0, 1248
BytebuddyAgent [baseline] (679.723 ms) : 0, 679723
BytebuddyAgent [candidate] (684.983 ms) : 0, 684983
AgentMeter [baseline] (12.294 ms) : 0, 12294
AgentMeter [candidate] (12.414 ms) : 0, 12414
GlobalTracer [baseline] (252.186 ms) : 0, 252186
GlobalTracer [candidate] (251.169 ms) : 0, 251169
IAST [baseline] (24.47 ms) : 0, 24470
IAST [candidate] (24.41 ms) : 0, 24410
AppSec [baseline] (188.621 ms) : 0, 188621
AppSec [candidate] (189.724 ms) : 0, 189724
Debugger [baseline] (65.907 ms) : 0, 65907
Debugger [candidate] (63.678 ms) : 0, 63678
Remote Config [baseline] (594.051 µs) : 0, 594
Remote Config [candidate] (576.124 µs) : 0, 576
Telemetry [baseline] (8.032 ms) : 0, 8032
Telemetry [candidate] (7.967 ms) : 0, 7967
Flare Poller [baseline] (3.523 ms) : 0, 3523
Flare Poller [candidate] (3.477 ms) : 0, 3477
section iast
crashtracking [baseline] (1.226 ms) : 0, 1226
crashtracking [candidate] (1.222 ms) : 0, 1222
BytebuddyAgent [baseline] (812.292 ms) : 0, 812292
BytebuddyAgent [candidate] (811.26 ms) : 0, 811260
AgentMeter [baseline] (11.42 ms) : 0, 11420
AgentMeter [candidate] (11.384 ms) : 0, 11384
GlobalTracer [baseline] (239.547 ms) : 0, 239547
GlobalTracer [candidate] (239.359 ms) : 0, 239359
IAST [baseline] (31.802 ms) : 0, 31802
IAST [candidate] (29.225 ms) : 0, 29225
AppSec [baseline] (28.243 ms) : 0, 28243
AppSec [candidate] (30.107 ms) : 0, 30107
Debugger [baseline] (62.863 ms) : 0, 62863
Debugger [candidate] (63.948 ms) : 0, 63948
Remote Config [baseline] (539.145 µs) : 0, 539
Remote Config [candidate] (531.161 µs) : 0, 531
Telemetry [baseline] (7.678 ms) : 0, 7678
Telemetry [candidate] (7.813 ms) : 0, 7813
Flare Poller [baseline] (3.397 ms) : 0, 3397
Flare Poller [candidate] (3.442 ms) : 0, 3442
section profiling
crashtracking [baseline] (1.193 ms) : 0, 1193
crashtracking [candidate] (1.171 ms) : 0, 1171
BytebuddyAgent [baseline] (697.398 ms) : 0, 697398
BytebuddyAgent [candidate] (693.81 ms) : 0, 693810
AgentMeter [baseline] (8.971 ms) : 0, 8971
AgentMeter [candidate] (9.002 ms) : 0, 9002
GlobalTracer [baseline] (208.761 ms) : 0, 208761
GlobalTracer [candidate] (207.85 ms) : 0, 207850
AppSec [baseline] (32.894 ms) : 0, 32894
AppSec [candidate] (32.552 ms) : 0, 32552
Debugger [baseline] (66.457 ms) : 0, 66457
Debugger [candidate] (65.57 ms) : 0, 65570
Remote Config [baseline] (587.921 µs) : 0, 588
Remote Config [candidate] (574.419 µs) : 0, 574
Telemetry [baseline] (7.828 ms) : 0, 7828
Telemetry [candidate] (7.81 ms) : 0, 7810
Flare Poller [baseline] (3.514 ms) : 0, 3514
Flare Poller [candidate] (3.483 ms) : 0, 3483
ProfilingAgent [baseline] (94.419 ms) : 0, 94419
ProfilingAgent [candidate] (93.568 ms) : 0, 93568
Profiling [baseline] (95.006 ms) : 0, 95006
Profiling [candidate] (94.125 ms) : 0, 94125
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 20 metrics, 15 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section baseline
no_agent (19.474 ms) : 19277, 19671
. : milestone, 19474,
appsec (18.69 ms) : 18498, 18883
. : milestone, 18690,
code_origins (17.964 ms) : 17787, 18142
. : milestone, 17964,
iast (17.956 ms) : 17782, 18131
. : milestone, 17956,
profiling (18.651 ms) : 18465, 18837
. : milestone, 18651,
tracing (18.064 ms) : 17883, 18245
. : milestone, 18064,
section candidate
no_agent (19.253 ms) : 19058, 19447
. : milestone, 19253,
appsec (18.768 ms) : 18575, 18962
. : milestone, 18768,
code_origins (17.698 ms) : 17525, 17870
. : milestone, 17698,
iast (18.074 ms) : 17896, 18253
. : milestone, 18074,
profiling (18.486 ms) : 18300, 18672
. : milestone, 18486,
tracing (17.868 ms) : 17688, 18047
. : milestone, 17868,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section baseline
no_agent (1.241 ms) : 1228, 1254
. : milestone, 1241,
iast (3.385 ms) : 3335, 3434
. : milestone, 3385,
iast_FULL (5.903 ms) : 5843, 5962
. : milestone, 5903,
iast_GLOBAL (3.756 ms) : 3693, 3819
. : milestone, 3756,
profiling (2.363 ms) : 2341, 2385
. : milestone, 2363,
tracing (1.883 ms) : 1866, 1900
. : milestone, 1883,
section candidate
no_agent (1.262 ms) : 1250, 1275
. : milestone, 1262,
iast (3.259 ms) : 3217, 3302
. : milestone, 3259,
iast_FULL (6.075 ms) : 6013, 6138
. : milestone, 6075,
iast_GLOBAL (3.564 ms) : 3504, 3624
. : milestone, 3564,
profiling (2.488 ms) : 2461, 2516
. : milestone, 2488,
tracing (1.861 ms) : 1846, 1876
. : milestone, 1861,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 2 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section baseline
no_agent (1.495 ms) : 1483, 1506
. : milestone, 1495,
appsec (2.537 ms) : 2483, 2592
. : milestone, 2537,
iast (2.284 ms) : 2214, 2354
. : milestone, 2284,
iast_GLOBAL (2.326 ms) : 2256, 2396
. : milestone, 2326,
profiling (2.528 ms) : 2363, 2693
. : milestone, 2528,
tracing (2.084 ms) : 2031, 2138
. : milestone, 2084,
section candidate
no_agent (1.492 ms) : 1481, 1504
. : milestone, 1492,
appsec (3.835 ms) : 3612, 4057
. : milestone, 3835,
iast (2.284 ms) : 2214, 2354
. : milestone, 2284,
iast_GLOBAL (2.326 ms) : 2256, 2396
. : milestone, 2326,
profiling (2.116 ms) : 2061, 2171
. : milestone, 2116,
tracing (2.082 ms) : 2029, 2136
. : milestone, 2082,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.62.0-SNAPSHOT~bdd3c272f9, baseline=1.62.0-SNAPSHOT~53af2b0381
dateFormat X
axisFormat %s
section baseline
no_agent (15.422 s) : 15422000, 15422000
. : milestone, 15422000,
appsec (14.589 s) : 14589000, 14589000
. : milestone, 14589000,
iast (18.232 s) : 18232000, 18232000
. : milestone, 18232000,
iast_GLOBAL (17.925 s) : 17925000, 17925000
. : milestone, 17925000,
profiling (15.051 s) : 15051000, 15051000
. : milestone, 15051000,
tracing (14.954 s) : 14954000, 14954000
. : milestone, 14954000,
section candidate
no_agent (15.386 s) : 15386000, 15386000
. : milestone, 15386000,
appsec (14.999 s) : 14999000, 14999000
. : milestone, 14999000,
iast (18.389 s) : 18389000, 18389000
. : milestone, 18389000,
iast_GLOBAL (17.77 s) : 17770000, 17770000
. : milestone, 17770000,
profiling (14.86 s) : 14860000, 14860000
. : milestone, 14860000,
tracing (14.985 s) : 14985000, 14985000
. : milestone, 14985000,
|
AlexeyKuznetsov-DD
left a comment
There was a problem hiding this comment.
LGTM, but worth to check if https -> http would help same way as in #11177.
| @@ -1,4 +1,3 @@ | |||
| <!DOCTYPE suite SYSTEM "https://testng.org/testng-1.0.dtd" > | |||
There was a problem hiding this comment.
Yep, recently I faced same issue with this tests failed to load DTD.
But just curious - would it works if replace https with http?
For hibernate that worked, see #11177.
There was a problem hiding this comment.
Great point! Seems like replacing them to http would indeed resolve them locally. Addressed in bdd3c27 to keep the validation but change the URL to do it locally
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
1ce5d90
into
master
What Does This Do
<!DOCTYPE suite SYSTEM …>fromhttps://testng.org/testng-1.0.dtdtohttp://testng.org/testng-1.0.dtdso the XML parser relies on the classpath-bundled DTD.Motivation
TestNG's XML parsers will try to resolve and validate the DTD definition when running the test suites. This is a source of flakiness for the instrumentation tests as the resolution depends on network calls.
Additional Notes
test-environment-trigger: skip
Contributor Checklist
type:and (comp:orinst:) labels in addition to any other useful labelsclose,fix, or any linking keywords when referencing an issueUse
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]
Note: Once your PR is ready to merge, add it to the merge queue by commenting
/merge./merge -ccancels the queue request./merge -f --reason "reason"skips all merge queue checks; please use this judiciously, as some checks do not run at the PR-level. For more information, see this doc.