-
Notifications
You must be signed in to change notification settings - Fork 277
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
Add enabledProducts tag to all spans (APM / DSM / DJM) #6967
Conversation
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.
There is already a pattern of adding product enablement tags to root spans for appsec and profiling, I suggest following this pattern. Let's discuss offline.
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 13 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.077 s) : 0, 1077211
Total [baseline] (10.38 s) : 0, 10380472
Agent [candidate] (1.076 s) : 0, 1075630
Total [candidate] (10.377 s) : 0, 10376729
section appsec
Agent [baseline] (1.193 s) : 0, 1192825
Total [baseline] (10.467 s) : 0, 10466817
Agent [candidate] (1.194 s) : 0, 1193684
Total [candidate] (10.502 s) : 0, 10501575
section iast
Agent [baseline] (1.201 s) : 0, 1201281
Total [baseline] (10.709 s) : 0, 10708878
Agent [candidate] (1.2 s) : 0, 1200290
Total [candidate] (10.727 s) : 0, 10726594
section profiling
Agent [baseline] (1.267 s) : 0, 1266536
Total [baseline] (10.6 s) : 0, 10600071
Agent [candidate] (1.27 s) : 0, 1269617
Total [candidate] (10.624 s) : 0, 10623500
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (675.358 ms) : 0, 675358
BytebuddyAgent [candidate] (673.853 ms) : 0, 673853
GlobalTracer [baseline] (309.726 ms) : 0, 309726
GlobalTracer [candidate] (309.743 ms) : 0, 309743
AppSec [baseline] (49.448 ms) : 0, 49448
AppSec [candidate] (49.535 ms) : 0, 49535
Remote Config [baseline] (654.284 µs) : 0, 654
Remote Config [candidate] (649.26 µs) : 0, 649
Telemetry [baseline] (7.551 ms) : 0, 7551
Telemetry [candidate] (7.476 ms) : 0, 7476
section appsec
BytebuddyAgent [baseline] (697.836 ms) : 0, 697836
BytebuddyAgent [candidate] (698.036 ms) : 0, 698036
GlobalTracer [baseline] (291.612 ms) : 0, 291612
GlobalTracer [candidate] (292.617 ms) : 0, 292617
AppSec [baseline] (149.195 ms) : 0, 149195
AppSec [candidate] (149.319 ms) : 0, 149319
Remote Config [baseline] (619.476 µs) : 0, 619
Remote Config [candidate] (621.11 µs) : 0, 621
Telemetry [baseline] (9.424 ms) : 0, 9424
Telemetry [candidate] (8.831 ms) : 0, 8831
IAST [baseline] (19.219 ms) : 0, 19219
IAST [candidate] (19.387 ms) : 0, 19387
section iast
BytebuddyAgent [baseline] (795.337 ms) : 0, 795337
BytebuddyAgent [candidate] (794.761 ms) : 0, 794761
GlobalTracer [baseline] (288.694 ms) : 0, 288694
GlobalTracer [candidate] (288.507 ms) : 0, 288507
AppSec [baseline] (49.064 ms) : 0, 49064
AppSec [candidate] (50.065 ms) : 0, 50065
Remote Config [baseline] (2.055 ms) : 0, 2055
Remote Config [candidate] (1.4 ms) : 0, 1400
Telemetry [baseline] (6.666 ms) : 0, 6666
Telemetry [candidate] (6.688 ms) : 0, 6688
IAST [baseline] (24.977 ms) : 0, 24977
IAST [candidate] (24.417 ms) : 0, 24417
section profiling
BytebuddyAgent [baseline] (676.685 ms) : 0, 676685
BytebuddyAgent [candidate] (678.58 ms) : 0, 678580
GlobalTracer [baseline] (379.805 ms) : 0, 379805
GlobalTracer [candidate] (380.342 ms) : 0, 380342
AppSec [baseline] (50.144 ms) : 0, 50144
AppSec [candidate] (50.126 ms) : 0, 50126
Remote Config [baseline] (697.851 µs) : 0, 698
Remote Config [candidate] (715.103 µs) : 0, 715
Telemetry [baseline] (7.468 ms) : 0, 7468
Telemetry [candidate] (7.468 ms) : 0, 7468
ProfilingAgent [baseline] (95.46 ms) : 0, 95460
ProfilingAgent [candidate] (95.849 ms) : 0, 95849
Profiling [baseline] (95.483 ms) : 0, 95483
Profiling [candidate] (95.873 ms) : 0, 95873
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.076 s) : 0, 1076073
Total [baseline] (8.526 s) : 0, 8525601
Agent [candidate] (1.076 s) : 0, 1075607
Total [candidate] (8.563 s) : 0, 8563251
section iast
Agent [baseline] (1.198 s) : 0, 1197721
Total [baseline] (8.999 s) : 0, 8998552
Agent [candidate] (1.201 s) : 0, 1200612
Total [candidate] (8.999 s) : 0, 8999027
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.2 s) : 0, 1200134
Total [baseline] (8.98 s) : 0, 8980429
Agent [candidate] (1.203 s) : 0, 1202720
Total [candidate] (8.957 s) : 0, 8957487
section iast_TELEMETRY_OFF
Agent [baseline] (1.197 s) : 0, 1197051
Total [baseline] (8.989 s) : 0, 8989028
Agent [candidate] (1.198 s) : 0, 1197842
Total [candidate] (8.986 s) : 0, 8985513
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (674.17 ms) : 0, 674170
BytebuddyAgent [candidate] (673.822 ms) : 0, 673822
GlobalTracer [baseline] (309.846 ms) : 0, 309846
GlobalTracer [candidate] (309.92 ms) : 0, 309920
AppSec [baseline] (49.46 ms) : 0, 49460
AppSec [candidate] (49.313 ms) : 0, 49313
Remote Config [baseline] (654.085 µs) : 0, 654
Remote Config [candidate] (659.687 µs) : 0, 660
Telemetry [baseline] (7.557 ms) : 0, 7557
Telemetry [candidate] (7.564 ms) : 0, 7564
section iast
BytebuddyAgent [baseline] (793.674 ms) : 0, 793674
BytebuddyAgent [candidate] (795.761 ms) : 0, 795761
GlobalTracer [baseline] (287.739 ms) : 0, 287739
GlobalTracer [candidate] (288.129 ms) : 0, 288129
AppSec [baseline] (51.694 ms) : 0, 51694
AppSec [candidate] (51.171 ms) : 0, 51171
Remote Config [baseline] (581.185 µs) : 0, 581
Remote Config [candidate] (583.107 µs) : 0, 583
Telemetry [baseline] (6.627 ms) : 0, 6627
Telemetry [candidate] (7.39 ms) : 0, 7390
IAST [baseline] (23.141 ms) : 0, 23141
IAST [candidate] (23.177 ms) : 0, 23177
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (793.769 ms) : 0, 793769
BytebuddyAgent [candidate] (796.504 ms) : 0, 796504
GlobalTracer [baseline] (288.474 ms) : 0, 288474
GlobalTracer [candidate] (288.914 ms) : 0, 288914
AppSec [baseline] (49.281 ms) : 0, 49281
AppSec [candidate] (50.675 ms) : 0, 50675
Remote Config [baseline] (1.397 ms) : 0, 1397
Remote Config [candidate] (1.296 ms) : 0, 1296
Telemetry [baseline] (6.65 ms) : 0, 6650
Telemetry [candidate] (6.644 ms) : 0, 6644
IAST [baseline] (26.093 ms) : 0, 26093
IAST [candidate] (24.158 ms) : 0, 24158
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (792.476 ms) : 0, 792476
BytebuddyAgent [candidate] (793.217 ms) : 0, 793217
GlobalTracer [baseline] (287.855 ms) : 0, 287855
GlobalTracer [candidate] (288.81 ms) : 0, 288810
AppSec [baseline] (51.429 ms) : 0, 51429
AppSec [candidate] (51.543 ms) : 0, 51543
Remote Config [baseline] (572.012 µs) : 0, 572
Remote Config [candidate] (581.379 µs) : 0, 581
Telemetry [baseline] (6.579 ms) : 0, 6579
Telemetry [candidate] (6.589 ms) : 0, 6589
IAST [baseline] (23.738 ms) : 0, 23738
IAST [candidate] (22.752 ms) : 0, 22752
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (1.354 ms) : 1335, 1374
. : milestone, 1354,
appsec (1.724 ms) : 1699, 1749
. : milestone, 1724,
appsec_no_iast (1.741 ms) : 1717, 1765
. : milestone, 1741,
iast (1.498 ms) : 1475, 1521
. : milestone, 1498,
profiling (1.497 ms) : 1472, 1522
. : milestone, 1497,
tracing (1.463 ms) : 1438, 1489
. : milestone, 1463,
section candidate
no_agent (1.358 ms) : 1339, 1378
. : milestone, 1358,
appsec (1.729 ms) : 1704, 1753
. : milestone, 1729,
appsec_no_iast (1.708 ms) : 1684, 1733
. : milestone, 1708,
iast (1.477 ms) : 1454, 1500
. : milestone, 1477,
profiling (1.501 ms) : 1476, 1526
. : milestone, 1501,
tracing (1.495 ms) : 1470, 1520
. : milestone, 1495,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (374.944 µs) : 355, 395
. : milestone, 375,
iast (479.047 µs) : 458, 500
. : milestone, 479,
iast_FULL (541.215 µs) : 521, 562
. : milestone, 541,
iast_GLOBAL (495.957 µs) : 476, 516
. : milestone, 496,
iast_HARDCODED_SECRET_DISABLED (474.62 µs) : 453, 496
. : milestone, 475,
iast_INACTIVE (454.827 µs) : 434, 475
. : milestone, 455,
iast_TELEMETRY_OFF (470.716 µs) : 450, 492
. : milestone, 471,
tracing (449.587 µs) : 429, 470
. : milestone, 450,
section candidate
no_agent (373.001 µs) : 353, 393
. : milestone, 373,
iast (475.852 µs) : 455, 496
. : milestone, 476,
iast_FULL (539.062 µs) : 518, 560
. : milestone, 539,
iast_GLOBAL (505.377 µs) : 483, 527
. : milestone, 505,
iast_HARDCODED_SECRET_DISABLED (481.158 µs) : 459, 503
. : milestone, 481,
iast_INACTIVE (455.189 µs) : 433, 477
. : milestone, 455,
iast_TELEMETRY_OFF (473.415 µs) : 452, 495
. : milestone, 473,
tracing (452.436 µs) : 431, 474
. : milestone, 452,
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.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (1.468 ms) : 1457, 1479
. : milestone, 1468,
appsec (2.205 ms) : 2172, 2238
. : milestone, 2205,
iast (1.894 ms) : 1859, 1930
. : milestone, 1894,
iast_GLOBAL (1.927 ms) : 1892, 1963
. : milestone, 1927,
profiling (1.854 ms) : 1820, 1888
. : milestone, 1854,
tracing (1.839 ms) : 1806, 1871
. : milestone, 1839,
section candidate
no_agent (1.463 ms) : 1451, 1474
. : milestone, 1463,
appsec (2.207 ms) : 2174, 2241
. : milestone, 2207,
iast (1.874 ms) : 1839, 1909
. : milestone, 1874,
iast_GLOBAL (1.93 ms) : 1894, 1966
. : milestone, 1930,
profiling (1.847 ms) : 1814, 1881
. : milestone, 1847,
tracing (1.832 ms) : 1800, 1864
. : milestone, 1832,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~0d3e23913a, baseline=1.34.0-SNAPSHOT~16c56bf74f
dateFormat X
axisFormat %s
section baseline
no_agent (15.1 s) : 15100000, 15100000
. : milestone, 15100000,
appsec (15.111 s) : 15111000, 15111000
. : milestone, 15111000,
iast (18.611 s) : 18611000, 18611000
. : milestone, 18611000,
iast_GLOBAL (18.008 s) : 18008000, 18008000
. : milestone, 18008000,
profiling (15.204 s) : 15204000, 15204000
. : milestone, 15204000,
tracing (15.338 s) : 15338000, 15338000
. : milestone, 15338000,
section candidate
no_agent (14.917 s) : 14917000, 14917000
. : milestone, 14917000,
appsec (15.124 s) : 15124000, 15124000
. : milestone, 15124000,
iast (18.896 s) : 18896000, 18896000
. : milestone, 18896000,
iast_GLOBAL (17.732 s) : 17732000, 17732000
. : milestone, 17732000,
profiling (15.187 s) : 15187000, 15187000
. : milestone, 15187000,
tracing (14.955 s) : 14955000, 14955000
. : milestone, 14955000,
|
public boolean isDataJobsEnabled() { | ||
// there's no dedicated flag to enabled DJM, it's enough to just enable | ||
// spark instrumentation | ||
return isIntegrationEnabled(Collections.singletonList("spark"), false); | ||
} |
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.
@paul-laffon-dd, we should update this as apart of #6972. Now that this PR is merged we can do both.
What Does This Do
This PR adds product tags for DSM and DJM.
Motivation
Because customers may use APM / DJM / DSM in any configuration we need an easy way of distinguishing spans produced by the tracer (to avoid double charging in some cases).
Additional Notes
Jira