-
Notifications
You must be signed in to change notification settings - Fork 291
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 log injection smoke test on IBM8 #8126
Draft
mcculls
wants to merge
4
commits into
master
Choose a base branch
from
mcculls/fix-log-injection-test
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
+152
−138
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
mcculls
added
type: bug
comp: testing
Testing
tag: no release notes
Changes to exclude from release notes
run-tests: ibm8
Run tests using IBM 8 JDK
run-tests: all
Run all tests
labels
Dec 20, 2024
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
from
December 20, 2024 19:25
4cf05c3
to
003528e
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 1 performance improvements and 2 performance regressions! Performance is the same for 54 metrics, 6 unstable metrics.
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.054 s) : 0, 1053982
Total [baseline] (10.411 s) : 0, 10410670
Agent [candidate] (1.052 s) : 0, 1052051
Total [candidate] (10.399 s) : 0, 10399282
section appsec
Agent [baseline] (1.194 s) : 0, 1193689
Total [baseline] (10.799 s) : 0, 10799326
Agent [candidate] (1.188 s) : 0, 1187553
Total [candidate] (10.741 s) : 0, 10740643
section iast
Agent [baseline] (1.187 s) : 0, 1186762
Total [baseline] (10.986 s) : 0, 10985755
Agent [candidate] (1.183 s) : 0, 1183264
Total [candidate] (11.035 s) : 0, 11035417
section profiling
Agent [baseline] (1.275 s) : 0, 1274917
Total [baseline] (10.872 s) : 0, 10871591
Agent [candidate] (1.285 s) : 0, 1285407
Total [candidate] (10.865 s) : 0, 10865121
gantt
title petclinic - break down per module: candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (711.285 ms) : 0, 711285
BytebuddyAgent [candidate] (711.681 ms) : 0, 711681
GlobalTracer [baseline] (256.08 ms) : 0, 256080
GlobalTracer [candidate] (256.58 ms) : 0, 256580
AppSec [baseline] (58.412 ms) : 0, 58412
AppSec [candidate] (55.214 ms) : 0, 55214
Remote Config [baseline] (695.151 µs) : 0, 695
Remote Config [candidate] (713.556 µs) : 0, 714
Telemetry [baseline] (12.475 ms) : 0, 12475
Telemetry [candidate] (12.752 ms) : 0, 12752
section appsec
BytebuddyAgent [baseline] (732.152 ms) : 0, 732152
BytebuddyAgent [candidate] (729.553 ms) : 0, 729553
GlobalTracer [baseline] (255.286 ms) : 0, 255286
GlobalTracer [candidate] (253.737 ms) : 0, 253737
AppSec [baseline] (172.636 ms) : 0, 172636
AppSec [candidate] (170.09 ms) : 0, 170090
Remote Config [baseline] (674.188 µs) : 0, 674
Remote Config [candidate] (662.545 µs) : 0, 663
Telemetry [baseline] (8.102 ms) : 0, 8102
Telemetry [candidate] (8.672 ms) : 0, 8672
IAST [baseline] (19.731 ms) : 0, 19731
IAST [candidate] (19.495 ms) : 0, 19495
section iast
BytebuddyAgent [baseline] (834.884 ms) : 0, 834884
BytebuddyAgent [candidate] (829.821 ms) : 0, 829821
GlobalTracer [baseline] (248.087 ms) : 0, 248087
GlobalTracer [candidate] (248.903 ms) : 0, 248903
AppSec [baseline] (58.301 ms) : 0, 58301
AppSec [candidate] (58.31 ms) : 0, 58310
Remote Config [baseline] (665.356 µs) : 0, 665
Remote Config [candidate] (685.832 µs) : 0, 686
Telemetry [baseline] (8.544 ms) : 0, 8544
Telemetry [candidate] (8.659 ms) : 0, 8659
IAST [baseline] (21.275 ms) : 0, 21275
IAST [candidate] (21.717 ms) : 0, 21717
section profiling
ProfilingAgent [baseline] (96.016 ms) : 0, 96016
ProfilingAgent [candidate] (96.709 ms) : 0, 96709
BytebuddyAgent [baseline] (702.565 ms) : 0, 702565
BytebuddyAgent [candidate] (707.577 ms) : 0, 707577
GlobalTracer [baseline] (371.858 ms) : 0, 371858
GlobalTracer [candidate] (375.98 ms) : 0, 375980
AppSec [baseline] (54.2 ms) : 0, 54200
AppSec [candidate] (54.292 ms) : 0, 54292
Remote Config [baseline] (655.951 µs) : 0, 656
Remote Config [candidate] (695.387 µs) : 0, 695
Telemetry [baseline] (7.845 ms) : 0, 7845
Telemetry [candidate] (7.945 ms) : 0, 7945
Profiling [baseline] (96.04 ms) : 0, 96040
Profiling [candidate] (96.733 ms) : 0, 96733
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.056 s) : 0, 1055915
Total [baseline] (8.613 s) : 0, 8612804
Agent [candidate] (1.055 s) : 0, 1055495
Total [candidate] (8.645 s) : 0, 8645307
section iast
Agent [baseline] (1.177 s) : 0, 1177487
Total [baseline] (9.204 s) : 0, 9203551
Agent [candidate] (1.187 s) : 0, 1187412
Total [candidate] (9.261 s) : 0, 9261377
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.178 s) : 0, 1177973
Total [baseline] (9.23 s) : 0, 9230340
Agent [candidate] (1.205 s) : 0, 1205222
Total [candidate] (9.221 s) : 0, 9221296
section iast_TELEMETRY_OFF
Agent [baseline] (1.173 s) : 0, 1173028
Total [baseline] (9.238 s) : 0, 9238201
Agent [candidate] (1.179 s) : 0, 1178600
Total [candidate] (9.222 s) : 0, 9222275
gantt
title insecure-bank - break down per module: candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (713.765 ms) : 0, 713765
BytebuddyAgent [candidate] (713.248 ms) : 0, 713248
GlobalTracer [baseline] (256.16 ms) : 0, 256160
GlobalTracer [candidate] (257.782 ms) : 0, 257782
AppSec [baseline] (59.296 ms) : 0, 59296
AppSec [candidate] (55.165 ms) : 0, 55165
Remote Config [baseline] (684.669 µs) : 0, 685
Remote Config [candidate] (724.121 µs) : 0, 724
Telemetry [baseline] (11.129 ms) : 0, 11129
Telemetry [candidate] (13.463 ms) : 0, 13463
section iast
BytebuddyAgent [baseline] (828.548 ms) : 0, 828548
BytebuddyAgent [candidate] (834.58 ms) : 0, 834580
GlobalTracer [baseline] (246.108 ms) : 0, 246108
GlobalTracer [candidate] (248.501 ms) : 0, 248501
AppSec [baseline] (57.719 ms) : 0, 57719
AppSec [candidate] (58.252 ms) : 0, 58252
IAST [baseline] (21.126 ms) : 0, 21126
IAST [candidate] (21.665 ms) : 0, 21665
Remote Config [baseline] (643.635 µs) : 0, 644
Remote Config [candidate] (667.204 µs) : 0, 667
Telemetry [baseline] (8.487 ms) : 0, 8487
Telemetry [candidate] (8.689 ms) : 0, 8689
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (828.263 ms) : 0, 828263
BytebuddyAgent [candidate] (847.138 ms) : 0, 847138
GlobalTracer [baseline] (246.575 ms) : 0, 246575
GlobalTracer [candidate] (251.541 ms) : 0, 251541
AppSec [baseline] (57.824 ms) : 0, 57824
AppSec [candidate] (59.478 ms) : 0, 59478
IAST [baseline] (21.235 ms) : 0, 21235
IAST [candidate] (22.148 ms) : 0, 22148
Remote Config [baseline] (659.909 µs) : 0, 660
Remote Config [candidate] (700.001 µs) : 0, 700
Telemetry [baseline] (8.48 ms) : 0, 8480
Telemetry [candidate] (8.843 ms) : 0, 8843
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (824.507 ms) : 0, 824507
BytebuddyAgent [candidate] (828.274 ms) : 0, 828274
GlobalTracer [baseline] (246.29 ms) : 0, 246290
GlobalTracer [candidate] (247.484 ms) : 0, 247484
AppSec [baseline] (57.854 ms) : 0, 57854
AppSec [candidate] (57.831 ms) : 0, 57831
IAST [baseline] (20.556 ms) : 0, 20556
IAST [candidate] (20.85 ms) : 0, 20850
Remote Config [baseline] (648.85 µs) : 0, 649
Remote Config [candidate] (662.444 µs) : 0, 662
Telemetry [baseline] (8.244 ms) : 0, 8244
Telemetry [candidate] (8.387 ms) : 0, 8387
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 16 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section baseline
no_agent (383.525 µs) : 363, 404
. : milestone, 384,
iast (500.743 µs) : 479, 523
. : milestone, 501,
iast_FULL (658.89 µs) : 637, 681
. : milestone, 659,
iast_GLOBAL (527.133 µs) : 505, 549
. : milestone, 527,
iast_HARDCODED_SECRET_DISABLED (491.381 µs) : 470, 513
. : milestone, 491,
iast_INACTIVE (456.995 µs) : 435, 479
. : milestone, 457,
iast_TELEMETRY_OFF (490.167 µs) : 469, 512
. : milestone, 490,
tracing (449.404 µs) : 428, 470
. : milestone, 449,
section candidate
no_agent (372.236 µs) : 352, 392
. : milestone, 372,
iast (503.279 µs) : 481, 526
. : milestone, 503,
iast_FULL (651.657 µs) : 630, 673
. : milestone, 652,
iast_GLOBAL (523.747 µs) : 502, 546
. : milestone, 524,
iast_HARDCODED_SECRET_DISABLED (492.9 µs) : 472, 514
. : milestone, 493,
iast_INACTIVE (461.866 µs) : 440, 484
. : milestone, 462,
iast_TELEMETRY_OFF (483.976 µs) : 462, 506
. : milestone, 484,
tracing (443.989 µs) : 423, 465
. : milestone, 444,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section baseline
no_agent (1.335 ms) : 1316, 1354
. : milestone, 1335,
appsec (1.736 ms) : 1712, 1760
. : milestone, 1736,
appsec_no_iast (1.745 ms) : 1720, 1769
. : milestone, 1745,
iast (1.502 ms) : 1479, 1525
. : milestone, 1502,
profiling (1.584 ms) : 1560, 1608
. : milestone, 1584,
tracing (1.476 ms) : 1451, 1501
. : milestone, 1476,
section candidate
no_agent (1.365 ms) : 1346, 1384
. : milestone, 1365,
appsec (1.745 ms) : 1721, 1769
. : milestone, 1745,
appsec_no_iast (1.761 ms) : 1737, 1785
. : milestone, 1761,
iast (1.5 ms) : 1477, 1523
. : milestone, 1500,
profiling (1.515 ms) : 1492, 1537
. : milestone, 1515,
tracing (1.478 ms) : 1454, 1503
. : milestone, 1478,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section baseline
no_agent (15.478 s) : 15478000, 15478000
. : milestone, 15478000,
appsec (15.02 s) : 15020000, 15020000
. : milestone, 15020000,
iast (19.016 s) : 19016000, 19016000
. : milestone, 19016000,
iast_GLOBAL (18.027 s) : 18027000, 18027000
. : milestone, 18027000,
profiling (14.963 s) : 14963000, 14963000
. : milestone, 14963000,
tracing (14.968 s) : 14968000, 14968000
. : milestone, 14968000,
section candidate
no_agent (15.444 s) : 15444000, 15444000
. : milestone, 15444000,
appsec (15.13 s) : 15130000, 15130000
. : milestone, 15130000,
iast (19.126 s) : 19126000, 19126000
. : milestone, 19126000,
iast_GLOBAL (17.579 s) : 17579000, 17579000
. : milestone, 17579000,
profiling (15.128 s) : 15128000, 15128000
. : milestone, 15128000,
tracing (15.147 s) : 15147000, 15147000
. : milestone, 15147000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.45.0-SNAPSHOT~ffca9bcb63, baseline=1.45.0-SNAPSHOT~46b5986f6a
dateFormat X
axisFormat %s
section baseline
no_agent (1.478 ms) : 1466, 1489
. : milestone, 1478,
appsec (2.359 ms) : 2317, 2401
. : milestone, 2359,
iast (2.099 ms) : 2046, 2152
. : milestone, 2099,
iast_GLOBAL (2.151 ms) : 2098, 2205
. : milestone, 2151,
profiling (2.413 ms) : 2236, 2590
. : milestone, 2413,
tracing (1.936 ms) : 1896, 1977
. : milestone, 1936,
section candidate
no_agent (1.475 ms) : 1464, 1487
. : milestone, 1475,
appsec (2.358 ms) : 2316, 2400
. : milestone, 2358,
iast (2.101 ms) : 2048, 2154
. : milestone, 2101,
iast_GLOBAL (2.148 ms) : 2094, 2201
. : milestone, 2148,
profiling (1.972 ms) : 1930, 2014
. : milestone, 1972,
tracing (1.94 ms) : 1899, 1980
. : milestone, 1940,
|
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
from
December 21, 2024 18:50
ed01097
to
a77a5e1
Compare
mcculls
changed the title
Wait for initial config to appear before starting test
Fix log injection smoke test on IBM8
Dec 21, 2024
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
10 times, most recently
from
December 23, 2024 23:51
2279f72
to
d3ed23d
Compare
or JMX builder was on the system classpath (because the main thread would find it there if OkHttp triggered initialization of JUL.). We now make OkHttp calls from our own background threads, which are isolated from the system classloader, not the main thread - so this exemption no longer makes sense.
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
5 times, most recently
from
December 28, 2024 00:55
e9292e0
to
f1ff830
Compare
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
from
December 28, 2024 01:06
f1ff830
to
fe39745
Compare
mcculls
force-pushed
the
mcculls/fix-log-injection-test
branch
from
December 28, 2024 03:07
551a255
to
ffca9bc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
comp: testing
Testing
run-tests: ibm8
Run tests using IBM 8 JDK
tag: no release notes
Changes to exclude from release notes
type: bug
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
Motivation
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]