-
Notifications
You must be signed in to change notification settings - Fork 292
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
Improve crashtracking support for older Bash versions #7956
Merged
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
…orted Associative arrays require Bash 4. I replaced them with dynamic variable names.
PerfectSlayer
changed the title
Improve crashtracking support for older Bash version
Improve crashtracking support for older Bash versions
Nov 14, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 10 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.083 s) : 0, 1082905
Total [baseline] (10.412 s) : 0, 10411561
Agent [candidate] (1.081 s) : 0, 1080830
Total [candidate] (10.407 s) : 0, 10407279
section appsec
Agent [baseline] (1.215 s) : 0, 1214781
Total [baseline] (10.695 s) : 0, 10694781
Agent [candidate] (1.218 s) : 0, 1218006
Total [candidate] (10.68 s) : 0, 10679706
section iast
Agent [baseline] (1.207 s) : 0, 1207474
Total [baseline] (10.958 s) : 0, 10958035
Agent [candidate] (1.208 s) : 0, 1208308
Total [candidate] (10.88 s) : 0, 10880494
section profiling
Agent [baseline] (1.278 s) : 0, 1277732
Total [baseline] (10.685 s) : 0, 10685019
Agent [candidate] (1.296 s) : 0, 1295991
Total [candidate] (10.835 s) : 0, 10835475
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.892 ms) : 0, 688892
BytebuddyAgent [candidate] (686.741 ms) : 0, 686741
GlobalTracer [baseline] (316.01 ms) : 0, 316010
GlobalTracer [candidate] (315.356 ms) : 0, 315356
AppSec [baseline] (54.331 ms) : 0, 54331
AppSec [candidate] (54.252 ms) : 0, 54252
Remote Config [baseline] (675.945 µs) : 0, 676
Remote Config [candidate] (682.451 µs) : 0, 682
Telemetry [baseline] (9.191 ms) : 0, 9191
Telemetry [candidate] (10.06 ms) : 0, 10060
section appsec
BytebuddyAgent [baseline] (703.694 ms) : 0, 703694
BytebuddyAgent [candidate] (706.259 ms) : 0, 706259
GlobalTracer [baseline] (312.503 ms) : 0, 312503
GlobalTracer [candidate] (313.941 ms) : 0, 313941
AppSec [baseline] (166.329 ms) : 0, 166329
AppSec [candidate] (165.156 ms) : 0, 165156
Remote Config [baseline] (632.639 µs) : 0, 633
Remote Config [candidate] (630.886 µs) : 0, 631
Telemetry [baseline] (7.806 ms) : 0, 7806
Telemetry [candidate] (7.767 ms) : 0, 7767
IAST [baseline] (20.264 ms) : 0, 20264
IAST [candidate] (21.008 ms) : 0, 21008
section iast
BytebuddyAgent [baseline] (802.525 ms) : 0, 802525
BytebuddyAgent [candidate] (803.716 ms) : 0, 803716
GlobalTracer [baseline] (305.048 ms) : 0, 305048
GlobalTracer [candidate] (304.696 ms) : 0, 304696
AppSec [baseline] (56.708 ms) : 0, 56708
AppSec [candidate] (57.383 ms) : 0, 57383
Remote Config [baseline] (612.609 µs) : 0, 613
Remote Config [candidate] (611.966 µs) : 0, 612
Telemetry [baseline] (7.485 ms) : 0, 7485
Telemetry [candidate] (7.462 ms) : 0, 7462
IAST [baseline] (21.375 ms) : 0, 21375
IAST [candidate] (20.673 ms) : 0, 20673
section profiling
ProfilingAgent [baseline] (92.831 ms) : 0, 92831
ProfilingAgent [candidate] (93.402 ms) : 0, 93402
BytebuddyAgent [baseline] (680.166 ms) : 0, 680166
BytebuddyAgent [candidate] (689.281 ms) : 0, 689281
GlobalTracer [baseline] (398.641 ms) : 0, 398641
GlobalTracer [candidate] (404.447 ms) : 0, 404447
AppSec [baseline] (54.567 ms) : 0, 54567
AppSec [candidate] (55.763 ms) : 0, 55763
Remote Config [baseline] (669.584 µs) : 0, 670
Remote Config [candidate] (683.889 µs) : 0, 684
Telemetry [baseline] (11.894 ms) : 0, 11894
Telemetry [candidate] (12.965 ms) : 0, 12965
Profiling [baseline] (92.854 ms) : 0, 92854
Profiling [candidate] (93.428 ms) : 0, 93428
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1083749
Total [baseline] (8.589 s) : 0, 8588692
Agent [candidate] (1.084 s) : 0, 1083741
Total [candidate] (8.596 s) : 0, 8595581
section iast
Agent [baseline] (1.207 s) : 0, 1206994
Total [baseline] (9.181 s) : 0, 9181446
Agent [candidate] (1.217 s) : 0, 1217298
Total [candidate] (9.162 s) : 0, 9162184
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.208 s) : 0, 1207558
Total [baseline] (9.122 s) : 0, 9121657
Agent [candidate] (1.208 s) : 0, 1208402
Total [candidate] (9.102 s) : 0, 9101795
section iast_TELEMETRY_OFF
Agent [baseline] (1.206 s) : 0, 1206103
Total [baseline] (9.116 s) : 0, 9115913
Agent [candidate] (1.213 s) : 0, 1213012
Total [candidate] (9.137 s) : 0, 9136845
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.955 ms) : 0, 687955
BytebuddyAgent [candidate] (686.48 ms) : 0, 686480
GlobalTracer [baseline] (315.525 ms) : 0, 315525
GlobalTracer [candidate] (317.042 ms) : 0, 317042
AppSec [baseline] (54.336 ms) : 0, 54336
AppSec [candidate] (54.58 ms) : 0, 54580
Remote Config [baseline] (684.879 µs) : 0, 685
Remote Config [candidate] (678.692 µs) : 0, 679
Telemetry [baseline] (11.519 ms) : 0, 11519
Telemetry [candidate] (11.18 ms) : 0, 11180
section iast
BytebuddyAgent [baseline] (802.354 ms) : 0, 802354
BytebuddyAgent [candidate] (810.224 ms) : 0, 810224
GlobalTracer [baseline] (304.068 ms) : 0, 304068
GlobalTracer [candidate] (306.667 ms) : 0, 306667
AppSec [baseline] (58.142 ms) : 0, 58142
AppSec [candidate] (56.769 ms) : 0, 56769
IAST [baseline] (20.534 ms) : 0, 20534
IAST [candidate] (21.64 ms) : 0, 21640
Remote Config [baseline] (615.591 µs) : 0, 616
Remote Config [candidate] (619.278 µs) : 0, 619
Telemetry [baseline] (7.551 ms) : 0, 7551
Telemetry [candidate] (7.469 ms) : 0, 7469
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (803.149 ms) : 0, 803149
BytebuddyAgent [candidate] (802.863 ms) : 0, 802863
GlobalTracer [baseline] (304.437 ms) : 0, 304437
GlobalTracer [candidate] (305.008 ms) : 0, 305008
AppSec [baseline] (57.377 ms) : 0, 57377
AppSec [candidate] (57.817 ms) : 0, 57817
IAST [baseline] (20.69 ms) : 0, 20690
IAST [candidate] (20.765 ms) : 0, 20765
Remote Config [baseline] (618.696 µs) : 0, 619
Remote Config [candidate] (612.811 µs) : 0, 613
Telemetry [baseline] (7.523 ms) : 0, 7523
Telemetry [candidate] (7.533 ms) : 0, 7533
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (801.068 ms) : 0, 801068
BytebuddyAgent [candidate] (806.118 ms) : 0, 806118
GlobalTracer [baseline] (304.855 ms) : 0, 304855
GlobalTracer [candidate] (306.178 ms) : 0, 306178
AppSec [baseline] (57.525 ms) : 0, 57525
AppSec [candidate] (58.299 ms) : 0, 58299
IAST [baseline] (20.944 ms) : 0, 20944
IAST [candidate] (20.454 ms) : 0, 20454
Remote Config [baseline] (593.101 µs) : 0, 593
Remote Config [candidate] (619.775 µs) : 0, 620
Telemetry [baseline] (7.353 ms) : 0, 7353
Telemetry [candidate] (7.465 ms) : 0, 7465
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section baseline
no_agent (370.995 µs) : 351, 391
. : milestone, 371,
iast (484.836 µs) : 463, 506
. : milestone, 485,
iast_FULL (639.891 µs) : 618, 661
. : milestone, 640,
iast_GLOBAL (508.261 µs) : 487, 529
. : milestone, 508,
iast_HARDCODED_SECRET_DISABLED (481.765 µs) : 461, 503
. : milestone, 482,
iast_INACTIVE (440.774 µs) : 420, 461
. : milestone, 441,
iast_TELEMETRY_OFF (478.878 µs) : 457, 501
. : milestone, 479,
tracing (441.783 µs) : 421, 462
. : milestone, 442,
section candidate
no_agent (368.04 µs) : 348, 388
. : milestone, 368,
iast (487.114 µs) : 465, 509
. : milestone, 487,
iast_FULL (650.305 µs) : 629, 672
. : milestone, 650,
iast_GLOBAL (513.33 µs) : 491, 536
. : milestone, 513,
iast_HARDCODED_SECRET_DISABLED (493.296 µs) : 472, 515
. : milestone, 493,
iast_INACTIVE (441.855 µs) : 421, 462
. : milestone, 442,
iast_TELEMETRY_OFF (475.163 µs) : 454, 497
. : milestone, 475,
tracing (440.364 µs) : 420, 461
. : milestone, 440,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section baseline
no_agent (1.331 ms) : 1311, 1351
. : milestone, 1331,
appsec (1.726 ms) : 1704, 1749
. : milestone, 1726,
appsec_no_iast (1.738 ms) : 1714, 1761
. : milestone, 1738,
iast (1.472 ms) : 1449, 1495
. : milestone, 1472,
profiling (1.495 ms) : 1472, 1518
. : milestone, 1495,
tracing (1.493 ms) : 1469, 1517
. : milestone, 1493,
section candidate
no_agent (1.355 ms) : 1336, 1374
. : milestone, 1355,
appsec (1.714 ms) : 1689, 1739
. : milestone, 1714,
appsec_no_iast (1.722 ms) : 1697, 1747
. : milestone, 1722,
iast (1.5 ms) : 1478, 1522
. : milestone, 1500,
profiling (1.508 ms) : 1483, 1533
. : milestone, 1508,
tracing (1.449 ms) : 1424, 1474
. : milestone, 1449,
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.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section baseline
no_agent (1.464 ms) : 1452, 1475
. : milestone, 1464,
appsec (2.333 ms) : 2292, 2375
. : milestone, 2333,
iast (2.074 ms) : 2022, 2126
. : milestone, 2074,
iast_GLOBAL (2.124 ms) : 2072, 2176
. : milestone, 2124,
profiling (1.944 ms) : 1902, 1986
. : milestone, 1944,
tracing (1.92 ms) : 1880, 1960
. : milestone, 1920,
section candidate
no_agent (1.464 ms) : 1453, 1476
. : milestone, 1464,
appsec (2.321 ms) : 2280, 2361
. : milestone, 2321,
iast (2.073 ms) : 2021, 2125
. : milestone, 2073,
iast_GLOBAL (2.106 ms) : 2054, 2158
. : milestone, 2106,
profiling (1.925 ms) : 1884, 1966
. : milestone, 1925,
tracing (1.922 ms) : 1882, 1962
. : milestone, 1922,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~9a707fd11e, baseline=1.43.0-SNAPSHOT~ea008a2aad
dateFormat X
axisFormat %s
section baseline
no_agent (15.721 s) : 15721000, 15721000
. : milestone, 15721000,
appsec (15.373 s) : 15373000, 15373000
. : milestone, 15373000,
iast (19.173 s) : 19173000, 19173000
. : milestone, 19173000,
iast_GLOBAL (18.262 s) : 18262000, 18262000
. : milestone, 18262000,
profiling (15.032 s) : 15032000, 15032000
. : milestone, 15032000,
tracing (15.251 s) : 15251000, 15251000
. : milestone, 15251000,
section candidate
no_agent (14.98 s) : 14980000, 14980000
. : milestone, 14980000,
appsec (15.166 s) : 15166000, 15166000
. : milestone, 15166000,
iast (18.924 s) : 18924000, 18924000
. : milestone, 18924000,
iast_GLOBAL (18.085 s) : 18085000, 18085000
. : milestone, 18085000,
profiling (15.832 s) : 15832000, 15832000
. : milestone, 15832000,
tracing (15.239 s) : 15239000, 15239000
. : milestone, 15239000,
|
jbachorik
approved these changes
Nov 21, 2024
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.
What Does This Do
This PR fixes case when Bash associative array are not supported as associative arrays require Bash 4.
I replaced them with dynamic variable names.
Motivation
It was reported by @dougqh encountering issues with its dev environment.
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]