-
Notifications
You must be signed in to change notification settings - Fork 304
Update impacted tests logic #8923
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
base: master
Are you sure you want to change the base?
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 58 metrics, 13 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.024 s) : 0, 1023531
Total [baseline] (8.54 s) : 0, 8540195
Agent [candidate] (1.025 s) : 0, 1025145
Total [candidate] (8.537 s) : 0, 8536504
section iast
Agent [baseline] (1.15 s) : 0, 1150020
Total [baseline] (9.171 s) : 0, 9170614
Agent [candidate] (1.156 s) : 0, 1156267
Total [candidate] (9.194 s) : 0, 9193602
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.152 s) : 0, 1151749
Total [baseline] (9.162 s) : 0, 9162217
Agent [candidate] (1.16 s) : 0, 1160167
Total [candidate] (9.164 s) : 0, 9163731
section iast_TELEMETRY_OFF
Agent [baseline] (1.15 s) : 0, 1150461
Total [baseline] (9.273 s) : 0, 9272691
Agent [candidate] (1.149 s) : 0, 1148715
Total [candidate] (9.238 s) : 0, 9237718
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (684.408 ms) : 0, 684408
BytebuddyAgent [candidate] (684.506 ms) : 0, 684506
GlobalTracer [baseline] (240.006 ms) : 0, 240006
GlobalTracer [candidate] (240.771 ms) : 0, 240771
AppSec [baseline] (59.02 ms) : 0, 59020
AppSec [candidate] (57.899 ms) : 0, 57899
Debugger [baseline] (6.099 ms) : 0, 6099
Debugger [candidate] (6.2 ms) : 0, 6200
Remote Config [baseline] (729.567 µs) : 0, 730
Remote Config [candidate] (738.855 µs) : 0, 739
Telemetry [baseline] (9.665 ms) : 0, 9665
Telemetry [candidate] (11.404 ms) : 0, 11404
section iast
BytebuddyAgent [baseline] (802.269 ms) : 0, 802269
BytebuddyAgent [candidate] (807.526 ms) : 0, 807526
GlobalTracer [baseline] (230.298 ms) : 0, 230298
GlobalTracer [candidate] (231.143 ms) : 0, 231143
AppSec [baseline] (53.34 ms) : 0, 53340
AppSec [candidate] (54.807 ms) : 0, 54807
Debugger [baseline] (5.988 ms) : 0, 5988
Debugger [candidate] (5.966 ms) : 0, 5966
Remote Config [baseline] (594.125 µs) : 0, 594
Remote Config [candidate] (602.487 µs) : 0, 602
Telemetry [baseline] (7.858 ms) : 0, 7858
Telemetry [candidate] (7.918 ms) : 0, 7918
IAST [baseline] (26.139 ms) : 0, 26139
IAST [candidate] (24.609 ms) : 0, 24609
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (802.845 ms) : 0, 802845
BytebuddyAgent [candidate] (809.805 ms) : 0, 809805
GlobalTracer [baseline] (230.705 ms) : 0, 230705
GlobalTracer [candidate] (231.936 ms) : 0, 231936
AppSec [baseline] (54.412 ms) : 0, 54412
AppSec [candidate] (50.52 ms) : 0, 50520
Debugger [baseline] (5.975 ms) : 0, 5975
Debugger [candidate] (5.957 ms) : 0, 5957
Remote Config [baseline] (603.867 µs) : 0, 604
Remote Config [candidate] (596.225 µs) : 0, 596
Telemetry [baseline] (7.994 ms) : 0, 7994
Telemetry [candidate] (8.061 ms) : 0, 8061
IAST [baseline] (25.623 ms) : 0, 25623
IAST [candidate] (29.597 ms) : 0, 29597
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (800.68 ms) : 0, 800680
BytebuddyAgent [candidate] (799.595 ms) : 0, 799595
GlobalTracer [baseline] (231.901 ms) : 0, 231901
GlobalTracer [candidate] (231.184 ms) : 0, 231184
AppSec [baseline] (51.285 ms) : 0, 51285
AppSec [candidate] (51.5 ms) : 0, 51500
Debugger [baseline] (6.009 ms) : 0, 6009
Debugger [candidate] (6.041 ms) : 0, 6041
Remote Config [baseline] (612.944 µs) : 0, 613
Remote Config [candidate] (608.83 µs) : 0, 609
Telemetry [baseline] (7.822 ms) : 0, 7822
Telemetry [candidate] (7.956 ms) : 0, 7956
IAST [baseline] (28.55 ms) : 0, 28550
IAST [candidate] (28.286 ms) : 0, 28286
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.031 s) : 0, 1031354
Total [baseline] (11.25 s) : 0, 11249642
Agent [candidate] (1.024 s) : 0, 1024299
Total [candidate] (11.087 s) : 0, 11087002
section appsec
Agent [baseline] (1.169 s) : 0, 1168986
Total [baseline] (11.256 s) : 0, 11255585
Agent [candidate] (1.163 s) : 0, 1163440
Total [candidate] (11.256 s) : 0, 11255709
section iast
Agent [baseline] (1.15 s) : 0, 1150379
Total [baseline] (11.398 s) : 0, 11397876
Agent [candidate] (1.152 s) : 0, 1151667
Total [candidate] (11.341 s) : 0, 11340894
section profiling
Agent [baseline] (1.265 s) : 0, 1265147
Total [baseline] (11.482 s) : 0, 11482339
Agent [candidate] (1.275 s) : 0, 1275269
Total [candidate] (11.484 s) : 0, 11484172
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.213 ms) : 0, 688213
BytebuddyAgent [candidate] (684.277 ms) : 0, 684277
GlobalTracer [baseline] (241.581 ms) : 0, 241581
GlobalTracer [candidate] (240.817 ms) : 0, 240817
AppSec [baseline] (57.259 ms) : 0, 57259
AppSec [candidate] (57.275 ms) : 0, 57275
Debugger [baseline] (6.93 ms) : 0, 6930
Debugger [candidate] (6.171 ms) : 0, 6171
Remote Config [baseline] (761.223 µs) : 0, 761
Remote Config [candidate] (733.625 µs) : 0, 734
Telemetry [baseline] (12.822 ms) : 0, 12822
Telemetry [candidate] (11.386 ms) : 0, 11386
section appsec
BytebuddyAgent [baseline] (704.416 ms) : 0, 704416
BytebuddyAgent [candidate] (700.394 ms) : 0, 700394
GlobalTracer [baseline] (238.813 ms) : 0, 238813
GlobalTracer [candidate] (238.01 ms) : 0, 238010
AppSec [baseline] (176.963 ms) : 0, 176963
AppSec [candidate] (176.558 ms) : 0, 176558
Debugger [baseline] (5.968 ms) : 0, 5968
Debugger [candidate] (5.982 ms) : 0, 5982
Remote Config [baseline] (640.277 µs) : 0, 640
Remote Config [candidate] (618.597 µs) : 0, 619
Telemetry [baseline] (7.383 ms) : 0, 7383
Telemetry [candidate] (7.32 ms) : 0, 7320
IAST [baseline] (22.173 ms) : 0, 22173
IAST [candidate] (21.847 ms) : 0, 21847
section iast
BytebuddyAgent [baseline] (801.982 ms) : 0, 801982
BytebuddyAgent [candidate] (802.9 ms) : 0, 802900
GlobalTracer [baseline] (230.551 ms) : 0, 230551
GlobalTracer [candidate] (231.011 ms) : 0, 231011
AppSec [baseline] (52.695 ms) : 0, 52695
AppSec [candidate] (52.851 ms) : 0, 52851
Debugger [baseline] (5.968 ms) : 0, 5968
Debugger [candidate] (5.971 ms) : 0, 5971
Remote Config [baseline] (601.974 µs) : 0, 602
Remote Config [candidate] (601.903 µs) : 0, 602
Telemetry [baseline] (7.951 ms) : 0, 7951
Telemetry [candidate] (7.893 ms) : 0, 7893
IAST [baseline] (27.043 ms) : 0, 27043
IAST [candidate] (26.87 ms) : 0, 26870
section profiling
BytebuddyAgent [baseline] (674.762 ms) : 0, 674762
BytebuddyAgent [candidate] (680.444 ms) : 0, 680444
GlobalTracer [baseline] (360.699 ms) : 0, 360699
GlobalTracer [candidate] (362.47 ms) : 0, 362470
AppSec [baseline] (61.53 ms) : 0, 61530
AppSec [candidate] (62.547 ms) : 0, 62547
Debugger [baseline] (6.066 ms) : 0, 6066
Debugger [candidate] (6.136 ms) : 0, 6136
Remote Config [baseline] (654.217 µs) : 0, 654
Remote Config [candidate] (637.527 µs) : 0, 638
Telemetry [baseline] (8.113 ms) : 0, 8113
Telemetry [candidate] (8.131 ms) : 0, 8131
ProfilingAgent [baseline] (102.531 ms) : 0, 102531
ProfilingAgent [candidate] (103.71 ms) : 0, 103710
Profiling [baseline] (102.555 ms) : 0, 102555
Profiling [candidate] (103.735 ms) : 0, 103735
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 6 performance regressions! Performance is the same for 0 metrics, 8 unstable metrics.
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.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1462, 1485
. : milestone, 1473,
appsec (2.389 ms) : 2341, 2438
. : milestone, 2389,
iast (2.18 ms) : 2119, 2241
. : milestone, 2180,
iast_GLOBAL (2.229 ms) : 2168, 2291
. : milestone, 2229,
profiling (2.022 ms) : 1973, 2071
. : milestone, 2022,
tracing (1.995 ms) : 1947, 2042
. : milestone, 1995,
section candidate
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (2.398 ms) : 2350, 2447
. : milestone, 2398,
iast (2.181 ms) : 2120, 2242
. : milestone, 2181,
iast_GLOBAL (2.221 ms) : 2159, 2283
. : milestone, 2221,
profiling (2.037 ms) : 1988, 2087
. : milestone, 2037,
tracing (1.994 ms) : 1946, 2041
. : milestone, 1994,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~99980be4b4, baseline=1.50.0-SNAPSHOT~7787af738f
dateFormat X
axisFormat %s
section baseline
no_agent (15.049 s) : 15049000, 15049000
. : milestone, 15049000,
appsec (14.908 s) : 14908000, 14908000
. : milestone, 14908000,
iast (18.467 s) : 18467000, 18467000
. : milestone, 18467000,
iast_GLOBAL (18.226 s) : 18226000, 18226000
. : milestone, 18226000,
profiling (15.403 s) : 15403000, 15403000
. : milestone, 15403000,
tracing (14.962 s) : 14962000, 14962000
. : milestone, 14962000,
section candidate
no_agent (15.385 s) : 15385000, 15385000
. : milestone, 15385000,
appsec (14.888 s) : 14888000, 14888000
. : milestone, 14888000,
iast (18.678 s) : 18678000, 18678000
. : milestone, 18678000,
iast_GLOBAL (17.905 s) : 17905000, 17905000
. : milestone, 17905000,
profiling (15.111 s) : 15111000, 15111000
. : milestone, 15111000,
tracing (14.924 s) : 14924000, 14924000
. : milestone, 14924000,
|
...visibility/src/main/java/datadog/trace/civisibility/config/ExecutionSettingsFactoryImpl.java
Show resolved
Hide resolved
...rnal-api/src/main/java/datadog/trace/api/civisibility/telemetry/CiVisibilityCountMetric.java
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
...nt/agent-ci-visibility/src/main/java/datadog/trace/civisibility/git/tree/ShellGitClient.java
Outdated
Show resolved
Hide resolved
itrEnabled = true | ||
codeCoverageEnabled = true | ||
testsSkippingEnabled = true | ||
flakyRetriesEnabled = false | ||
impactedTestsDetectionEnabled = false | ||
knownTestsEnabled = false | ||
testManagementEnabled = false | ||
attemptToFixRetries = 0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The mock backend now sets each setting back to its default value before every test run, which was causing some tests to be marked with enabled settings which shouldn't be. Wasn't failing on CI because the only case of this happening just added test_management.enabled
to the traces
What Does This Do
Updates the logic behind the impacted tests feature:
default_branch
to the expected response from the Settings endpointContributor 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]