-
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
Fix crashtracking log parser #7697
Merged
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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 9 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.071 s) : 0, 1071446
Total [baseline] (8.577 s) : 0, 8576677
Agent [candidate] (1.077 s) : 0, 1076687
Total [candidate] (8.56 s) : 0, 8559852
section iast
Agent [baseline] (1.203 s) : 0, 1202956
Total [baseline] (9.098 s) : 0, 9098206
Agent [candidate] (1.194 s) : 0, 1194177
Total [candidate] (9.09 s) : 0, 9089602
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.196 s) : 0, 1195577
Total [baseline] (9.1 s) : 0, 9100190
Agent [candidate] (1.205 s) : 0, 1204594
Total [candidate] (9.1 s) : 0, 9100408
section iast_TELEMETRY_OFF
Agent [baseline] (1.193 s) : 0, 1192630
Total [baseline] (9.077 s) : 0, 9077191
Agent [candidate] (1.193 s) : 0, 1192535
Total [candidate] (9.131 s) : 0, 9131229
gantt
title insecure-bank - break down per module: candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (683.894 ms) : 0, 683894
BytebuddyAgent [candidate] (687.104 ms) : 0, 687104
GlobalTracer [baseline] (311.878 ms) : 0, 311878
GlobalTracer [candidate] (313.057 ms) : 0, 313057
AppSec [baseline] (53.649 ms) : 0, 53649
AppSec [candidate] (54.316 ms) : 0, 54316
Remote Config [baseline] (659.841 µs) : 0, 660
Remote Config [candidate] (663.591 µs) : 0, 664
Telemetry [baseline] (7.637 ms) : 0, 7637
Telemetry [candidate] (7.713 ms) : 0, 7713
section iast
BytebuddyAgent [baseline] (801.49 ms) : 0, 801490
BytebuddyAgent [candidate] (795.108 ms) : 0, 795108
GlobalTracer [baseline] (301.793 ms) : 0, 301793
GlobalTracer [candidate] (300.282 ms) : 0, 300282
AppSec [baseline] (55.178 ms) : 0, 55178
AppSec [candidate] (56.349 ms) : 0, 56349
IAST [baseline] (22.917 ms) : 0, 22917
IAST [candidate] (20.988 ms) : 0, 20988
Remote Config [baseline] (620.744 µs) : 0, 621
Remote Config [candidate] (627.626 µs) : 0, 628
Telemetry [baseline] (7.121 ms) : 0, 7121
Telemetry [candidate] (7.072 ms) : 0, 7072
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (795.748 ms) : 0, 795748
BytebuddyAgent [candidate] (802.021 ms) : 0, 802021
GlobalTracer [baseline] (299.86 ms) : 0, 299860
GlobalTracer [candidate] (302.874 ms) : 0, 302874
AppSec [baseline] (52.519 ms) : 0, 52519
AppSec [candidate] (55.571 ms) : 0, 55571
IAST [baseline] (25.939 ms) : 0, 25939
IAST [candidate] (22.578 ms) : 0, 22578
Remote Config [baseline] (627.989 µs) : 0, 628
Remote Config [candidate] (612.308 µs) : 0, 612
Telemetry [baseline] (7.095 ms) : 0, 7095
Telemetry [candidate] (7.066 ms) : 0, 7066
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (792.869 ms) : 0, 792869
BytebuddyAgent [candidate] (792.288 ms) : 0, 792288
GlobalTracer [baseline] (300.359 ms) : 0, 300359
GlobalTracer [candidate] (300.652 ms) : 0, 300652
AppSec [baseline] (54.985 ms) : 0, 54985
AppSec [candidate] (56.327 ms) : 0, 56327
IAST [baseline] (23.192 ms) : 0, 23192
IAST [candidate] (21.87 ms) : 0, 21870
Remote Config [baseline] (606.666 µs) : 0, 607
Remote Config [candidate] (619.816 µs) : 0, 620
Telemetry [baseline] (6.884 ms) : 0, 6884
Telemetry [candidate] (6.983 ms) : 0, 6983
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.078 s) : 0, 1078032
Total [baseline] (10.462 s) : 0, 10462088
Agent [candidate] (1.071 s) : 0, 1070943
Total [candidate] (10.424 s) : 0, 10423705
section appsec
Agent [baseline] (1.204 s) : 0, 1203902
Total [baseline] (10.594 s) : 0, 10594262
Agent [candidate] (1.204 s) : 0, 1203794
Total [candidate] (10.623 s) : 0, 10622604
section iast
Agent [baseline] (1.196 s) : 0, 1196487
Total [baseline] (10.816 s) : 0, 10816139
Agent [candidate] (1.206 s) : 0, 1205978
Total [candidate] (10.933 s) : 0, 10932824
section profiling
Agent [baseline] (1.266 s) : 0, 1266009
Total [baseline] (10.664 s) : 0, 10663831
Agent [candidate] (1.267 s) : 0, 1267485
Total [candidate] (10.604 s) : 0, 10604402
gantt
title petclinic - break down per module: candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.244 ms) : 0, 686244
BytebuddyAgent [candidate] (683.074 ms) : 0, 683074
GlobalTracer [baseline] (314.987 ms) : 0, 314987
GlobalTracer [candidate] (311.859 ms) : 0, 311859
AppSec [baseline] (54.519 ms) : 0, 54519
AppSec [candidate] (54.032 ms) : 0, 54032
Remote Config [baseline] (686.761 µs) : 0, 687
Remote Config [candidate] (662.605 µs) : 0, 663
Telemetry [baseline] (7.773 ms) : 0, 7773
Telemetry [candidate] (7.623 ms) : 0, 7623
section appsec
BytebuddyAgent [baseline] (699.118 ms) : 0, 699118
BytebuddyAgent [candidate] (699.399 ms) : 0, 699399
GlobalTracer [baseline] (308.404 ms) : 0, 308404
GlobalTracer [candidate] (308.145 ms) : 0, 308145
AppSec [baseline] (163.068 ms) : 0, 163068
AppSec [candidate] (163.321 ms) : 0, 163321
Remote Config [baseline] (639.895 µs) : 0, 640
Remote Config [candidate] (631.921 µs) : 0, 632
Telemetry [baseline] (8.927 ms) : 0, 8927
Telemetry [candidate] (8.561 ms) : 0, 8561
IAST [baseline] (20.037 ms) : 0, 20037
IAST [candidate] (20.048 ms) : 0, 20048
section iast
BytebuddyAgent [baseline] (796.428 ms) : 0, 796428
BytebuddyAgent [candidate] (803.886 ms) : 0, 803886
GlobalTracer [baseline] (300.631 ms) : 0, 300631
GlobalTracer [candidate] (302.398 ms) : 0, 302398
AppSec [baseline] (53.702 ms) : 0, 53702
AppSec [candidate] (56.074 ms) : 0, 56074
Remote Config [baseline] (610.564 µs) : 0, 611
Remote Config [candidate] (623.15 µs) : 0, 623
Telemetry [baseline] (6.97 ms) : 0, 6970
Telemetry [candidate] (7.133 ms) : 0, 7133
IAST [baseline] (24.426 ms) : 0, 24426
IAST [candidate] (22.02 ms) : 0, 22020
section profiling
BytebuddyAgent [baseline] (674.541 ms) : 0, 674541
BytebuddyAgent [candidate] (675.474 ms) : 0, 675474
GlobalTracer [baseline] (393.713 ms) : 0, 393713
GlobalTracer [candidate] (394.14 ms) : 0, 394140
AppSec [baseline] (54.481 ms) : 0, 54481
AppSec [candidate] (54.427 ms) : 0, 54427
Remote Config [baseline] (649.838 µs) : 0, 650
Remote Config [candidate] (646.028 µs) : 0, 646
Telemetry [baseline] (7.409 ms) : 0, 7409
Telemetry [candidate] (7.382 ms) : 0, 7382
ProfilingAgent [baseline] (96.547 ms) : 0, 96547
ProfilingAgent [candidate] (96.715 ms) : 0, 96715
Profiling [baseline] (96.57 ms) : 0, 96570
Profiling [candidate] (96.738 ms) : 0, 96738
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section baseline
no_agent (372.438 µs) : 352, 392
. : milestone, 372,
iast (481.624 µs) : 460, 503
. : milestone, 482,
iast_FULL (553.624 µs) : 532, 575
. : milestone, 554,
iast_GLOBAL (510.816 µs) : 489, 533
. : milestone, 511,
iast_HARDCODED_SECRET_DISABLED (480.047 µs) : 459, 501
. : milestone, 480,
iast_INACTIVE (447.971 µs) : 427, 469
. : milestone, 448,
iast_TELEMETRY_OFF (480.907 µs) : 459, 503
. : milestone, 481,
tracing (444.027 µs) : 423, 465
. : milestone, 444,
section candidate
no_agent (370.996 µs) : 352, 390
. : milestone, 371,
iast (486.334 µs) : 465, 508
. : milestone, 486,
iast_FULL (549.961 µs) : 529, 571
. : milestone, 550,
iast_GLOBAL (504.672 µs) : 483, 527
. : milestone, 505,
iast_HARDCODED_SECRET_DISABLED (477.104 µs) : 456, 498
. : milestone, 477,
iast_INACTIVE (446.071 µs) : 425, 468
. : milestone, 446,
iast_TELEMETRY_OFF (483.256 µs) : 461, 505
. : milestone, 483,
tracing (444.168 µs) : 423, 465
. : milestone, 444,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section baseline
no_agent (1.343 ms) : 1324, 1362
. : milestone, 1343,
appsec (1.712 ms) : 1687, 1737
. : milestone, 1712,
appsec_no_iast (1.717 ms) : 1693, 1740
. : milestone, 1717,
iast (1.47 ms) : 1447, 1492
. : milestone, 1470,
profiling (1.481 ms) : 1456, 1505
. : milestone, 1481,
tracing (1.449 ms) : 1425, 1474
. : milestone, 1449,
section candidate
no_agent (1.335 ms) : 1316, 1355
. : milestone, 1335,
appsec (1.723 ms) : 1699, 1747
. : milestone, 1723,
appsec_no_iast (1.731 ms) : 1707, 1755
. : milestone, 1731,
iast (1.461 ms) : 1438, 1484
. : milestone, 1461,
profiling (1.528 ms) : 1503, 1552
. : milestone, 1528,
tracing (1.466 ms) : 1441, 1491
. : milestone, 1466,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section baseline
no_agent (15.25 s) : 15250000, 15250000
. : milestone, 15250000,
appsec (15.238 s) : 15238000, 15238000
. : milestone, 15238000,
iast (18.846 s) : 18846000, 18846000
. : milestone, 18846000,
iast_GLOBAL (18.046 s) : 18046000, 18046000
. : milestone, 18046000,
profiling (15.031 s) : 15031000, 15031000
. : milestone, 15031000,
tracing (15.131 s) : 15131000, 15131000
. : milestone, 15131000,
section candidate
no_agent (15.118 s) : 15118000, 15118000
. : milestone, 15118000,
appsec (15.034 s) : 15034000, 15034000
. : milestone, 15034000,
iast (18.982 s) : 18982000, 18982000
. : milestone, 18982000,
iast_GLOBAL (18.208 s) : 18208000, 18208000
. : milestone, 18208000,
profiling (15.005 s) : 15005000, 15005000
. : milestone, 15005000,
tracing (15.295 s) : 15295000, 15295000
. : milestone, 15295000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.41.0-SNAPSHOT~b1fd9f6ece, baseline=1.41.0-SNAPSHOT~794ff14344
dateFormat X
axisFormat %s
section baseline
no_agent (1.46 ms) : 1449, 1471
. : milestone, 1460,
appsec (2.302 ms) : 2262, 2342
. : milestone, 2302,
iast (2.069 ms) : 2017, 2120
. : milestone, 2069,
iast_GLOBAL (2.11 ms) : 2058, 2162
. : milestone, 2110,
profiling (1.934 ms) : 1893, 1975
. : milestone, 1934,
tracing (1.899 ms) : 1861, 1938
. : milestone, 1899,
section candidate
no_agent (1.457 ms) : 1446, 1468
. : milestone, 1457,
appsec (2.301 ms) : 2261, 2341
. : milestone, 2301,
iast (2.068 ms) : 2016, 2120
. : milestone, 2068,
iast_GLOBAL (2.103 ms) : 2052, 2155
. : milestone, 2103,
profiling (1.938 ms) : 1896, 1980
. : milestone, 1938,
tracing (1.908 ms) : 1868, 1947
. : milestone, 1908,
|
smola
approved these changes
Oct 1, 2024
1. Summary uses time format with space padding for days of the month while our parser uses zero padded number. Change the DateTimeFormatter pattern and add a test to cover this case. 2. Summary time can be based of offset and time zone name. Add two parsers to support both formats.
PerfectSlayer
force-pushed
the
bbujon/crashtracking
branch
from
October 1, 2024 13:53
cdb49e8
to
b1fd9f6
Compare
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 the date time parser from the crashtracking log parser to support:
It also adds some unit tests that cover this change to quickly test for new values.
Motivation
The Summary section uses time format with space padding for days of the month while our parser uses zero padded number (
dd
, reference).The Summary section time can also be offset based (
Z
,+08
,+08:00
) or time zone based (CEST
,UTC
,GMT
).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]