Disable retry instrumentations if both Early Flakiness Detection and Flaky Test Retries are turned off#6925
Merged
nikita-tkachenko-datadog merged 1 commit intomasterfrom Apr 17, 2024
Conversation
…Flaky Test Retries are turned off
romainkomorn-exdatadog
approved these changes
Apr 17, 2024
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 insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.083 s) : 0, 1082928
Total [baseline] (8.587 s) : 0, 8586816
Agent [candidate] (1.074 s) : 0, 1074127
Total [candidate] (8.561 s) : 0, 8561140
section iast
Agent [baseline] (1.2 s) : 0, 1200055
Total [baseline] (9.044 s) : 0, 9044220
Agent [candidate] (1.207 s) : 0, 1207145
Total [candidate] (9.053 s) : 0, 9053310
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.198 s) : 0, 1197937
Total [baseline] (8.975 s) : 0, 8975495
Agent [candidate] (1.197 s) : 0, 1196624
Total [candidate] (8.993 s) : 0, 8992971
section iast_TELEMETRY_OFF
Agent [baseline] (1.205 s) : 0, 1204982
Total [baseline] (9.008 s) : 0, 9007957
Agent [candidate] (1.196 s) : 0, 1195914
Total [candidate] (9.006 s) : 0, 9005739
gantt
title insecure-bank - break down per module: candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (678.225 ms) : 0, 678225
BytebuddyAgent [candidate] (672.813 ms) : 0, 672813
GlobalTracer [baseline] (311.65 ms) : 0, 311650
GlobalTracer [candidate] (309.413 ms) : 0, 309413
AppSec [baseline] (49.866 ms) : 0, 49866
AppSec [candidate] (49.411 ms) : 0, 49411
Remote Config [baseline] (673.912 µs) : 0, 674
Remote Config [candidate] (668.463 µs) : 0, 668
Telemetry [baseline] (7.712 ms) : 0, 7712
Telemetry [candidate] (7.587 ms) : 0, 7587
section iast
BytebuddyAgent [baseline] (795.511 ms) : 0, 795511
BytebuddyAgent [candidate] (800.337 ms) : 0, 800337
GlobalTracer [baseline] (288.432 ms) : 0, 288432
GlobalTracer [candidate] (289.887 ms) : 0, 289887
AppSec [baseline] (49.664 ms) : 0, 49664
AppSec [candidate] (51.353 ms) : 0, 51353
IAST [baseline] (23.95 ms) : 0, 23950
IAST [candidate] (23.02 ms) : 0, 23020
Remote Config [baseline] (582.269 µs) : 0, 582
Remote Config [candidate] (575.479 µs) : 0, 575
Telemetry [baseline] (7.456 ms) : 0, 7456
Telemetry [candidate] (7.479 ms) : 0, 7479
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (793.648 ms) : 0, 793648
BytebuddyAgent [candidate] (793.146 ms) : 0, 793146
GlobalTracer [baseline] (287.878 ms) : 0, 287878
GlobalTracer [candidate] (287.677 ms) : 0, 287677
AppSec [baseline] (50.408 ms) : 0, 50408
AppSec [candidate] (51.307 ms) : 0, 51307
IAST [baseline] (23.013 ms) : 0, 23013
IAST [candidate] (22.88 ms) : 0, 22880
Remote Config [baseline] (583.055 µs) : 0, 583
Remote Config [candidate] (575.872 µs) : 0, 576
Telemetry [baseline] (8.109 ms) : 0, 8109
Telemetry [candidate] (6.673 ms) : 0, 6673
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.052 ms) : 0, 799052
BytebuddyAgent [candidate] (791.655 ms) : 0, 791655
GlobalTracer [baseline] (289.992 ms) : 0, 289992
GlobalTracer [candidate] (288.309 ms) : 0, 288309
AppSec [baseline] (48.673 ms) : 0, 48673
AppSec [candidate] (49.514 ms) : 0, 49514
IAST [baseline] (25.455 ms) : 0, 25455
IAST [candidate] (24.114 ms) : 0, 24114
Remote Config [baseline] (580.317 µs) : 0, 580
Remote Config [candidate] (585.494 µs) : 0, 585
Telemetry [baseline] (6.531 ms) : 0, 6531
Telemetry [candidate] (7.357 ms) : 0, 7357
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.074 s) : 0, 1073513
Total [baseline] (10.354 s) : 0, 10353646
Agent [candidate] (1.074 s) : 0, 1074285
Total [candidate] (10.521 s) : 0, 10521400
section appsec
Agent [baseline] (1.19 s) : 0, 1189742
Total [baseline] (10.551 s) : 0, 10551013
Agent [candidate] (1.187 s) : 0, 1186578
Total [candidate] (10.491 s) : 0, 10491334
section iast
Agent [baseline] (1.208 s) : 0, 1207731
Total [baseline] (10.781 s) : 0, 10780505
Agent [candidate] (1.203 s) : 0, 1203108
Total [candidate] (10.834 s) : 0, 10833766
section profiling
Agent [baseline] (1.276 s) : 0, 1276131
Total [baseline] (10.623 s) : 0, 10623145
Agent [candidate] (1.284 s) : 0, 1283716
Total [candidate] (10.698 s) : 0, 10698397
gantt
title petclinic - break down per module: candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (672.727 ms) : 0, 672727
BytebuddyAgent [candidate] (673.004 ms) : 0, 673004
GlobalTracer [baseline] (308.815 ms) : 0, 308815
GlobalTracer [candidate] (309.397 ms) : 0, 309397
AppSec [baseline] (49.386 ms) : 0, 49386
AppSec [candidate] (49.226 ms) : 0, 49226
Remote Config [baseline] (662.646 µs) : 0, 663
Remote Config [candidate] (659.465 µs) : 0, 659
Telemetry [baseline] (7.634 ms) : 0, 7634
Telemetry [candidate] (7.62 ms) : 0, 7620
section appsec
BytebuddyAgent [baseline] (696.129 ms) : 0, 696129
BytebuddyAgent [candidate] (694.717 ms) : 0, 694717
GlobalTracer [baseline] (291.632 ms) : 0, 291632
GlobalTracer [candidate] (291.147 ms) : 0, 291147
AppSec [baseline] (149.888 ms) : 0, 149888
AppSec [candidate] (149.113 ms) : 0, 149113
IAST [baseline] (18.936 ms) : 0, 18936
IAST [candidate] (18.83 ms) : 0, 18830
Remote Config [baseline] (628.118 µs) : 0, 628
Remote Config [candidate] (616.376 µs) : 0, 616
Telemetry [baseline] (7.543 ms) : 0, 7543
Telemetry [candidate] (7.18 ms) : 0, 7180
section iast
BytebuddyAgent [baseline] (801.069 ms) : 0, 801069
BytebuddyAgent [candidate] (795.78 ms) : 0, 795780
GlobalTracer [baseline] (289.931 ms) : 0, 289931
GlobalTracer [candidate] (290.323 ms) : 0, 290323
AppSec [baseline] (51.611 ms) : 0, 51611
AppSec [candidate] (52.051 ms) : 0, 52051
IAST [baseline] (22.393 ms) : 0, 22393
IAST [candidate] (23.314 ms) : 0, 23314
Remote Config [baseline] (583.826 µs) : 0, 584
Remote Config [candidate] (574.228 µs) : 0, 574
Telemetry [baseline] (7.458 ms) : 0, 7458
Telemetry [candidate] (6.66 ms) : 0, 6660
section profiling
BytebuddyAgent [baseline] (682.194 ms) : 0, 682194
BytebuddyAgent [candidate] (686.478 ms) : 0, 686478
GlobalTracer [baseline] (381.853 ms) : 0, 381853
GlobalTracer [candidate] (384.496 ms) : 0, 384496
AppSec [baseline] (50.294 ms) : 0, 50294
AppSec [candidate] (50.762 ms) : 0, 50762
Remote Config [baseline] (714.074 µs) : 0, 714
Remote Config [candidate] (712.133 µs) : 0, 712
Telemetry [baseline] (7.553 ms) : 0, 7553
Telemetry [candidate] (7.538 ms) : 0, 7538
ProfilingAgent [baseline] (96.843 ms) : 0, 96843
ProfilingAgent [candidate] (96.6 ms) : 0, 96600
Profiling [baseline] (96.867 ms) : 0, 96867
Profiling [candidate] (96.624 ms) : 0, 96624
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 13 metrics, 15 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section baseline
no_agent (1.355 ms) : 1335, 1375
. : milestone, 1355,
appsec (1.72 ms) : 1695, 1745
. : milestone, 1720,
appsec_no_iast (1.741 ms) : 1716, 1765
. : milestone, 1741,
iast (1.515 ms) : 1492, 1537
. : milestone, 1515,
profiling (1.554 ms) : 1528, 1579
. : milestone, 1554,
tracing (1.494 ms) : 1469, 1519
. : milestone, 1494,
section candidate
no_agent (1.352 ms) : 1332, 1371
. : milestone, 1352,
appsec (1.739 ms) : 1715, 1762
. : milestone, 1739,
appsec_no_iast (1.73 ms) : 1706, 1753
. : milestone, 1730,
iast (1.512 ms) : 1490, 1535
. : milestone, 1512,
profiling (1.536 ms) : 1510, 1562
. : milestone, 1536,
tracing (1.487 ms) : 1464, 1511
. : milestone, 1487,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section baseline
no_agent (371.11 µs) : 350, 392
. : milestone, 371,
iast (488.455 µs) : 466, 511
. : milestone, 488,
iast_FULL (545.303 µs) : 524, 566
. : milestone, 545,
iast_GLOBAL (497.103 µs) : 476, 519
. : milestone, 497,
iast_HARDCODED_SECRET_DISABLED (478.326 µs) : 457, 499
. : milestone, 478,
iast_INACTIVE (460.413 µs) : 439, 482
. : milestone, 460,
iast_TELEMETRY_OFF (479.265 µs) : 458, 501
. : milestone, 479,
tracing (445.712 µs) : 425, 466
. : milestone, 446,
section candidate
no_agent (373.775 µs) : 354, 393
. : milestone, 374,
iast (472.647 µs) : 452, 493
. : milestone, 473,
iast_FULL (549.991 µs) : 529, 571
. : milestone, 550,
iast_GLOBAL (502.078 µs) : 481, 524
. : milestone, 502,
iast_HARDCODED_SECRET_DISABLED (487.641 µs) : 466, 509
. : milestone, 488,
iast_INACTIVE (449.993 µs) : 429, 471
. : milestone, 450,
iast_TELEMETRY_OFF (471.996 µs) : 451, 493
. : milestone, 472,
tracing (447.938 µs) : 427, 469
. : milestone, 448,
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.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section baseline
no_agent (1.456 ms) : 1445, 1468
. : milestone, 1456,
appsec (2.197 ms) : 2164, 2231
. : milestone, 2197,
iast (1.866 ms) : 1831, 1901
. : milestone, 1866,
iast_GLOBAL (1.914 ms) : 1878, 1949
. : milestone, 1914,
profiling (1.849 ms) : 1815, 1883
. : milestone, 1849,
tracing (1.813 ms) : 1782, 1845
. : milestone, 1813,
section candidate
no_agent (1.455 ms) : 1443, 1466
. : milestone, 1455,
appsec (2.19 ms) : 2157, 2224
. : milestone, 2190,
iast (1.873 ms) : 1838, 1908
. : milestone, 1873,
iast_GLOBAL (1.906 ms) : 1872, 1941
. : milestone, 1906,
profiling (1.839 ms) : 1806, 1873
. : milestone, 1839,
tracing (1.814 ms) : 1782, 1845
. : milestone, 1814,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.33.0-SNAPSHOT~78ae2bdf91, baseline=1.33.0-SNAPSHOT~34e706697c
dateFormat X
axisFormat %s
section baseline
no_agent (15.385 s) : 15385000, 15385000
. : milestone, 15385000,
appsec (15.073 s) : 15073000, 15073000
. : milestone, 15073000,
iast (18.892 s) : 18892000, 18892000
. : milestone, 18892000,
iast_GLOBAL (17.851 s) : 17851000, 17851000
. : milestone, 17851000,
profiling (15.34 s) : 15340000, 15340000
. : milestone, 15340000,
tracing (14.861 s) : 14861000, 14861000
. : milestone, 14861000,
section candidate
no_agent (15.333 s) : 15333000, 15333000
. : milestone, 15333000,
appsec (14.835 s) : 14835000, 14835000
. : milestone, 14835000,
iast (18.974 s) : 18974000, 18974000
. : milestone, 18974000,
iast_GLOBAL (18.1 s) : 18100000, 18100000
. : milestone, 18100000,
profiling (15.519 s) : 15519000, 15519000
. : milestone, 15519000,
tracing (15.025 s) : 15025000, 15025000
. : milestone, 15025000,
|
anmarchenko
approved these changes
Apr 17, 2024
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
Disables instrumentations that are related to CI Visibility's test retry functionality in case both test retry features (Early Flakiness Detection and Failed Test Retries) are turned off.
Motivation
Applying those instrumentations if test retries are disabled is redundant: even though it does not break anything, it adds some unnecessary overhead.
Jira ticket: CIVIS-9800