-
Notifications
You must be signed in to change notification settings - Fork 324
Fixing pekko test: revert previous change and fix test server #9530
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
|
🎯 Code Coverage 🔗 Commit SHA: fc8d09b | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 11 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.063 s) : 0, 1062548
Total [baseline] (10.685 s) : 0, 10684724
Agent [candidate] (1.063 s) : 0, 1063320
Total [candidate] (10.67 s) : 0, 10669954
section appsec
Agent [baseline] (1.238 s) : 0, 1237710
Total [baseline] (10.974 s) : 0, 10973927
Agent [candidate] (1.237 s) : 0, 1237162
Total [candidate] (10.938 s) : 0, 10938222
section iast
Agent [baseline] (1.2 s) : 0, 1199616
Total [baseline] (11.087 s) : 0, 11087137
Agent [candidate] (1.202 s) : 0, 1202186
Total [candidate] (11.125 s) : 0, 11124778
section profiling
Agent [baseline] (1.219 s) : 0, 1219026
Total [baseline] (11.094 s) : 0, 11094139
Agent [candidate] (1.218 s) : 0, 1218388
Total [candidate] (11.043 s) : 0, 11043033
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.458 ms) : 0, 1458
crashtracking [candidate] (1.455 ms) : 0, 1455
BytebuddyAgent [baseline] (733.183 ms) : 0, 733183
BytebuddyAgent [candidate] (733.522 ms) : 0, 733522
GlobalTracer [baseline] (252.736 ms) : 0, 252736
GlobalTracer [candidate] (252.992 ms) : 0, 252992
AppSec [baseline] (30.549 ms) : 0, 30549
AppSec [candidate] (30.638 ms) : 0, 30638
Debugger [baseline] (6.411 ms) : 0, 6411
Debugger [candidate] (6.353 ms) : 0, 6353
Remote Config [baseline] (702.284 µs) : 0, 702
Remote Config [candidate] (695.307 µs) : 0, 695
Telemetry [baseline] (16.4 ms) : 0, 16400
Telemetry [candidate] (16.418 ms) : 0, 16418
section appsec
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.456 ms) : 0, 1456
BytebuddyAgent [baseline] (757.681 ms) : 0, 757681
BytebuddyAgent [candidate] (757.272 ms) : 0, 757272
GlobalTracer [baseline] (246.266 ms) : 0, 246266
GlobalTracer [candidate] (246.283 ms) : 0, 246283
AppSec [baseline] (170.408 ms) : 0, 170408
AppSec [candidate] (171.981 ms) : 0, 171981
Debugger [baseline] (6.832 ms) : 0, 6832
Debugger [candidate] (5.992 ms) : 0, 5992
Remote Config [baseline] (639.214 µs) : 0, 639
Remote Config [candidate] (635.031 µs) : 0, 635
Telemetry [baseline] (9.196 ms) : 0, 9196
Telemetry [candidate] (8.442 ms) : 0, 8442
IAST [baseline] (23.868 ms) : 0, 23868
IAST [candidate] (23.839 ms) : 0, 23839
section iast
crashtracking [baseline] (1.476 ms) : 0, 1476
crashtracking [candidate] (1.485 ms) : 0, 1485
BytebuddyAgent [baseline] (859.104 ms) : 0, 859104
BytebuddyAgent [candidate] (861.475 ms) : 0, 861475
GlobalTracer [baseline] (248.056 ms) : 0, 248056
GlobalTracer [candidate] (248.788 ms) : 0, 248788
AppSec [baseline] (25.745 ms) : 0, 25745
AppSec [candidate] (26.596 ms) : 0, 26596
Debugger [baseline] (6.11 ms) : 0, 6110
Debugger [candidate] (6.086 ms) : 0, 6086
Remote Config [baseline] (605.939 µs) : 0, 606
Remote Config [candidate] (617.126 µs) : 0, 617
Telemetry [baseline] (8.281 ms) : 0, 8281
Telemetry [candidate] (8.357 ms) : 0, 8357
IAST [baseline] (29.043 ms) : 0, 29043
IAST [candidate] (27.532 ms) : 0, 27532
section profiling
ProfilingAgent [baseline] (107.863 ms) : 0, 107863
ProfilingAgent [candidate] (109.413 ms) : 0, 109413
crashtracking [baseline] (1.447 ms) : 0, 1447
crashtracking [candidate] (1.472 ms) : 0, 1472
BytebuddyAgent [baseline] (768.227 ms) : 0, 768227
BytebuddyAgent [candidate] (767.56 ms) : 0, 767560
GlobalTracer [baseline] (235.345 ms) : 0, 235345
GlobalTracer [candidate] (234.285 ms) : 0, 234285
AppSec [baseline] (30.628 ms) : 0, 30628
AppSec [candidate] (31.131 ms) : 0, 31131
Debugger [baseline] (7.485 ms) : 0, 7485
Debugger [candidate] (7.484 ms) : 0, 7484
Remote Config [baseline] (746.023 µs) : 0, 746
Remote Config [candidate] (719.69 µs) : 0, 720
Telemetry [baseline] (15.848 ms) : 0, 15848
Telemetry [candidate] (14.898 ms) : 0, 14898
Profiling [baseline] (108.528 ms) : 0, 108528
Profiling [candidate] (110.077 ms) : 0, 110077
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.059 s) : 0, 1059430
Total [baseline] (8.652 s) : 0, 8652281
Agent [candidate] (1.063 s) : 0, 1062824
Total [candidate] (8.663 s) : 0, 8662672
section iast
Agent [baseline] (1.195 s) : 0, 1195178
Total [baseline] (9.307 s) : 0, 9307195
Agent [candidate] (1.193 s) : 0, 1192739
Total [candidate] (9.357 s) : 0, 9357165
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.457 ms) : 0, 1457
crashtracking [candidate] (1.449 ms) : 0, 1449
BytebuddyAgent [baseline] (731.339 ms) : 0, 731339
BytebuddyAgent [candidate] (733.86 ms) : 0, 733860
GlobalTracer [baseline] (251.922 ms) : 0, 251922
GlobalTracer [candidate] (252.898 ms) : 0, 252898
AppSec [baseline] (30.523 ms) : 0, 30523
AppSec [candidate] (30.549 ms) : 0, 30549
Debugger [baseline] (6.414 ms) : 0, 6414
Debugger [candidate] (6.391 ms) : 0, 6391
Remote Config [baseline] (697.265 µs) : 0, 697
Remote Config [candidate] (687.351 µs) : 0, 687
Telemetry [baseline] (16.05 ms) : 0, 16050
Telemetry [candidate] (15.948 ms) : 0, 15948
section iast
crashtracking [baseline] (1.47 ms) : 0, 1470
crashtracking [candidate] (1.456 ms) : 0, 1456
BytebuddyAgent [baseline] (855.39 ms) : 0, 855390
BytebuddyAgent [candidate] (852.108 ms) : 0, 852108
GlobalTracer [baseline] (245.106 ms) : 0, 245106
GlobalTracer [candidate] (246.229 ms) : 0, 246229
AppSec [baseline] (26.547 ms) : 0, 26547
AppSec [candidate] (27.692 ms) : 0, 27692
Debugger [baseline] (6.119 ms) : 0, 6119
Debugger [candidate] (6.95 ms) : 0, 6950
Remote Config [baseline] (613.032 µs) : 0, 613
Remote Config [candidate] (603.378 µs) : 0, 603
Telemetry [baseline] (8.224 ms) : 0, 8224
Telemetry [candidate] (8.282 ms) : 0, 8282
IAST [baseline] (30.657 ms) : 0, 30657
IAST [candidate] (28.401 ms) : 0, 28401
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 2 performance regressions! Performance is the same for 8 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section baseline
no_agent (4.269 ms) : 4221, 4316
. : milestone, 4269,
iast (9.533 ms) : 9377, 9690
. : milestone, 9533,
iast_FULL (14.578 ms) : 14288, 14868
. : milestone, 14578,
iast_GLOBAL (10.722 ms) : 10518, 10925
. : milestone, 10722,
profiling (8.977 ms) : 8826, 9128
. : milestone, 8977,
tracing (7.547 ms) : 7439, 7655
. : milestone, 7547,
section candidate
no_agent (4.49 ms) : 4434, 4547
. : milestone, 4490,
iast (9.757 ms) : 9588, 9925
. : milestone, 9757,
iast_FULL (13.951 ms) : 13670, 14232
. : milestone, 13951,
iast_GLOBAL (10.27 ms) : 10088, 10452
. : milestone, 10270,
profiling (8.817 ms) : 8672, 8962
. : milestone, 8817,
tracing (7.574 ms) : 7466, 7681
. : milestone, 7574,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section baseline
no_agent (36.799 ms) : 36503, 37095
. : milestone, 36799,
appsec (48.405 ms) : 47972, 48839
. : milestone, 48405,
code_origins (45.108 ms) : 44720, 45496
. : milestone, 45108,
iast (43.359 ms) : 42986, 43732
. : milestone, 43359,
profiling (48.119 ms) : 47641, 48596
. : milestone, 48119,
tracing (42.99 ms) : 42626, 43354
. : milestone, 42990,
section candidate
no_agent (37.73 ms) : 37422, 38038
. : milestone, 37730,
appsec (50.618 ms) : 50178, 51058
. : milestone, 50618,
code_origins (45.605 ms) : 45224, 45985
. : milestone, 45605,
iast (43.223 ms) : 42864, 43582
. : milestone, 43223,
profiling (47.334 ms) : 46898, 47769
. : milestone, 47334,
tracing (43.792 ms) : 43418, 44167
. : milestone, 43792,
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.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section baseline
no_agent (15.414 s) : 15414000, 15414000
. : milestone, 15414000,
appsec (15.009 s) : 15009000, 15009000
. : milestone, 15009000,
iast (18.513 s) : 18513000, 18513000
. : milestone, 18513000,
iast_GLOBAL (18.218 s) : 18218000, 18218000
. : milestone, 18218000,
profiling (16.034 s) : 16034000, 16034000
. : milestone, 16034000,
tracing (15.12 s) : 15120000, 15120000
. : milestone, 15120000,
section candidate
no_agent (15.295 s) : 15295000, 15295000
. : milestone, 15295000,
appsec (15.031 s) : 15031000, 15031000
. : milestone, 15031000,
iast (18.212 s) : 18212000, 18212000
. : milestone, 18212000,
iast_GLOBAL (18.055 s) : 18055000, 18055000
. : milestone, 18055000,
profiling (15.244 s) : 15244000, 15244000
. : milestone, 15244000,
tracing (15.001 s) : 15001000, 15001000
. : milestone, 15001000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~fc8d09bc7a, baseline=1.54.0-SNAPSHOT~81cdab0f050
dateFormat X
axisFormat %s
section baseline
no_agent (1.483 ms) : 1471, 1494
. : milestone, 1483,
appsec (3.741 ms) : 3517, 3965
. : milestone, 3741,
iast (2.205 ms) : 2142, 2268
. : milestone, 2205,
iast_GLOBAL (2.253 ms) : 2190, 2317
. : milestone, 2253,
profiling (2.074 ms) : 2022, 2126
. : milestone, 2074,
tracing (2.031 ms) : 1982, 2080
. : milestone, 2031,
section candidate
no_agent (1.486 ms) : 1474, 1497
. : milestone, 1486,
appsec (3.719 ms) : 3500, 3938
. : milestone, 3719,
iast (2.211 ms) : 2147, 2274
. : milestone, 2211,
iast_GLOBAL (2.262 ms) : 2199, 2325
. : milestone, 2262,
profiling (2.036 ms) : 1986, 2087
. : milestone, 2036,
tracing (2.038 ms) : 1989, 2088
. : milestone, 2038,
|
manuel-alvarez-alvarez
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What Does This Do
Motivation
honestly this is a bit beyond my comprehension, and AI helped me find this, but it seems there is a race condition in which handler gets to treat the exception, if it's the instrumentation handler things work fin, but if it's the test server handler, then the span is not finished. At least that's how I understand it.
Additional Notes
Pekko's instrumentation was submitted by an external contributor 2 years ago.
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]