-
Notifications
You must be signed in to change notification settings - Fork 300
Improve error reporting on profiler startup #8714
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
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 61 metrics, 10 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.056 s) : 0, 1056184
Total [baseline] (8.671 s) : 0, 8671307
Agent [candidate] (1.063 s) : 0, 1062785
Total [candidate] (8.654 s) : 0, 8654257
section iast
Agent [baseline] (1.192 s) : 0, 1191685
Total [baseline] (9.231 s) : 0, 9231321
Agent [candidate] (1.182 s) : 0, 1181747
Total [candidate] (9.195 s) : 0, 9195119
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.187 s) : 0, 1186540
Total [baseline] (9.206 s) : 0, 9205531
Agent [candidate] (1.185 s) : 0, 1184662
Total [candidate] (9.23 s) : 0, 9230341
section iast_TELEMETRY_OFF
Agent [baseline] (1.19 s) : 0, 1190017
Total [baseline] (9.213 s) : 0, 9213482
Agent [candidate] (1.191 s) : 0, 1190636
Total [candidate] (9.243 s) : 0, 9242579
gantt
title insecure-bank - break down per module: candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (722.366 ms) : 0, 722366
BytebuddyAgent [candidate] (723.027 ms) : 0, 723027
GlobalTracer [baseline] (238.913 ms) : 0, 238913
GlobalTracer [candidate] (239.905 ms) : 0, 239905
AppSec [baseline] (54.368 ms) : 0, 54368
AppSec [candidate] (56.24 ms) : 0, 56240
Debugger [baseline] (6.537 ms) : 0, 6537
Debugger [candidate] (4.401 ms) : 0, 4401
Remote Config [baseline] (694.401 µs) : 0, 694
Remote Config [candidate] (3.631 ms) : 0, 3631
Telemetry [baseline] (9.94 ms) : 0, 9940
Telemetry [candidate] (12.152 ms) : 0, 12152
section iast
BytebuddyAgent [baseline] (846.307 ms) : 0, 846307
BytebuddyAgent [candidate] (837.323 ms) : 0, 837323
GlobalTracer [baseline] (229.783 ms) : 0, 229783
GlobalTracer [candidate] (228.832 ms) : 0, 228832
IAST [baseline] (25.223 ms) : 0, 25223
IAST [candidate] (22.841 ms) : 0, 22841
AppSec [baseline] (51.974 ms) : 0, 51974
AppSec [candidate] (55.976 ms) : 0, 55976
Debugger [baseline] (4.216 ms) : 0, 4216
Debugger [candidate] (4.229 ms) : 0, 4229
Remote Config [baseline] (600.814 µs) : 0, 601
Remote Config [candidate] (609.415 µs) : 0, 609
Telemetry [baseline] (8.623 ms) : 0, 8623
Telemetry [candidate] (8.617 ms) : 0, 8617
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (839.821 ms) : 0, 839821
BytebuddyAgent [candidate] (840.052 ms) : 0, 840052
GlobalTracer [baseline] (229.718 ms) : 0, 229718
GlobalTracer [candidate] (229.049 ms) : 0, 229049
IAST [baseline] (23.131 ms) : 0, 23131
IAST [candidate] (22.857 ms) : 0, 22857
AppSec [baseline] (55.311 ms) : 0, 55311
AppSec [candidate] (55.757 ms) : 0, 55757
Debugger [baseline] (4.248 ms) : 0, 4248
Debugger [candidate] (4.243 ms) : 0, 4243
Remote Config [baseline] (616.375 µs) : 0, 616
Remote Config [candidate] (601.421 µs) : 0, 601
Telemetry [baseline] (8.765 ms) : 0, 8765
Telemetry [candidate] (8.658 ms) : 0, 8658
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (843.68 ms) : 0, 843680
BytebuddyAgent [candidate] (844.433 ms) : 0, 844433
GlobalTracer [baseline] (230.318 ms) : 0, 230318
GlobalTracer [candidate] (230.85 ms) : 0, 230850
IAST [baseline] (22.648 ms) : 0, 22648
IAST [candidate] (22.627 ms) : 0, 22627
AppSec [baseline] (56.324 ms) : 0, 56324
AppSec [candidate] (55.807 ms) : 0, 55807
Debugger [baseline] (4.23 ms) : 0, 4230
Debugger [candidate] (4.164 ms) : 0, 4164
Remote Config [baseline] (638.175 µs) : 0, 638
Remote Config [candidate] (626.261 µs) : 0, 626
Telemetry [baseline] (8.711 ms) : 0, 8711
Telemetry [candidate] (8.577 ms) : 0, 8577
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.057 s) : 0, 1056820
Total [baseline] (10.473 s) : 0, 10472837
Agent [candidate] (1.064 s) : 0, 1063823
Total [candidate] (10.446 s) : 0, 10445763
section appsec
Agent [baseline] (1.198 s) : 0, 1197789
Total [baseline] (10.843 s) : 0, 10843238
Agent [candidate] (1.195 s) : 0, 1195060
Total [candidate] (10.735 s) : 0, 10734932
section iast
Agent [baseline] (1.189 s) : 0, 1189022
Total [baseline] (11.02 s) : 0, 11019774
Agent [candidate] (1.185 s) : 0, 1184644
Total [candidate] (10.95 s) : 0, 10949788
section profiling
Agent [baseline] (1.328 s) : 0, 1327693
Total [baseline] (10.954 s) : 0, 10954145
Agent [candidate] (1.306 s) : 0, 1305786
Total [candidate] (10.891 s) : 0, 10890801
gantt
title petclinic - break down per module: candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (721.63 ms) : 0, 721630
BytebuddyAgent [candidate] (726.105 ms) : 0, 726105
GlobalTracer [baseline] (238.865 ms) : 0, 238865
GlobalTracer [candidate] (240.632 ms) : 0, 240632
AppSec [baseline] (55.156 ms) : 0, 55156
AppSec [candidate] (55.037 ms) : 0, 55037
Debugger [baseline] (7.238 ms) : 0, 7238
Debugger [candidate] (4.399 ms) : 0, 4399
Remote Config [baseline] (685.267 µs) : 0, 685
Remote Config [candidate] (1.453 ms) : 0, 1453
Telemetry [baseline] (9.869 ms) : 0, 9869
Telemetry [candidate] (12.774 ms) : 0, 12774
section appsec
BytebuddyAgent [baseline] (739.676 ms) : 0, 739676
BytebuddyAgent [candidate] (737.141 ms) : 0, 737141
GlobalTracer [baseline] (235.563 ms) : 0, 235563
GlobalTracer [candidate] (235.029 ms) : 0, 235029
IAST [baseline] (21.69 ms) : 0, 21690
IAST [candidate] (21.709 ms) : 0, 21709
AppSec [baseline] (175.238 ms) : 0, 175238
AppSec [candidate] (175.66 ms) : 0, 175660
Debugger [baseline] (4.206 ms) : 0, 4206
Debugger [candidate] (4.223 ms) : 0, 4223
Remote Config [baseline] (627.933 µs) : 0, 628
Remote Config [candidate] (626.766 µs) : 0, 627
Telemetry [baseline] (8.203 ms) : 0, 8203
Telemetry [candidate] (8.188 ms) : 0, 8188
section iast
BytebuddyAgent [baseline] (843.014 ms) : 0, 843014
BytebuddyAgent [candidate] (839.303 ms) : 0, 839303
GlobalTracer [baseline] (230.322 ms) : 0, 230322
GlobalTracer [candidate] (229.523 ms) : 0, 229523
IAST [baseline] (22.94 ms) : 0, 22940
IAST [candidate] (23.663 ms) : 0, 23663
AppSec [baseline] (52.697 ms) : 0, 52697
AppSec [candidate] (55.152 ms) : 0, 55152
Debugger [baseline] (4.251 ms) : 0, 4251
Debugger [candidate] (4.261 ms) : 0, 4261
Remote Config [baseline] (605.727 µs) : 0, 606
Remote Config [candidate] (606.734 µs) : 0, 607
Telemetry [baseline] (8.665 ms) : 0, 8665
Telemetry [candidate] (8.767 ms) : 0, 8767
section profiling
BytebuddyAgent [baseline] (725.795 ms) : 0, 725795
BytebuddyAgent [candidate] (711.051 ms) : 0, 711051
GlobalTracer [baseline] (379.395 ms) : 0, 379395
GlobalTracer [candidate] (375.953 ms) : 0, 375953
AppSec [baseline] (54.783 ms) : 0, 54783
AppSec [candidate] (53.145 ms) : 0, 53145
Debugger [baseline] (4.462 ms) : 0, 4462
Debugger [candidate] (4.357 ms) : 0, 4357
Remote Config [baseline] (684.679 µs) : 0, 685
Remote Config [candidate] (652.928 µs) : 0, 653
Telemetry [baseline] (9.078 ms) : 0, 9078
Telemetry [candidate] (8.965 ms) : 0, 8965
ProfilingAgent [baseline] (102.81 ms) : 0, 102810
ProfilingAgent [candidate] (101.503 ms) : 0, 101503
Profiling [baseline] (102.835 ms) : 0, 102835
Profiling [candidate] (101.529 ms) : 0, 101529
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 14 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section baseline
no_agent (390.469 µs) : 370, 411
. : milestone, 390,
iast (515.0 µs) : 492, 538
. : milestone, 515,
iast_FULL (737.786 µs) : 716, 760
. : milestone, 738,
iast_GLOBAL (568.429 µs) : 546, 591
. : milestone, 568,
iast_HARDCODED_SECRET_DISABLED (517.022 µs) : 495, 539
. : milestone, 517,
iast_INACTIVE (472.642 µs) : 451, 494
. : milestone, 473,
iast_TELEMETRY_OFF (503.543 µs) : 482, 525
. : milestone, 504,
tracing (462.318 µs) : 442, 483
. : milestone, 462,
section candidate
no_agent (389.137 µs) : 369, 409
. : milestone, 389,
iast (515.51 µs) : 494, 537
. : milestone, 516,
iast_FULL (740.186 µs) : 718, 762
. : milestone, 740,
iast_GLOBAL (565.989 µs) : 544, 588
. : milestone, 566,
iast_HARDCODED_SECRET_DISABLED (516.603 µs) : 495, 538
. : milestone, 517,
iast_INACTIVE (472.724 µs) : 451, 494
. : milestone, 473,
iast_TELEMETRY_OFF (511.369 µs) : 489, 533
. : milestone, 511,
tracing (459.707 µs) : 439, 480
. : milestone, 460,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section baseline
no_agent (1.36 ms) : 1340, 1379
. : milestone, 1360,
appsec (1.737 ms) : 1713, 1761
. : milestone, 1737,
appsec_no_iast (1.733 ms) : 1708, 1757
. : milestone, 1733,
code_origins (1.7 ms) : 1673, 1728
. : milestone, 1700,
iast (1.53 ms) : 1505, 1554
. : milestone, 1530,
profiling (1.534 ms) : 1509, 1559
. : milestone, 1534,
tracing (1.54 ms) : 1516, 1564
. : milestone, 1540,
section candidate
no_agent (1.378 ms) : 1357, 1398
. : milestone, 1378,
appsec (1.762 ms) : 1739, 1785
. : milestone, 1762,
appsec_no_iast (1.771 ms) : 1746, 1795
. : milestone, 1771,
code_origins (1.696 ms) : 1669, 1722
. : milestone, 1696,
iast (1.514 ms) : 1490, 1537
. : milestone, 1514,
profiling (1.527 ms) : 1504, 1550
. : milestone, 1527,
tracing (1.528 ms) : 1503, 1553
. : milestone, 1528,
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.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section baseline
no_agent (1.477 ms) : 1465, 1488
. : milestone, 1477,
appsec (2.359 ms) : 2315, 2402
. : milestone, 2359,
iast (2.144 ms) : 2089, 2200
. : milestone, 2144,
iast_GLOBAL (2.169 ms) : 2113, 2225
. : milestone, 2169,
profiling (1.978 ms) : 1935, 2022
. : milestone, 1978,
tracing (1.96 ms) : 1918, 2003
. : milestone, 1960,
section candidate
no_agent (1.473 ms) : 1461, 1484
. : milestone, 1473,
appsec (2.355 ms) : 2312, 2399
. : milestone, 2355,
iast (2.131 ms) : 2076, 2187
. : milestone, 2131,
iast_GLOBAL (2.178 ms) : 2122, 2234
. : milestone, 2178,
profiling (1.972 ms) : 1929, 2016
. : milestone, 1972,
tracing (1.965 ms) : 1923, 2007
. : milestone, 1965,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.49.0-SNAPSHOT~786d17e3a3, baseline=1.49.0-SNAPSHOT~6b6cbf6499
dateFormat X
axisFormat %s
section baseline
no_agent (14.937 s) : 14937000, 14937000
. : milestone, 14937000,
appsec (14.898 s) : 14898000, 14898000
. : milestone, 14898000,
iast (18.856 s) : 18856000, 18856000
. : milestone, 18856000,
iast_GLOBAL (18.337 s) : 18337000, 18337000
. : milestone, 18337000,
profiling (15.062 s) : 15062000, 15062000
. : milestone, 15062000,
tracing (14.972 s) : 14972000, 14972000
. : milestone, 14972000,
section candidate
no_agent (14.977 s) : 14977000, 14977000
. : milestone, 14977000,
appsec (15.269 s) : 15269000, 15269000
. : milestone, 15269000,
iast (18.743 s) : 18743000, 18743000
. : milestone, 18743000,
iast_GLOBAL (18.048 s) : 18048000, 18048000
. : milestone, 18048000,
profiling (15.556 s) : 15556000, 15556000
. : milestone, 15556000,
tracing (15.315 s) : 15315000, 15315000
. : milestone, 15315000,
|
febaf43
to
19b6ee4
Compare
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.
Overall, LGTM. As an eventual follow-up, let's establish what kind of telemetry we want to collect (OS, vendor, runtime...) and standardize it for the "comprehensive picture" work discussed earlier on Slack
@@ -167,7 +167,8 @@ public static synchronized void run( | |||
} | |||
} catch (final UnsupportedEnvironmentException e) { | |||
log.warn(e.getMessage()); | |||
log.debug(SEND_TELEMETRY, "Unsupported environment for Datadog profiler", e); | |||
// no need to send telemetry for this aggregate message | |||
// a detailed telemetry message has been sent from the attempts to enable the controllers |
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.
Nit: was this indentation generated by Spotless?
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.
No. This was manual. I find it a bit more readable when spotless kicks in and reformats my comments. It didn't happen this time, but you never know.
log.warn(SEND_TELEMETRY, msg, e); | ||
} | ||
} catch (IOException ignored) { | ||
// should not happen, but let's ignore it anyway' |
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.
Nit: typo
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.
Good catch.
LLM generated comment 😆
Will fix.
19b6ee4
to
786d17e
Compare
What Does This Do
This modifies the way how we report the failures to initialize profiler components to DD telemetry and enhances the visibility to why the expected temporary directory structure may not get properly created.
Motivation
Improve the supportability of the profiler configuration issues
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: PROF-11667