-
Notifications
You must be signed in to change notification settings - Fork 312
Fix Exception Probes to not emit any probe status #9669
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
Conversation
Debugger benchmarksParameters
See matching parameters
SummaryFound 5 performance improvements and 0 performance regressions! Performance is the same for 4 metrics, 6 unstable metrics.
See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (310.393 µs) : 287, 334
. : milestone, 310,
basic (279.954 µs) : 274, 286
. : milestone, 280,
loop (12.585 ms) : 12180, 12989
. : milestone, 12585,
section candidate
noprobe (320.741 µs) : 276, 365
. : milestone, 321,
basic (277.188 µs) : 272, 282
. : milestone, 277,
loop (8.972 ms) : 8967, 8977
. : milestone, 8972,
|
🎯 Code Coverage 🔗 Commit SHA: 2dac9e5 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.027 s) : 0, 1026957
Total [baseline] (10.673 s) : 0, 10672917
Agent [candidate] (1.018 s) : 0, 1018361
Total [candidate] (10.642 s) : 0, 10641802
section appsec
Agent [baseline] (1.192 s) : 0, 1191672
Total [baseline] (10.936 s) : 0, 10935623
Agent [candidate] (1.191 s) : 0, 1191184
Total [candidate] (10.996 s) : 0, 10996229
section iast
Agent [baseline] (1.153 s) : 0, 1152983
Total [baseline] (10.873 s) : 0, 10872657
Agent [candidate] (1.15 s) : 0, 1149758
Total [candidate] (10.979 s) : 0, 10978620
section profiling
Agent [baseline] (1.161 s) : 0, 1160640
Total [baseline] (11.035 s) : 0, 11035351
Agent [candidate] (1.165 s) : 0, 1164925
Total [candidate] (10.984 s) : 0, 10984388
gantt
title petclinic - break down per module: candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.488 ms) : 0, 1488
crashtracking [candidate] (1.462 ms) : 0, 1462
BytebuddyAgent [baseline] (700.215 ms) : 0, 700215
BytebuddyAgent [candidate] (693.088 ms) : 0, 693088
GlobalTracer [baseline] (243.728 ms) : 0, 243728
GlobalTracer [candidate] (243.051 ms) : 0, 243051
AppSec [baseline] (33.016 ms) : 0, 33016
AppSec [candidate] (32.489 ms) : 0, 32489
Debugger [baseline] (6.443 ms) : 0, 6443
Debugger [candidate] (6.311 ms) : 0, 6311
Remote Config [baseline] (701.358 µs) : 0, 701
Remote Config [candidate] (673.214 µs) : 0, 673
Telemetry [baseline] (9.331 ms) : 0, 9331
Telemetry [candidate] (9.229 ms) : 0, 9229
Flare Poller [baseline] (10.871 ms) : 0, 10871
Flare Poller [candidate] (10.871 ms) : 0, 10871
section appsec
crashtracking [baseline] (1.455 ms) : 0, 1455
crashtracking [candidate] (1.462 ms) : 0, 1462
BytebuddyAgent [baseline] (716.944 ms) : 0, 716944
BytebuddyAgent [candidate] (715.676 ms) : 0, 715676
GlobalTracer [baseline] (233.617 ms) : 0, 233617
GlobalTracer [candidate] (235.392 ms) : 0, 235392
IAST [baseline] (24.664 ms) : 0, 24664
IAST [candidate] (24.744 ms) : 0, 24744
AppSec [baseline] (174.902 ms) : 0, 174902
AppSec [candidate] (172.958 ms) : 0, 172958
Debugger [baseline] (6.132 ms) : 0, 6132
Debugger [candidate] (6.038 ms) : 0, 6038
Remote Config [baseline] (630.878 µs) : 0, 631
Remote Config [candidate] (631.695 µs) : 0, 632
Telemetry [baseline] (8.377 ms) : 0, 8377
Telemetry [candidate] (9.262 ms) : 0, 9262
Flare Poller [baseline] (3.942 ms) : 0, 3942
Flare Poller [candidate] (3.972 ms) : 0, 3972
section iast
crashtracking [baseline] (1.453 ms) : 0, 1453
crashtracking [candidate] (1.457 ms) : 0, 1457
BytebuddyAgent [baseline] (815.743 ms) : 0, 815743
BytebuddyAgent [candidate] (812.954 ms) : 0, 812954
GlobalTracer [baseline] (232.914 ms) : 0, 232914
GlobalTracer [candidate] (232.996 ms) : 0, 232996
IAST [baseline] (26.941 ms) : 0, 26941
IAST [candidate] (26.135 ms) : 0, 26135
AppSec [baseline] (35.489 ms) : 0, 35489
AppSec [candidate] (35.493 ms) : 0, 35493
Debugger [baseline] (6.017 ms) : 0, 6017
Debugger [candidate] (6.128 ms) : 0, 6128
Remote Config [baseline] (600.514 µs) : 0, 601
Remote Config [candidate] (587.84 µs) : 0, 588
Telemetry [baseline] (8.444 ms) : 0, 8444
Telemetry [candidate] (8.566 ms) : 0, 8566
Flare Poller [baseline] (4.167 ms) : 0, 4167
Flare Poller [candidate] (4.238 ms) : 0, 4238
section profiling
crashtracking [baseline] (1.418 ms) : 0, 1418
crashtracking [candidate] (1.418 ms) : 0, 1418
BytebuddyAgent [baseline] (721.328 ms) : 0, 721328
BytebuddyAgent [candidate] (722.787 ms) : 0, 722787
GlobalTracer [baseline] (217.47 ms) : 0, 217470
GlobalTracer [candidate] (219.166 ms) : 0, 219166
AppSec [baseline] (33.061 ms) : 0, 33061
AppSec [candidate] (32.728 ms) : 0, 32728
Debugger [baseline] (6.42 ms) : 0, 6420
Debugger [candidate] (8.014 ms) : 0, 8014
Remote Config [baseline] (754.119 µs) : 0, 754
Remote Config [candidate] (732.038 µs) : 0, 732
Telemetry [baseline] (16.399 ms) : 0, 16399
Telemetry [candidate] (15.068 ms) : 0, 15068
Flare Poller [baseline] (4.13 ms) : 0, 4130
Flare Poller [candidate] (4.124 ms) : 0, 4124
ProfilingAgent [baseline] (106.755 ms) : 0, 106755
ProfilingAgent [candidate] (108.216 ms) : 0, 108216
Profiling [baseline] (107.754 ms) : 0, 107754
Profiling [candidate] (109.048 ms) : 0, 109048
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.02 s) : 0, 1020241
Total [baseline] (8.672 s) : 0, 8672063
Agent [candidate] (1.021 s) : 0, 1020550
Total [candidate] (8.61 s) : 0, 8610051
section iast
Agent [baseline] (1.153 s) : 0, 1152744
Total [baseline] (9.225 s) : 0, 9224536
Agent [candidate] (1.152 s) : 0, 1152060
Total [candidate] (9.247 s) : 0, 9247124
gantt
title insecure-bank - break down per module: candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.464 ms) : 0, 1464
crashtracking [candidate] (1.454 ms) : 0, 1454
BytebuddyAgent [baseline] (695.716 ms) : 0, 695716
BytebuddyAgent [candidate] (694.128 ms) : 0, 694128
GlobalTracer [baseline] (242.277 ms) : 0, 242277
GlobalTracer [candidate] (243.598 ms) : 0, 243598
AppSec [baseline] (32.996 ms) : 0, 32996
AppSec [candidate] (32.539 ms) : 0, 32539
Debugger [baseline] (6.415 ms) : 0, 6415
Debugger [candidate] (6.339 ms) : 0, 6339
Remote Config [baseline] (686.06 µs) : 0, 686
Remote Config [candidate] (669.795 µs) : 0, 670
Telemetry [baseline] (9.373 ms) : 0, 9373
Telemetry [candidate] (9.268 ms) : 0, 9268
Flare Poller [baseline] (10.152 ms) : 0, 10152
Flare Poller [candidate] (11.462 ms) : 0, 11462
section iast
crashtracking [baseline] (1.467 ms) : 0, 1467
crashtracking [candidate] (1.458 ms) : 0, 1458
BytebuddyAgent [baseline] (815.973 ms) : 0, 815973
BytebuddyAgent [candidate] (815.212 ms) : 0, 815212
GlobalTracer [baseline] (232.671 ms) : 0, 232671
GlobalTracer [candidate] (233.3 ms) : 0, 233300
AppSec [baseline] (35.278 ms) : 0, 35278
AppSec [candidate] (35.084 ms) : 0, 35084
Debugger [baseline] (6.07 ms) : 0, 6070
Debugger [candidate] (6.174 ms) : 0, 6174
Remote Config [baseline] (597.976 µs) : 0, 598
Remote Config [candidate] (592.684 µs) : 0, 593
Telemetry [baseline] (8.531 ms) : 0, 8531
Telemetry [candidate] (8.487 ms) : 0, 8487
Flare Poller [baseline] (4.13 ms) : 0, 4130
Flare Poller [candidate] (4.192 ms) : 0, 4192
IAST [baseline] (26.72 ms) : 0, 26720
IAST [candidate] (26.352 ms) : 0, 26352
LoadParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section baseline
no_agent (4.48 ms) : 4428, 4532
. : milestone, 4480,
iast (9.747 ms) : 9584, 9910
. : milestone, 9747,
iast_FULL (14.725 ms) : 14434, 15015
. : milestone, 14725,
iast_GLOBAL (10.596 ms) : 10392, 10800
. : milestone, 10596,
profiling (9.113 ms) : 8957, 9269
. : milestone, 9113,
tracing (8.015 ms) : 7897, 8134
. : milestone, 8015,
section candidate
no_agent (4.318 ms) : 4263, 4373
. : milestone, 4318,
iast (9.573 ms) : 9411, 9735
. : milestone, 9573,
iast_FULL (14.401 ms) : 14113, 14690
. : milestone, 14401,
iast_GLOBAL (10.365 ms) : 10168, 10561
. : milestone, 10365,
profiling (8.935 ms) : 8786, 9084
. : milestone, 8935,
tracing (7.697 ms) : 7584, 7809
. : milestone, 7697,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section baseline
no_agent (37.554 ms) : 37250, 37858
. : milestone, 37554,
appsec (47.625 ms) : 47202, 48049
. : milestone, 47625,
code_origins (43.592 ms) : 43218, 43965
. : milestone, 43592,
iast (45.334 ms) : 44952, 45717
. : milestone, 45334,
profiling (50.189 ms) : 49686, 50693
. : milestone, 50189,
tracing (43.815 ms) : 43439, 44191
. : milestone, 43815,
section candidate
no_agent (36.53 ms) : 36239, 36820
. : milestone, 36530,
appsec (48.446 ms) : 48023, 48870
. : milestone, 48446,
code_origins (43.321 ms) : 42951, 43690
. : milestone, 43321,
iast (45.225 ms) : 44836, 45614
. : milestone, 45225,
profiling (46.709 ms) : 46272, 47146
. : milestone, 46709,
tracing (42.982 ms) : 42612, 43353
. : milestone, 42982,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 2 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section baseline
no_agent (1.481 ms) : 1470, 1493
. : milestone, 1481,
appsec (3.69 ms) : 3475, 3904
. : milestone, 3690,
iast (2.217 ms) : 2154, 2280
. : milestone, 2217,
iast_GLOBAL (2.275 ms) : 2211, 2339
. : milestone, 2275,
profiling (2.537 ms) : 2364, 2710
. : milestone, 2537,
tracing (2.049 ms) : 1999, 2099
. : milestone, 2049,
section candidate
no_agent (1.482 ms) : 1471, 1494
. : milestone, 1482,
appsec (3.754 ms) : 3534, 3973
. : milestone, 3754,
iast (2.231 ms) : 2167, 2295
. : milestone, 2231,
iast_GLOBAL (2.276 ms) : 2211, 2340
. : milestone, 2276,
profiling (2.065 ms) : 2014, 2117
. : milestone, 2065,
tracing (2.053 ms) : 2003, 2103
. : milestone, 2053,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.54.0-SNAPSHOT~2dac9e5ffc, baseline=1.54.0-SNAPSHOT~55230eb888
dateFormat X
axisFormat %s
section baseline
no_agent (14.932 s) : 14932000, 14932000
. : milestone, 14932000,
appsec (14.981 s) : 14981000, 14981000
. : milestone, 14981000,
iast (18.59 s) : 18590000, 18590000
. : milestone, 18590000,
iast_GLOBAL (18.022 s) : 18022000, 18022000
. : milestone, 18022000,
profiling (16.004 s) : 16004000, 16004000
. : milestone, 16004000,
tracing (14.773 s) : 14773000, 14773000
. : milestone, 14773000,
section candidate
no_agent (15.644 s) : 15644000, 15644000
. : milestone, 15644000,
appsec (14.851 s) : 14851000, 14851000
. : milestone, 14851000,
iast (18.834 s) : 18834000, 18834000
. : milestone, 18834000,
iast_GLOBAL (18.005 s) : 18005000, 18005000
. : milestone, 18005000,
profiling (15.358 s) : 15358000, 15358000
. : milestone, 15358000,
tracing (15.022 s) : 15022000, 15022000
. : milestone, 15022000,
|
filter on probe definition type
698665f
to
1c380db
Compare
64e6575
to
7798aab
Compare
def additionalDynamicTags = ["content.meta.['_dd.debug.error.3.snapshot_id']", "content.meta.['_dd.debug.error.exception_id']"] | ||
verifyEventsAndCoverages(projectName, "maven", mavenVersion, mockBackend.waitForEvents(7), mockBackend.waitForCoverages(0), additionalDynamicTags) | ||
verifySnapshotLogs(mockBackend.waitForLogs(5), 1, 2) | ||
verifySnapshotLogs(mockBackend.waitForLogs(2), 0, 2) |
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.
In the CiVis smoke tests we checked for RECEIVED, INSTALLED, and EMITTING status from the probe (apart from the test snapshots). If none of these statuses are reported anymore, I think it's safe to change this call to just verifySnapshots(mockBackend.waitForLogs(2), 2)
, making the method protected in datadog.trace.civisibility.CiVisibilitySmokeTest
. The call would need to be updated in datadog.smoketest.JUnitConsoleSmokeTest
also
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.
done
What Does This Do
filter on probe definition type
Motivation
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any useful labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: DEBUG-4559