Fail smoke tests if received traces cannot be decoded#7520
Merged
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 15 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.047 s) : 0, 1047450
Total [baseline] (10.381 s) : 0, 10380624
Agent [candidate] (1.046 s) : 0, 1046227
Total [candidate] (10.427 s) : 0, 10426984
section appsec
Agent [baseline] (1.176 s) : 0, 1175857
Total [baseline] (10.513 s) : 0, 10513145
Agent [candidate] (1.178 s) : 0, 1177530
Total [candidate] (10.444 s) : 0, 10444399
section iast
Agent [baseline] (1.174 s) : 0, 1174294
Total [baseline] (10.89 s) : 0, 10890302
Agent [candidate] (1.172 s) : 0, 1172313
Total [candidate] (10.808 s) : 0, 10807574
section profiling
Agent [baseline] (1.255 s) : 0, 1255172
Total [baseline] (10.584 s) : 0, 10584093
Agent [candidate] (1.263 s) : 0, 1263437
Total [candidate] (10.637 s) : 0, 10637209
gantt
title petclinic - break down per module: candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (668.603 ms) : 0, 668603
BytebuddyAgent [candidate] (667.514 ms) : 0, 667514
GlobalTracer [baseline] (305.848 ms) : 0, 305848
GlobalTracer [candidate] (305.836 ms) : 0, 305836
AppSec [baseline] (51.298 ms) : 0, 51298
AppSec [candidate] (51.206 ms) : 0, 51206
Remote Config [baseline] (668.13 µs) : 0, 668
Remote Config [candidate] (662.13 µs) : 0, 662
Telemetry [baseline] (7.48 ms) : 0, 7480
Telemetry [candidate] (7.449 ms) : 0, 7449
section appsec
BytebuddyAgent [baseline] (681.982 ms) : 0, 681982
BytebuddyAgent [candidate] (682.993 ms) : 0, 682993
GlobalTracer [baseline] (302.624 ms) : 0, 302624
GlobalTracer [candidate] (301.964 ms) : 0, 301964
AppSec [baseline] (160.998 ms) : 0, 160998
AppSec [candidate] (161.026 ms) : 0, 161026
IAST [baseline] (18.371 ms) : 0, 18371
IAST [candidate] (18.885 ms) : 0, 18885
Remote Config [baseline] (637.75 µs) : 0, 638
Remote Config [candidate] (641.215 µs) : 0, 641
Telemetry [baseline] (6.834 ms) : 0, 6834
Telemetry [candidate] (7.812 ms) : 0, 7812
section iast
BytebuddyAgent [baseline] (779.409 ms) : 0, 779409
BytebuddyAgent [candidate] (777.807 ms) : 0, 777807
GlobalTracer [baseline] (296.569 ms) : 0, 296569
GlobalTracer [candidate] (296.096 ms) : 0, 296096
AppSec [baseline] (50.449 ms) : 0, 50449
AppSec [candidate] (50.487 ms) : 0, 50487
IAST [baseline] (25.006 ms) : 0, 25006
IAST [candidate] (24.243 ms) : 0, 24243
Remote Config [baseline] (1.281 ms) : 0, 1281
Remote Config [candidate] (597.894 µs) : 0, 598
Telemetry [baseline] (7.977 ms) : 0, 7977
Telemetry [candidate] (9.486 ms) : 0, 9486
section profiling
BytebuddyAgent [baseline] (669.309 ms) : 0, 669309
BytebuddyAgent [candidate] (673.712 ms) : 0, 673712
GlobalTracer [baseline] (391.281 ms) : 0, 391281
GlobalTracer [candidate] (393.433 ms) : 0, 393433
AppSec [baseline] (52.438 ms) : 0, 52438
AppSec [candidate] (53.051 ms) : 0, 53051
Remote Config [baseline] (684.518 µs) : 0, 685
Remote Config [candidate] (709.546 µs) : 0, 710
Telemetry [baseline] (7.396 ms) : 0, 7396
Telemetry [candidate] (7.527 ms) : 0, 7527
ProfilingAgent [baseline] (96.381 ms) : 0, 96381
ProfilingAgent [candidate] (97.16 ms) : 0, 97160
Profiling [baseline] (96.405 ms) : 0, 96405
Profiling [candidate] (97.184 ms) : 0, 97184
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.059 s) : 0, 1059351
Total [baseline] (8.508 s) : 0, 8508094
Agent [candidate] (1.048 s) : 0, 1048212
Total [candidate] (8.467 s) : 0, 8467476
section iast
Agent [baseline] (1.171 s) : 0, 1171027
Total [baseline] (8.954 s) : 0, 8953660
Agent [candidate] (1.173 s) : 0, 1172842
Total [candidate] (8.966 s) : 0, 8965972
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.171 s) : 0, 1170977
Total [baseline] (8.941 s) : 0, 8941458
Agent [candidate] (1.179 s) : 0, 1179035
Total [candidate] (8.927 s) : 0, 8927347
section iast_TELEMETRY_OFF
Agent [baseline] (1.186 s) : 0, 1186320
Total [baseline] (8.998 s) : 0, 8998391
Agent [candidate] (1.168 s) : 0, 1167903
Total [candidate] (8.947 s) : 0, 8946595
gantt
title insecure-bank - break down per module: candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (677.079 ms) : 0, 677079
BytebuddyAgent [candidate] (668.855 ms) : 0, 668855
GlobalTracer [baseline] (308.582 ms) : 0, 308582
GlobalTracer [candidate] (306.185 ms) : 0, 306185
AppSec [baseline] (51.701 ms) : 0, 51701
AppSec [candidate] (51.456 ms) : 0, 51456
Remote Config [baseline] (677.48 µs) : 0, 677
Remote Config [candidate] (672.327 µs) : 0, 672
Telemetry [baseline] (7.559 ms) : 0, 7559
Telemetry [candidate] (7.482 ms) : 0, 7482
section iast
BytebuddyAgent [baseline] (777.054 ms) : 0, 777054
BytebuddyAgent [candidate] (779.149 ms) : 0, 779149
GlobalTracer [baseline] (295.737 ms) : 0, 295737
GlobalTracer [candidate] (296.368 ms) : 0, 296368
AppSec [baseline] (48.349 ms) : 0, 48349
AppSec [candidate] (51.914 ms) : 0, 51914
IAST [baseline] (27.792 ms) : 0, 27792
IAST [candidate] (24.05 ms) : 0, 24050
Remote Config [baseline] (586.388 µs) : 0, 586
Remote Config [candidate] (580.982 µs) : 0, 581
Telemetry [baseline] (7.96 ms) : 0, 7960
Telemetry [candidate] (7.18 ms) : 0, 7180
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (777.765 ms) : 0, 777765
BytebuddyAgent [candidate] (782.949 ms) : 0, 782949
GlobalTracer [baseline] (296.325 ms) : 0, 296325
GlobalTracer [candidate] (298.379 ms) : 0, 298379
AppSec [baseline] (49.03 ms) : 0, 49030
AppSec [candidate] (50.761 ms) : 0, 50761
IAST [baseline] (24.97 ms) : 0, 24970
IAST [candidate] (23.965 ms) : 0, 23965
Remote Config [baseline] (590.857 µs) : 0, 591
Remote Config [candidate] (577.316 µs) : 0, 577
Telemetry [baseline] (8.681 ms) : 0, 8681
Telemetry [candidate] (8.728 ms) : 0, 8728
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (786.463 ms) : 0, 786463
BytebuddyAgent [candidate] (774.618 ms) : 0, 774618
GlobalTracer [baseline] (300.096 ms) : 0, 300096
GlobalTracer [candidate] (296.149 ms) : 0, 296149
AppSec [baseline] (52.018 ms) : 0, 52018
AppSec [candidate] (51.188 ms) : 0, 51188
IAST [baseline] (24.617 ms) : 0, 24617
IAST [candidate] (23.208 ms) : 0, 23208
Remote Config [baseline] (593.78 µs) : 0, 594
Remote Config [candidate] (582.275 µs) : 0, 582
Telemetry [baseline] (8.731 ms) : 0, 8731
Telemetry [candidate] (8.54 ms) : 0, 8540
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 18 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section baseline
no_agent (364.344 µs) : 345, 384
. : milestone, 364,
iast (482.041 µs) : 459, 505
. : milestone, 482,
iast_FULL (544.634 µs) : 523, 566
. : milestone, 545,
iast_GLOBAL (503.817 µs) : 482, 525
. : milestone, 504,
iast_HARDCODED_SECRET_DISABLED (481.679 µs) : 460, 504
. : milestone, 482,
iast_INACTIVE (437.761 µs) : 417, 458
. : milestone, 438,
iast_TELEMETRY_OFF (470.082 µs) : 448, 492
. : milestone, 470,
tracing (433.17 µs) : 413, 453
. : milestone, 433,
section candidate
no_agent (365.43 µs) : 346, 385
. : milestone, 365,
iast (485.161 µs) : 462, 508
. : milestone, 485,
iast_FULL (549.019 µs) : 528, 570
. : milestone, 549,
iast_GLOBAL (501.879 µs) : 480, 524
. : milestone, 502,
iast_HARDCODED_SECRET_DISABLED (481.12 µs) : 459, 504
. : milestone, 481,
iast_INACTIVE (446.37 µs) : 425, 467
. : milestone, 446,
iast_TELEMETRY_OFF (472.289 µs) : 450, 495
. : milestone, 472,
tracing (438.142 µs) : 418, 458
. : milestone, 438,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section baseline
no_agent (1.348 ms) : 1328, 1367
. : milestone, 1348,
appsec (1.744 ms) : 1721, 1767
. : milestone, 1744,
appsec_no_iast (1.718 ms) : 1694, 1742
. : milestone, 1718,
iast (1.464 ms) : 1441, 1487
. : milestone, 1464,
profiling (1.477 ms) : 1454, 1499
. : milestone, 1477,
tracing (1.464 ms) : 1439, 1489
. : milestone, 1464,
section candidate
no_agent (1.363 ms) : 1343, 1382
. : milestone, 1363,
appsec (1.696 ms) : 1674, 1718
. : milestone, 1696,
appsec_no_iast (1.7 ms) : 1676, 1724
. : milestone, 1700,
iast (1.468 ms) : 1446, 1491
. : milestone, 1468,
profiling (1.478 ms) : 1455, 1502
. : milestone, 1478,
tracing (1.464 ms) : 1440, 1489
. : milestone, 1464,
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.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section baseline
no_agent (15.397 s) : 15397000, 15397000
. : milestone, 15397000,
appsec (15.095 s) : 15095000, 15095000
. : milestone, 15095000,
iast (18.818 s) : 18818000, 18818000
. : milestone, 18818000,
iast_GLOBAL (17.99 s) : 17990000, 17990000
. : milestone, 17990000,
profiling (15.046 s) : 15046000, 15046000
. : milestone, 15046000,
tracing (15.156 s) : 15156000, 15156000
. : milestone, 15156000,
section candidate
no_agent (14.973 s) : 14973000, 14973000
. : milestone, 14973000,
appsec (15.315 s) : 15315000, 15315000
. : milestone, 15315000,
iast (18.876 s) : 18876000, 18876000
. : milestone, 18876000,
iast_GLOBAL (17.734 s) : 17734000, 17734000
. : milestone, 17734000,
profiling (14.744 s) : 14744000, 14744000
. : milestone, 14744000,
tracing (15.156 s) : 15156000, 15156000
. : milestone, 15156000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.39.0-SNAPSHOT~b26e81274c, baseline=1.39.0-SNAPSHOT~115998f9f0
dateFormat X
axisFormat %s
section baseline
no_agent (1.462 ms) : 1451, 1473
. : milestone, 1462,
appsec (2.224 ms) : 2189, 2258
. : milestone, 2224,
iast (1.967 ms) : 1925, 2008
. : milestone, 1967,
iast_GLOBAL (2.024 ms) : 1981, 2067
. : milestone, 2024,
profiling (1.871 ms) : 1836, 1905
. : milestone, 1871,
tracing (1.85 ms) : 1817, 1884
. : milestone, 1850,
section candidate
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.233 ms) : 2198, 2268
. : milestone, 2233,
iast (1.986 ms) : 1943, 2029
. : milestone, 1986,
iast_GLOBAL (2.026 ms) : 1982, 2070
. : milestone, 2026,
profiling (2.305 ms) : 2066, 2544
. : milestone, 2305,
tracing (1.85 ms) : 1817, 1883
. : milestone, 1850,
|
This file contains hidden or 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
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
If smoke tests "mock agent" fails to decode a trace, bubble the exception up to waitForTraces (or eventually to cleanup). This way we get a clearer indication of the issue, rather than just a timeout on receiving the traces.
Motivation
Faster troubleshooting.
Additional Notes
Before this change:
After this change:
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 issue[ ] Update the public documentation in case of new configuration flag or behavior