-
Notifications
You must be signed in to change notification settings - Fork 319
Only enable client side stats if the host agent is at least 7.65.0 #10041
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 59 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.088 s) : 0, 1087770
Total [baseline] (10.951 s) : 0, 10950865
Agent [candidate] (1.082 s) : 0, 1082404
Total [candidate] (10.879 s) : 0, 10878680
section appsec
Agent [baseline] (1.262 s) : 0, 1262199
Total [baseline] (11.112 s) : 0, 11111752
Agent [candidate] (1.266 s) : 0, 1265620
Total [candidate] (11.141 s) : 0, 11141052
section iast
Agent [baseline] (1.223 s) : 0, 1222516
Total [baseline] (11.13 s) : 0, 11129535
Agent [candidate] (1.23 s) : 0, 1229584
Total [candidate] (11.202 s) : 0, 11202135
section profiling
Agent [baseline] (1.203 s) : 0, 1202745
Total [baseline] (11.091 s) : 0, 11090965
Agent [candidate] (1.21 s) : 0, 1210060
Total [candidate] (11.048 s) : 0, 11047962
gantt
title petclinic - break down per module: candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.215 ms) : 0, 1215
crashtracking [candidate] (1.19 ms) : 0, 1190
BytebuddyAgent [baseline] (652.298 ms) : 0, 652298
BytebuddyAgent [candidate] (649.168 ms) : 0, 649168
GlobalTracer [baseline] (283.426 ms) : 0, 283426
GlobalTracer [candidate] (282.57 ms) : 0, 282570
AppSec [baseline] (32.501 ms) : 0, 32501
AppSec [candidate] (32.384 ms) : 0, 32384
Debugger [baseline] (69.095 ms) : 0, 69095
Debugger [candidate] (68.168 ms) : 0, 68168
Remote Config [baseline] (654.755 µs) : 0, 655
Remote Config [candidate] (645.513 µs) : 0, 646
Telemetry [baseline] (8.994 ms) : 0, 8994
Telemetry [candidate] (8.945 ms) : 0, 8945
Flare Poller [baseline] (3.727 ms) : 0, 3727
Flare Poller [candidate] (3.715 ms) : 0, 3715
section appsec
crashtracking [baseline] (1.188 ms) : 0, 1188
crashtracking [candidate] (1.229 ms) : 0, 1229
BytebuddyAgent [baseline] (688.091 ms) : 0, 688091
BytebuddyAgent [candidate] (689.943 ms) : 0, 689943
GlobalTracer [baseline] (258.696 ms) : 0, 258696
GlobalTracer [candidate] (259.317 ms) : 0, 259317
IAST [baseline] (24.433 ms) : 0, 24433
IAST [candidate] (24.522 ms) : 0, 24522
AppSec [baseline] (174.397 ms) : 0, 174397
AppSec [candidate] (174.679 ms) : 0, 174679
Debugger [baseline] (66.43 ms) : 0, 66430
Debugger [candidate] (66.78 ms) : 0, 66780
Remote Config [baseline] (699.553 µs) : 0, 700
Remote Config [candidate] (704.038 µs) : 0, 704
Telemetry [baseline] (8.858 ms) : 0, 8858
Telemetry [candidate] (8.805 ms) : 0, 8805
Flare Poller [baseline] (3.844 ms) : 0, 3844
Flare Poller [candidate] (3.87 ms) : 0, 3870
section iast
crashtracking [baseline] (1.19 ms) : 0, 1190
crashtracking [candidate] (1.189 ms) : 0, 1189
BytebuddyAgent [baseline] (790.569 ms) : 0, 790569
BytebuddyAgent [candidate] (793.239 ms) : 0, 793239
GlobalTracer [baseline] (254.638 ms) : 0, 254638
GlobalTracer [candidate] (257.441 ms) : 0, 257441
IAST [baseline] (27.032 ms) : 0, 27032
IAST [candidate] (27.14 ms) : 0, 27140
AppSec [baseline] (35.232 ms) : 0, 35232
AppSec [candidate] (35.467 ms) : 0, 35467
Debugger [baseline] (65.901 ms) : 0, 65901
Debugger [candidate] (66.868 ms) : 0, 66868
Remote Config [baseline] (559.096 µs) : 0, 559
Remote Config [candidate] (573.054 µs) : 0, 573
Telemetry [baseline] (8.435 ms) : 0, 8435
Telemetry [candidate] (8.623 ms) : 0, 8623
Flare Poller [baseline] (3.441 ms) : 0, 3441
Flare Poller [candidate] (3.548 ms) : 0, 3548
section profiling
crashtracking [baseline] (1.196 ms) : 0, 1196
crashtracking [candidate] (1.195 ms) : 0, 1195
BytebuddyAgent [baseline] (700.495 ms) : 0, 700495
BytebuddyAgent [candidate] (703.8 ms) : 0, 703800
GlobalTracer [baseline] (220.462 ms) : 0, 220462
GlobalTracer [candidate] (222.337 ms) : 0, 222337
AppSec [baseline] (32.113 ms) : 0, 32113
AppSec [candidate] (32.54 ms) : 0, 32540
Debugger [baseline] (68.335 ms) : 0, 68335
Debugger [candidate] (68.902 ms) : 0, 68902
Remote Config [baseline] (642.305 µs) : 0, 642
Remote Config [candidate] (644.149 µs) : 0, 644
Telemetry [baseline] (8.865 ms) : 0, 8865
Telemetry [candidate] (9.052 ms) : 0, 9052
Flare Poller [baseline] (3.728 ms) : 0, 3728
Flare Poller [candidate] (3.786 ms) : 0, 3786
ProfilingAgent [baseline] (97.397 ms) : 0, 97397
ProfilingAgent [candidate] (98.061 ms) : 0, 98061
Profiling [baseline] (97.967 ms) : 0, 97967
Profiling [candidate] (98.648 ms) : 0, 98648
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.08 s) : 0, 1080052
Total [baseline] (8.747 s) : 0, 8746597
Agent [candidate] (1.089 s) : 0, 1088951
Total [candidate] (8.756 s) : 0, 8756418
section iast
Agent [baseline] (1.232 s) : 0, 1231540
Total [baseline] (9.466 s) : 0, 9466155
Agent [candidate] (1.222 s) : 0, 1221901
Total [candidate] (9.493 s) : 0, 9493387
gantt
title insecure-bank - break down per module: candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.206 ms) : 0, 1206
crashtracking [candidate] (1.224 ms) : 0, 1224
BytebuddyAgent [baseline] (647.774 ms) : 0, 647774
BytebuddyAgent [candidate] (653.972 ms) : 0, 653972
GlobalTracer [baseline] (281.692 ms) : 0, 281692
GlobalTracer [candidate] (283.974 ms) : 0, 283974
AppSec [baseline] (32.523 ms) : 0, 32523
AppSec [candidate] (32.809 ms) : 0, 32809
Debugger [baseline] (67.769 ms) : 0, 67769
Debugger [candidate] (67.834 ms) : 0, 67834
Remote Config [baseline] (648.612 µs) : 0, 649
Remote Config [candidate] (647.849 µs) : 0, 648
Telemetry [baseline] (9.049 ms) : 0, 9049
Telemetry [candidate] (8.985 ms) : 0, 8985
Flare Poller [baseline] (3.801 ms) : 0, 3801
Flare Poller [candidate] (3.721 ms) : 0, 3721
section iast
crashtracking [baseline] (1.211 ms) : 0, 1211
crashtracking [candidate] (1.187 ms) : 0, 1187
BytebuddyAgent [baseline] (797.654 ms) : 0, 797654
BytebuddyAgent [candidate] (789.9 ms) : 0, 789900
GlobalTracer [baseline] (256.899 ms) : 0, 256899
GlobalTracer [candidate] (255.892 ms) : 0, 255892
IAST [baseline] (27.153 ms) : 0, 27153
IAST [candidate] (26.988 ms) : 0, 26988
AppSec [baseline] (35.659 ms) : 0, 35659
AppSec [candidate] (35.611 ms) : 0, 35611
Debugger [baseline] (64.961 ms) : 0, 64961
Debugger [candidate] (64.465 ms) : 0, 64465
Remote Config [baseline] (547.565 µs) : 0, 548
Remote Config [candidate] (543.225 µs) : 0, 543
Telemetry [baseline] (8.402 ms) : 0, 8402
Telemetry [candidate] (8.413 ms) : 0, 8413
Flare Poller [baseline] (3.433 ms) : 0, 3433
Flare Poller [candidate] (3.456 ms) : 0, 3456
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 5 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.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section baseline
no_agent (1.202 ms) : 1191, 1214
. : milestone, 1202,
iast (3.052 ms) : 3020, 3085
. : milestone, 3052,
iast_FULL (5.588 ms) : 5534, 5642
. : milestone, 5588,
iast_GLOBAL (3.589 ms) : 3538, 3641
. : milestone, 3589,
profiling (2.156 ms) : 2136, 2176
. : milestone, 2156,
tracing (1.769 ms) : 1755, 1784
. : milestone, 1769,
section candidate
no_agent (1.304 ms) : 1290, 1318
. : milestone, 1304,
iast (3.174 ms) : 3131, 3217
. : milestone, 3174,
iast_FULL (5.955 ms) : 5895, 6015
. : milestone, 5955,
iast_GLOBAL (3.592 ms) : 3540, 3645
. : milestone, 3592,
profiling (1.955 ms) : 1939, 1971
. : milestone, 1955,
tracing (1.823 ms) : 1808, 1839
. : milestone, 1823,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section baseline
no_agent (18.304 ms) : 18116, 18492
. : milestone, 18304,
appsec (18.585 ms) : 18395, 18775
. : milestone, 18585,
code_origins (17.505 ms) : 17334, 17676
. : milestone, 17505,
iast (18.811 ms) : 18621, 19001
. : milestone, 18811,
profiling (18.707 ms) : 18520, 18894
. : milestone, 18707,
tracing (17.56 ms) : 17389, 17731
. : milestone, 17560,
section candidate
no_agent (19.189 ms) : 18990, 19388
. : milestone, 19189,
appsec (19.621 ms) : 19418, 19823
. : milestone, 19621,
code_origins (17.647 ms) : 17471, 17824
. : milestone, 17647,
iast (17.77 ms) : 17591, 17950
. : milestone, 17770,
profiling (19.808 ms) : 19604, 20011
. : milestone, 19808,
tracing (17.635 ms) : 17460, 17809
. : milestone, 17635,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section baseline
no_agent (14.754 s) : 14754000, 14754000
. : milestone, 14754000,
appsec (14.345 s) : 14345000, 14345000
. : milestone, 14345000,
iast (18.121 s) : 18121000, 18121000
. : milestone, 18121000,
iast_GLOBAL (17.953 s) : 17953000, 17953000
. : milestone, 17953000,
profiling (14.569 s) : 14569000, 14569000
. : milestone, 14569000,
tracing (14.797 s) : 14797000, 14797000
. : milestone, 14797000,
section candidate
no_agent (14.894 s) : 14894000, 14894000
. : milestone, 14894000,
appsec (14.702 s) : 14702000, 14702000
. : milestone, 14702000,
iast (18.128 s) : 18128000, 18128000
. : milestone, 18128000,
iast_GLOBAL (18.084 s) : 18084000, 18084000
. : milestone, 18084000,
profiling (14.71 s) : 14710000, 14710000
. : milestone, 14710000,
tracing (14.784 s) : 14784000, 14784000
. : milestone, 14784000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~71829fa545, baseline=1.57.0-SNAPSHOT~89639f7e8c
dateFormat X
axisFormat %s
section baseline
no_agent (1.487 ms) : 1475, 1499
. : milestone, 1487,
appsec (3.722 ms) : 3503, 3940
. : milestone, 3722,
iast (2.22 ms) : 2155, 2285
. : milestone, 2220,
iast_GLOBAL (2.273 ms) : 2208, 2338
. : milestone, 2273,
profiling (2.098 ms) : 2044, 2152
. : milestone, 2098,
tracing (2.056 ms) : 2004, 2107
. : milestone, 2056,
section candidate
no_agent (1.478 ms) : 1466, 1489
. : milestone, 1478,
appsec (3.721 ms) : 3503, 3939
. : milestone, 3721,
iast (2.224 ms) : 2159, 2288
. : milestone, 2224,
iast_GLOBAL (2.26 ms) : 2195, 2325
. : milestone, 2260,
profiling (2.106 ms) : 2052, 2160
. : milestone, 2106,
tracing (2.049 ms) : 1998, 2100
. : milestone, 2049,
|
communication/src/main/java/datadog/communication/ddagent/DDAgentFeaturesDiscovery.java
Outdated
Show resolved
Hide resolved
35b0813 to
414bb53
Compare
communication/src/main/java/datadog/communication/ddagent/DDAgentFeaturesDiscovery.java
Outdated
Show resolved
Hide resolved
communication/src/main/java/datadog/communication/ddagent/DDAgentFeaturesDiscovery.java
Show resolved
Hide resolved
414bb53 to
c08baa1
Compare
amarziali
left a comment
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.
thanks for the fix
|
Currently blocked on system tests, because the test agent version is hardcoded to |
…-after-agent-7.65.0
| // Assumes the 'version' is below if it can't be parsed. | ||
| int majorDot = version.indexOf('.'); | ||
| if (majorDot == -1) { | ||
| return 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.
there's a potential mistake
shouldn't it be the opposite: custom agent builds would be assumed to be more recent. The comment at the top of the function describes it correctly
false otherwise (including when version is null or unparseable)
| */ | ||
| public static boolean isVersionBelow(String version, int maxMajor, int maxMinor, int maxPatch) { | ||
| if (version == null || version.isEmpty()) { | ||
| return 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.
same as below
What Does This Do
Only enable client side stats when the agent is at least 7.65.0.
Motivation
The reason is that container based primary tags are not supported by earlier versions of the host agent and as such can't be reported when CSS is enabled.
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]