-
Notifications
You must be signed in to change notification settings - Fork 290
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
Check logs for errors at smoke tests cleanup #8111
Open
smola
wants to merge
1
commit into
master
Choose a base branch
from
smola/smoke-test-log-errors
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.
Open
+178
−134
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
smola
added
comp: testing
Testing
tag: no release notes
Changes to exclude from release notes
labels
Dec 18, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 52 metrics, 11 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.092 s) : 0, 1091748
Total [baseline] (10.47 s) : 0, 10469628
Agent [candidate] (1.102 s) : 0, 1101504
Total [candidate] (10.461 s) : 0, 10460996
section appsec
Agent [baseline] (1.226 s) : 0, 1225662
Total [baseline] (10.69 s) : 0, 10690144
Agent [candidate] (1.227 s) : 0, 1227089
Total [candidate] (10.704 s) : 0, 10704325
section iast
Agent [baseline] (1.219 s) : 0, 1219471
Total [baseline] (10.984 s) : 0, 10984301
Agent [candidate] (1.225 s) : 0, 1224894
Total [candidate] (11.051 s) : 0, 11050643
section profiling
Agent [baseline] (1.333 s) : 0, 1332783
Total [baseline] (10.93 s) : 0, 10929966
Agent [candidate] (1.325 s) : 0, 1324895
Total [candidate] (10.898 s) : 0, 10898440
gantt
title petclinic - break down per module: candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (695.1 ms) : 0, 695100
BytebuddyAgent [candidate] (701.279 ms) : 0, 701279
GlobalTracer [baseline] (317.379 ms) : 0, 317379
GlobalTracer [candidate] (319.773 ms) : 0, 319773
AppSec [baseline] (54.125 ms) : 0, 54125
AppSec [candidate] (55.116 ms) : 0, 55116
Remote Config [baseline] (659.637 µs) : 0, 660
Remote Config [candidate] (666.534 µs) : 0, 667
Telemetry [baseline] (10.789 ms) : 0, 10789
Telemetry [candidate] (10.833 ms) : 0, 10833
section appsec
BytebuddyAgent [baseline] (712.146 ms) : 0, 712146
BytebuddyAgent [candidate] (713.44 ms) : 0, 713440
GlobalTracer [baseline] (314.734 ms) : 0, 314734
GlobalTracer [candidate] (314.779 ms) : 0, 314779
AppSec [baseline] (165.425 ms) : 0, 165425
AppSec [candidate] (165.368 ms) : 0, 165368
Remote Config [baseline] (666.748 µs) : 0, 667
Remote Config [candidate] (668.677 µs) : 0, 669
Telemetry [baseline] (8.44 ms) : 0, 8440
Telemetry [candidate] (8.46 ms) : 0, 8460
IAST [baseline] (21.141 ms) : 0, 21141
IAST [candidate] (21.265 ms) : 0, 21265
section iast
BytebuddyAgent [baseline] (812.616 ms) : 0, 812616
BytebuddyAgent [candidate] (817.932 ms) : 0, 817932
GlobalTracer [baseline] (306.052 ms) : 0, 306052
GlobalTracer [candidate] (305.947 ms) : 0, 305947
AppSec [baseline] (57.293 ms) : 0, 57293
AppSec [candidate] (57.947 ms) : 0, 57947
Remote Config [baseline] (599.358 µs) : 0, 599
Remote Config [candidate] (618.068 µs) : 0, 618
Telemetry [baseline] (7.47 ms) : 0, 7470
Telemetry [candidate] (7.43 ms) : 0, 7430
IAST [baseline] (21.707 ms) : 0, 21707
IAST [candidate] (21.195 ms) : 0, 21195
section profiling
BytebuddyAgent [baseline] (697.865 ms) : 0, 697865
BytebuddyAgent [candidate] (691.638 ms) : 0, 691638
GlobalTracer [baseline] (437.303 ms) : 0, 437303
GlobalTracer [candidate] (437.005 ms) : 0, 437005
AppSec [baseline] (54.156 ms) : 0, 54156
AppSec [candidate] (54.061 ms) : 0, 54061
Remote Config [baseline] (662.271 µs) : 0, 662
Remote Config [candidate] (679.167 µs) : 0, 679
Telemetry [baseline] (7.761 ms) : 0, 7761
Telemetry [candidate] (7.734 ms) : 0, 7734
ProfilingAgent [baseline] (95.369 ms) : 0, 95369
ProfilingAgent [candidate] (94.57 ms) : 0, 94570
Profiling [baseline] (95.392 ms) : 0, 95392
Profiling [candidate] (94.595 ms) : 0, 94595
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.103 s) : 0, 1102751
Total [baseline] (8.681 s) : 0, 8680713
Agent [candidate] (1.095 s) : 0, 1095404
Total [candidate] (8.665 s) : 0, 8664584
section iast
Agent [baseline] (1.231 s) : 0, 1231164
Total [baseline] (9.278 s) : 0, 9277673
Agent [candidate] (1.222 s) : 0, 1221573
Total [candidate] (9.213 s) : 0, 9212887
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.221 s) : 0, 1221441
Total [baseline] (9.177 s) : 0, 9176704
Agent [candidate] (1.221 s) : 0, 1221184
Total [candidate] (9.19 s) : 0, 9190090
section iast_TELEMETRY_OFF
Agent [baseline] (1.223 s) : 0, 1223022
Total [baseline] (9.182 s) : 0, 9181697
Agent [candidate] (1.228 s) : 0, 1227528
Total [candidate] (9.193 s) : 0, 9192864
gantt
title insecure-bank - break down per module: candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (701.72 ms) : 0, 701720
BytebuddyAgent [candidate] (697.072 ms) : 0, 697072
GlobalTracer [baseline] (319.253 ms) : 0, 319253
GlobalTracer [candidate] (318.316 ms) : 0, 318316
AppSec [baseline] (55.002 ms) : 0, 55002
AppSec [candidate] (54.305 ms) : 0, 54305
Remote Config [baseline] (671.291 µs) : 0, 671
Remote Config [candidate] (670.972 µs) : 0, 671
Telemetry [baseline] (12.262 ms) : 0, 12262
Telemetry [candidate] (11.304 ms) : 0, 11304
section iast
BytebuddyAgent [baseline] (819.624 ms) : 0, 819624
BytebuddyAgent [candidate] (814.507 ms) : 0, 814507
GlobalTracer [baseline] (310.02 ms) : 0, 310020
GlobalTracer [candidate] (306.619 ms) : 0, 306619
AppSec [baseline] (57.597 ms) : 0, 57597
AppSec [candidate] (57.032 ms) : 0, 57032
Remote Config [baseline] (624.696 µs) : 0, 625
Remote Config [candidate] (610.875 µs) : 0, 611
Telemetry [baseline] (7.509 ms) : 0, 7509
Telemetry [candidate] (7.424 ms) : 0, 7424
IAST [baseline] (21.941 ms) : 0, 21941
IAST [candidate] (21.634 ms) : 0, 21634
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (814.059 ms) : 0, 814059
BytebuddyAgent [candidate] (813.804 ms) : 0, 813804
GlobalTracer [baseline] (306.463 ms) : 0, 306463
GlobalTracer [candidate] (306.083 ms) : 0, 306083
AppSec [baseline] (58.052 ms) : 0, 58052
AppSec [candidate] (56.564 ms) : 0, 56564
Remote Config [baseline] (620.042 µs) : 0, 620
Remote Config [candidate] (618.745 µs) : 0, 619
Telemetry [baseline] (7.46 ms) : 0, 7460
Telemetry [candidate] (7.471 ms) : 0, 7471
IAST [baseline] (21.046 ms) : 0, 21046
IAST [candidate] (22.897 ms) : 0, 22897
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (817.195 ms) : 0, 817195
BytebuddyAgent [candidate] (818.682 ms) : 0, 818682
GlobalTracer [baseline] (305.509 ms) : 0, 305509
GlobalTracer [candidate] (308.328 ms) : 0, 308328
AppSec [baseline] (58.083 ms) : 0, 58083
AppSec [candidate] (57.064 ms) : 0, 57064
Remote Config [baseline] (598.623 µs) : 0, 599
Remote Config [candidate] (608.866 µs) : 0, 609
Telemetry [baseline] (7.309 ms) : 0, 7309
Telemetry [candidate] (7.405 ms) : 0, 7405
IAST [baseline] (20.499 ms) : 0, 20499
IAST [candidate] (21.574 ms) : 0, 21574
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 petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section baseline
no_agent (1.336 ms) : 1316, 1355
. : milestone, 1336,
appsec (1.748 ms) : 1724, 1771
. : milestone, 1748,
appsec_no_iast (1.748 ms) : 1723, 1773
. : milestone, 1748,
iast (1.501 ms) : 1479, 1523
. : milestone, 1501,
profiling (1.497 ms) : 1473, 1521
. : milestone, 1497,
tracing (1.49 ms) : 1466, 1514
. : milestone, 1490,
section candidate
no_agent (1.342 ms) : 1323, 1362
. : milestone, 1342,
appsec (1.735 ms) : 1712, 1758
. : milestone, 1735,
appsec_no_iast (1.747 ms) : 1722, 1772
. : milestone, 1747,
iast (1.49 ms) : 1468, 1512
. : milestone, 1490,
profiling (1.515 ms) : 1492, 1538
. : milestone, 1515,
tracing (1.486 ms) : 1461, 1511
. : milestone, 1486,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section baseline
no_agent (374.483 µs) : 353, 396
. : milestone, 374,
iast (488.21 µs) : 467, 510
. : milestone, 488,
iast_FULL (646.883 µs) : 625, 668
. : milestone, 647,
iast_GLOBAL (516.146 µs) : 494, 538
. : milestone, 516,
iast_HARDCODED_SECRET_DISABLED (487.353 µs) : 466, 509
. : milestone, 487,
iast_INACTIVE (451.867 µs) : 430, 473
. : milestone, 452,
iast_TELEMETRY_OFF (476.608 µs) : 455, 498
. : milestone, 477,
tracing (447.562 µs) : 427, 469
. : milestone, 448,
section candidate
no_agent (371.084 µs) : 351, 391
. : milestone, 371,
iast (491.001 µs) : 469, 513
. : milestone, 491,
iast_FULL (643.783 µs) : 622, 665
. : milestone, 644,
iast_GLOBAL (525.045 µs) : 502, 548
. : milestone, 525,
iast_HARDCODED_SECRET_DISABLED (484.06 µs) : 463, 505
. : milestone, 484,
iast_INACTIVE (447.485 µs) : 426, 469
. : milestone, 447,
iast_TELEMETRY_OFF (474.91 µs) : 453, 497
. : milestone, 475,
tracing (449.544 µs) : 428, 471
. : milestone, 450,
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~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section baseline
no_agent (15.043 s) : 15043000, 15043000
. : milestone, 15043000,
appsec (14.735 s) : 14735000, 14735000
. : milestone, 14735000,
iast (19.102 s) : 19102000, 19102000
. : milestone, 19102000,
iast_GLOBAL (17.822 s) : 17822000, 17822000
. : milestone, 17822000,
profiling (15.43 s) : 15430000, 15430000
. : milestone, 15430000,
tracing (14.922 s) : 14922000, 14922000
. : milestone, 14922000,
section candidate
no_agent (15.191 s) : 15191000, 15191000
. : milestone, 15191000,
appsec (14.929 s) : 14929000, 14929000
. : milestone, 14929000,
iast (19.102 s) : 19102000, 19102000
. : milestone, 19102000,
iast_GLOBAL (18.066 s) : 18066000, 18066000
. : milestone, 18066000,
profiling (14.913 s) : 14913000, 14913000
. : milestone, 14913000,
tracing (15.092 s) : 15092000, 15092000
. : milestone, 15092000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.45.0-SNAPSHOT~acfbbb9657, baseline=1.45.0-SNAPSHOT~a19f73a5ea
dateFormat X
axisFormat %s
section baseline
no_agent (1.465 ms) : 1453, 1476
. : milestone, 1465,
appsec (2.337 ms) : 2295, 2379
. : milestone, 2337,
iast (2.087 ms) : 2034, 2140
. : milestone, 2087,
iast_GLOBAL (2.128 ms) : 2075, 2182
. : milestone, 2128,
profiling (1.936 ms) : 1894, 1978
. : milestone, 1936,
tracing (1.922 ms) : 1881, 1962
. : milestone, 1922,
section candidate
no_agent (1.464 ms) : 1452, 1475
. : milestone, 1464,
appsec (2.328 ms) : 2286, 2370
. : milestone, 2328,
iast (2.074 ms) : 2021, 2127
. : milestone, 2074,
iast_GLOBAL (2.13 ms) : 2076, 2184
. : milestone, 2130,
profiling (2.422 ms) : 2244, 2599
. : milestone, 2422,
tracing (1.926 ms) : 1885, 1967
. : milestone, 1926,
|
smola
force-pushed
the
smola/smoke-test-log-errors
branch
6 times, most recently
from
December 18, 2024 20:53
2e82afe
to
112f4f1
Compare
smola
force-pushed
the
smola/smoke-test-log-errors
branch
from
December 18, 2024 21:34
112f4f1
to
acfbbb9
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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
boolean isErrorLog(String line)
defining what is considered an error log for a given smoke test. Most smoke tests can just rely on the default. This PR includes several examples on how to extend it, both to add additional errors, or to exclude some errors.void assertNoErrorLogs(final Closure<Boolean> extraIsErrorLog)
, which can be used to assert there are no error logs. Most tests should not use directly, and when they do, they should do it only after the test process has finished (otherwise, the test may be flaky).assertNoErrorLogs
oncleanupSpec
for every smoke test. Most suites were not checking error logs.processTestLogLines
when we need to wait for a log line that is supposed to be printed. This should be less flaky thanassertNoErrorLogs
in the case where the test process has not ended.checkLogPostExit
, which was superseded by the above methods.Motivation
We have various flaky tests on smoke tests, and sometimes non-flaky failures that are hard to diagnose, that show up in errors all just as:
This means there was an error printed to logs, but to find which one we need to download
reports.tar
and dig into the logs.After this PR, they look like this:
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]