-
Notifications
You must be signed in to change notification settings - Fork 279
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
enable timeline events by default when ddprof disabled #7224
Conversation
@@ -216,7 +216,7 @@ public final class ProfilingConfig { | |||
|
|||
public static final String PROFILING_TIMELINE_EVENTS_ENABLED = | |||
"profiling.timeline.events.enabled"; | |||
public static final boolean PROFILING_TIMELINE_EVENTS_ENABLED_DEFAULT = false; | |||
public static final boolean PROFILING_TIMELINE_EVENTS_ENABLED_DEFAULT = true; |
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.
This will always enable thoese events, not only when ddprof is disabled, correct?
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.
Only when ddprof is disabled (or DatadogProfilingIntegration fails to load) according to the logic in Agent#createProfilingContextIntegration
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.
Ok. Can you add a comment for the default, please?
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 51 metrics, 12 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.068 s) : 0, 1067557
Total [baseline] (8.608 s) : 0, 8607765
Agent [candidate] (1.062 s) : 0, 1061884
Total [candidate] (8.542 s) : 0, 8542423
section iast
Agent [baseline] (1.17 s) : 0, 1169863
Total [baseline] (9.001 s) : 0, 9000844
Agent [candidate] (1.171 s) : 0, 1170681
Total [candidate] (8.997 s) : 0, 8997457
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.17 s) : 0, 1170180
Total [baseline] (8.994 s) : 0, 8994175
Agent [candidate] (1.172 s) : 0, 1172238
Total [candidate] (8.959 s) : 0, 8959377
section iast_TELEMETRY_OFF
Agent [baseline] (1.169 s) : 0, 1168984
Total [baseline] (9.01 s) : 0, 9009685
Agent [candidate] (1.172 s) : 0, 1172332
Total [candidate] (9.022 s) : 0, 9021745
gantt
title insecure-bank - break down per module: candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (669.089 ms) : 0, 669089
BytebuddyAgent [candidate] (665.802 ms) : 0, 665802
GlobalTracer [baseline] (305.203 ms) : 0, 305203
GlobalTracer [candidate] (303.506 ms) : 0, 303506
AppSec [baseline] (50.452 ms) : 0, 50452
AppSec [candidate] (50.009 ms) : 0, 50009
Remote Config [baseline] (684.467 µs) : 0, 684
Remote Config [candidate] (678.093 µs) : 0, 678
Telemetry [baseline] (7.59 ms) : 0, 7590
Telemetry [candidate] (7.568 ms) : 0, 7568
section iast
BytebuddyAgent [baseline] (780.106 ms) : 0, 780106
BytebuddyAgent [candidate] (779.953 ms) : 0, 779953
GlobalTracer [baseline] (293.078 ms) : 0, 293078
GlobalTracer [candidate] (293.218 ms) : 0, 293218
AppSec [baseline] (47.09 ms) : 0, 47090
AppSec [candidate] (47.231 ms) : 0, 47231
IAST [baseline] (28.809 ms) : 0, 28809
IAST [candidate] (28.495 ms) : 0, 28495
Remote Config [baseline] (628.939 µs) : 0, 629
Remote Config [candidate] (682.421 µs) : 0, 682
Telemetry [baseline] (6.891 ms) : 0, 6891
Telemetry [candidate] (7.823 ms) : 0, 7823
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (780.377 ms) : 0, 780377
BytebuddyAgent [candidate] (781.345 ms) : 0, 781345
GlobalTracer [baseline] (293.767 ms) : 0, 293767
GlobalTracer [candidate] (294.167 ms) : 0, 294167
AppSec [baseline] (47.374 ms) : 0, 47374
AppSec [candidate] (47.345 ms) : 0, 47345
IAST [baseline] (27.735 ms) : 0, 27735
IAST [candidate] (27.515 ms) : 0, 27515
Remote Config [baseline] (647.434 µs) : 0, 647
Remote Config [candidate] (698.819 µs) : 0, 699
Telemetry [baseline] (6.963 ms) : 0, 6963
Telemetry [candidate] (7.814 ms) : 0, 7814
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (779.977 ms) : 0, 779977
BytebuddyAgent [candidate] (782.215 ms) : 0, 782215
GlobalTracer [baseline] (294.946 ms) : 0, 294946
GlobalTracer [candidate] (293.588 ms) : 0, 293588
AppSec [baseline] (47.081 ms) : 0, 47081
AppSec [candidate] (47.036 ms) : 0, 47036
IAST [baseline] (25.319 ms) : 0, 25319
IAST [candidate] (27.064 ms) : 0, 27064
Remote Config [baseline] (603.449 µs) : 0, 603
Remote Config [candidate] (593.382 µs) : 0, 593
Telemetry [baseline] (7.725 ms) : 0, 7725
Telemetry [candidate] (8.437 ms) : 0, 8437
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.061 s) : 0, 1061258
Total [baseline] (10.457 s) : 0, 10457095
Agent [candidate] (1.061 s) : 0, 1061048
Total [candidate] (10.373 s) : 0, 10373351
section appsec
Agent [baseline] (1.186 s) : 0, 1185928
Total [baseline] (10.456 s) : 0, 10456394
Agent [candidate] (1.182 s) : 0, 1182132
Total [candidate] (10.494 s) : 0, 10494153
section iast
Agent [baseline] (1.171 s) : 0, 1171163
Total [baseline] (10.65 s) : 0, 10650079
Agent [candidate] (1.173 s) : 0, 1172765
Total [candidate] (10.73 s) : 0, 10730018
section profiling
Agent [baseline] (1.272 s) : 0, 1272495
Total [baseline] (10.722 s) : 0, 10721648
Agent [candidate] (1.268 s) : 0, 1268421
Total [candidate] (10.688 s) : 0, 10687525
gantt
title petclinic - break down per module: candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (665.228 ms) : 0, 665228
BytebuddyAgent [candidate] (665.115 ms) : 0, 665115
GlobalTracer [baseline] (303.629 ms) : 0, 303629
GlobalTracer [candidate] (303.148 ms) : 0, 303148
AppSec [baseline] (49.891 ms) : 0, 49891
AppSec [candidate] (50.289 ms) : 0, 50289
Remote Config [baseline] (687.019 µs) : 0, 687
Remote Config [candidate] (682.195 µs) : 0, 682
Telemetry [baseline] (7.554 ms) : 0, 7554
Telemetry [candidate] (7.572 ms) : 0, 7572
section appsec
BytebuddyAgent [baseline] (678.355 ms) : 0, 678355
BytebuddyAgent [candidate] (676.484 ms) : 0, 676484
GlobalTracer [baseline] (298.038 ms) : 0, 298038
GlobalTracer [candidate] (297.65 ms) : 0, 297650
AppSec [baseline] (154.266 ms) : 0, 154266
AppSec [candidate] (153.908 ms) : 0, 153908
Remote Config [baseline] (634.564 µs) : 0, 635
Remote Config [candidate] (630.181 µs) : 0, 630
Telemetry [baseline] (9.534 ms) : 0, 9534
Telemetry [candidate] (8.84 ms) : 0, 8840
IAST [baseline] (21.085 ms) : 0, 21085
IAST [candidate] (20.237 ms) : 0, 20237
section iast
BytebuddyAgent [baseline] (781.339 ms) : 0, 781339
BytebuddyAgent [candidate] (782.341 ms) : 0, 782341
GlobalTracer [baseline] (293.946 ms) : 0, 293946
GlobalTracer [candidate] (294.675 ms) : 0, 294675
AppSec [baseline] (47.122 ms) : 0, 47122
AppSec [candidate] (46.992 ms) : 0, 46992
Remote Config [baseline] (609.797 µs) : 0, 610
Remote Config [candidate] (645.805 µs) : 0, 646
Telemetry [baseline] (6.868 ms) : 0, 6868
Telemetry [candidate] (6.891 ms) : 0, 6891
IAST [baseline] (27.98 ms) : 0, 27980
IAST [candidate] (27.907 ms) : 0, 27907
section profiling
ProfilingAgent [baseline] (97.116 ms) : 0, 97116
ProfilingAgent [candidate] (96.527 ms) : 0, 96527
BytebuddyAgent [baseline] (668.81 ms) : 0, 668810
BytebuddyAgent [candidate] (668.03 ms) : 0, 668030
GlobalTracer [baseline] (388.84 ms) : 0, 388840
GlobalTracer [candidate] (386.601 ms) : 0, 386601
AppSec [baseline] (52.035 ms) : 0, 52035
AppSec [candidate] (51.724 ms) : 0, 51724
Remote Config [baseline] (750.805 µs) : 0, 751
Remote Config [candidate] (747.311 µs) : 0, 747
Telemetry [baseline] (7.414 ms) : 0, 7414
Telemetry [candidate] (7.442 ms) : 0, 7442
Profiling [baseline] (97.141 ms) : 0, 97141
Profiling [candidate] (96.551 ms) : 0, 96551
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section baseline
no_agent (375.855 µs) : 356, 396
. : milestone, 376,
iast (487.946 µs) : 467, 509
. : milestone, 488,
iast_FULL (562.367 µs) : 541, 584
. : milestone, 562,
iast_GLOBAL (510.063 µs) : 489, 531
. : milestone, 510,
iast_HARDCODED_SECRET_DISABLED (498.763 µs) : 477, 521
. : milestone, 499,
iast_INACTIVE (460.86 µs) : 439, 482
. : milestone, 461,
iast_TELEMETRY_OFF (483.374 µs) : 461, 506
. : milestone, 483,
tracing (450.929 µs) : 431, 471
. : milestone, 451,
section candidate
no_agent (370.375 µs) : 350, 391
. : milestone, 370,
iast (502.249 µs) : 480, 524
. : milestone, 502,
iast_FULL (560.948 µs) : 539, 583
. : milestone, 561,
iast_GLOBAL (525.753 µs) : 503, 549
. : milestone, 526,
iast_HARDCODED_SECRET_DISABLED (494.605 µs) : 473, 516
. : milestone, 495,
iast_INACTIVE (467.879 µs) : 446, 490
. : milestone, 468,
iast_TELEMETRY_OFF (483.396 µs) : 462, 505
. : milestone, 483,
tracing (444.662 µs) : 425, 465
. : milestone, 445,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section baseline
no_agent (1.35 ms) : 1330, 1370
. : milestone, 1350,
appsec (1.753 ms) : 1728, 1777
. : milestone, 1753,
appsec_no_iast (1.728 ms) : 1703, 1753
. : milestone, 1728,
iast (1.49 ms) : 1467, 1513
. : milestone, 1490,
profiling (1.513 ms) : 1488, 1538
. : milestone, 1513,
tracing (1.478 ms) : 1454, 1502
. : milestone, 1478,
section candidate
no_agent (1.37 ms) : 1351, 1390
. : milestone, 1370,
appsec (1.761 ms) : 1738, 1784
. : milestone, 1761,
appsec_no_iast (1.736 ms) : 1711, 1761
. : milestone, 1736,
iast (1.519 ms) : 1497, 1541
. : milestone, 1519,
profiling (1.56 ms) : 1534, 1586
. : milestone, 1560,
tracing (1.504 ms) : 1480, 1529
. : milestone, 1504,
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.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section baseline
no_agent (1.461 ms) : 1449, 1472
. : milestone, 1461,
appsec (2.196 ms) : 2162, 2230
. : milestone, 2196,
iast (1.956 ms) : 1915, 1998
. : milestone, 1956,
iast_GLOBAL (1.996 ms) : 1955, 2037
. : milestone, 1996,
profiling (1.839 ms) : 1806, 1873
. : milestone, 1839,
tracing (1.822 ms) : 1790, 1854
. : milestone, 1822,
section candidate
no_agent (1.462 ms) : 1450, 1473
. : milestone, 1462,
appsec (2.196 ms) : 2162, 2229
. : milestone, 2196,
iast (1.966 ms) : 1924, 2008
. : milestone, 1966,
iast_GLOBAL (1.996 ms) : 1955, 2037
. : milestone, 1996,
profiling (1.838 ms) : 1805, 1871
. : milestone, 1838,
tracing (1.819 ms) : 1787, 1851
. : milestone, 1819,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~484ef6ccba, baseline=1.36.0-SNAPSHOT~01fd984c16
dateFormat X
axisFormat %s
section baseline
no_agent (14.817 s) : 14817000, 14817000
. : milestone, 14817000,
appsec (15.122 s) : 15122000, 15122000
. : milestone, 15122000,
iast (18.748 s) : 18748000, 18748000
. : milestone, 18748000,
iast_GLOBAL (17.978 s) : 17978000, 17978000
. : milestone, 17978000,
profiling (15.243 s) : 15243000, 15243000
. : milestone, 15243000,
tracing (15.328 s) : 15328000, 15328000
. : milestone, 15328000,
section candidate
no_agent (15.023 s) : 15023000, 15023000
. : milestone, 15023000,
appsec (15.164 s) : 15164000, 15164000
. : milestone, 15164000,
iast (18.688 s) : 18688000, 18688000
. : milestone, 18688000,
iast_GLOBAL (18.058 s) : 18058000, 18058000
. : milestone, 18058000,
profiling (15.623 s) : 15623000, 15623000
. : milestone, 15623000,
tracing (15.182 s) : 15182000, 15182000
. : milestone, 15182000,
|
What Does This Do
Enables timeline events by default when JFR profiler is used, which allows for better integration into the timeline.
Motivation
Additional Notes
Jira ticket: PROF-10003