Rename request body sample rate config variable#9791
Rename request body sample rate config variable#9791manuel-alvarez-alvarez merged 4 commits intomasterfrom
Conversation
jandro996
left a comment
There was a problem hiding this comment.
Do we know if any user is actually using the old env variable? I guess not, since it was released in the last version and probably isn’t in the public docs. Might be worth keeping both variables for now, just in case? I assume the answer is no, but just wanted to flag it in case you hadn’t considered it 🙂
|
🎯 Code Coverage 🔗 Commit SHA: a38383e | Docs | Datadog PR Page | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 53 metrics, 12 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.029 s) : 0, 1028597
Total [baseline] (8.67 s) : 0, 8669963
Agent [candidate] (1.029 s) : 0, 1028744
Total [candidate] (8.631 s) : 0, 8630992
section iast
Agent [baseline] (1.169 s) : 0, 1169366
Total [baseline] (9.392 s) : 0, 9392307
Agent [candidate] (1.16 s) : 0, 1159646
Total [candidate] (9.343 s) : 0, 9343413
gantt
title insecure-bank - break down per module: candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.464 ms) : 0, 1464
crashtracking [candidate] (1.451 ms) : 0, 1451
BytebuddyAgent [baseline] (702.53 ms) : 0, 702530
BytebuddyAgent [candidate] (701.363 ms) : 0, 701363
GlobalTracer [baseline] (244.091 ms) : 0, 244091
GlobalTracer [candidate] (244.576 ms) : 0, 244576
AppSec [baseline] (32.256 ms) : 0, 32256
AppSec [candidate] (32.18 ms) : 0, 32180
Debugger [baseline] (6.324 ms) : 0, 6324
Debugger [candidate] (6.374 ms) : 0, 6374
Remote Config [baseline] (684.869 µs) : 0, 685
Remote Config [candidate] (684.689 µs) : 0, 685
Telemetry [baseline] (13.587 ms) : 0, 13587
Telemetry [candidate] (14.588 ms) : 0, 14588
Flare Poller [baseline] (6.615 ms) : 0, 6615
Flare Poller [candidate] (6.514 ms) : 0, 6514
section iast
crashtracking [baseline] (1.478 ms) : 0, 1478
crashtracking [candidate] (1.46 ms) : 0, 1460
BytebuddyAgent [baseline] (830.629 ms) : 0, 830629
BytebuddyAgent [candidate] (822.512 ms) : 0, 822512
GlobalTracer [baseline] (235.05 ms) : 0, 235050
GlobalTracer [candidate] (233.639 ms) : 0, 233639
IAST [baseline] (32.651 ms) : 0, 32651
IAST [candidate] (32.648 ms) : 0, 32648
AppSec [baseline] (28.768 ms) : 0, 28768
AppSec [candidate] (28.715 ms) : 0, 28715
Debugger [baseline] (6.146 ms) : 0, 6146
Debugger [candidate] (6.173 ms) : 0, 6173
Remote Config [baseline] (613.393 µs) : 0, 613
Remote Config [candidate] (600.318 µs) : 0, 600
Telemetry [baseline] (8.48 ms) : 0, 8480
Telemetry [candidate] (8.505 ms) : 0, 8505
Flare Poller [baseline] (4.114 ms) : 0, 4114
Flare Poller [candidate] (4.17 ms) : 0, 4170
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.036 s) : 0, 1035546
Total [baseline] (10.948 s) : 0, 10947877
Agent [candidate] (1.029 s) : 0, 1028575
Total [candidate] (10.89 s) : 0, 10889631
section appsec
Agent [baseline] (1.201 s) : 0, 1201036
Total [baseline] (10.919 s) : 0, 10919056
Agent [candidate] (1.201 s) : 0, 1201140
Total [candidate] (10.922 s) : 0, 10921578
section iast
Agent [baseline] (1.18 s) : 0, 1179813
Total [baseline] (11.126 s) : 0, 11125666
Agent [candidate] (1.158 s) : 0, 1157888
Total [candidate] (11.053 s) : 0, 11052730
section profiling
Agent [baseline] (1.173 s) : 0, 1172904
Total [baseline] (10.914 s) : 0, 10914072
Agent [candidate] (1.179 s) : 0, 1179346
Total [candidate] (10.912 s) : 0, 10912061
gantt
title petclinic - break down per module: candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.487 ms) : 0, 1487
crashtracking [candidate] (1.463 ms) : 0, 1463
BytebuddyAgent [baseline] (706.774 ms) : 0, 706774
BytebuddyAgent [candidate] (700.951 ms) : 0, 700951
GlobalTracer [baseline] (246.015 ms) : 0, 246015
GlobalTracer [candidate] (244.812 ms) : 0, 244812
AppSec [baseline] (32.617 ms) : 0, 32617
AppSec [candidate] (32.248 ms) : 0, 32248
Debugger [baseline] (6.453 ms) : 0, 6453
Debugger [candidate] (6.353 ms) : 0, 6353
Remote Config [baseline] (676.404 µs) : 0, 676
Remote Config [candidate] (669.079 µs) : 0, 669
Telemetry [baseline] (13.674 ms) : 0, 13674
Telemetry [candidate] (14.522 ms) : 0, 14522
Flare Poller [baseline] (6.569 ms) : 0, 6569
Flare Poller [candidate] (6.426 ms) : 0, 6426
section appsec
crashtracking [baseline] (1.463 ms) : 0, 1463
crashtracking [candidate] (1.466 ms) : 0, 1466
BytebuddyAgent [baseline] (725.171 ms) : 0, 725171
BytebuddyAgent [candidate] (725.456 ms) : 0, 725456
GlobalTracer [baseline] (235.517 ms) : 0, 235517
GlobalTracer [candidate] (235.625 ms) : 0, 235625
IAST [baseline] (24.971 ms) : 0, 24971
IAST [candidate] (25.016 ms) : 0, 25016
AppSec [baseline] (174.011 ms) : 0, 174011
AppSec [candidate] (173.776 ms) : 0, 173776
Debugger [baseline] (5.913 ms) : 0, 5913
Debugger [candidate] (5.892 ms) : 0, 5892
Remote Config [baseline] (627.575 µs) : 0, 628
Remote Config [candidate] (624.553 µs) : 0, 625
Telemetry [baseline] (8.38 ms) : 0, 8380
Telemetry [candidate] (8.329 ms) : 0, 8329
Flare Poller [baseline] (3.895 ms) : 0, 3895
Flare Poller [candidate] (3.842 ms) : 0, 3842
section iast
crashtracking [baseline] (1.492 ms) : 0, 1492
crashtracking [candidate] (1.476 ms) : 0, 1476
BytebuddyAgent [baseline] (839.633 ms) : 0, 839633
BytebuddyAgent [candidate] (821.045 ms) : 0, 821045
GlobalTracer [baseline] (235.378 ms) : 0, 235378
GlobalTracer [candidate] (233.251 ms) : 0, 233251
IAST [baseline] (32.85 ms) : 0, 32850
IAST [candidate] (31.846 ms) : 0, 31846
AppSec [baseline] (29.025 ms) : 0, 29025
AppSec [candidate] (29.726 ms) : 0, 29726
Debugger [baseline] (6.224 ms) : 0, 6224
Debugger [candidate] (6.09 ms) : 0, 6090
Remote Config [baseline] (610.263 µs) : 0, 610
Remote Config [candidate] (606.58 µs) : 0, 607
Telemetry [baseline] (8.711 ms) : 0, 8711
Telemetry [candidate] (8.444 ms) : 0, 8444
Flare Poller [baseline] (4.15 ms) : 0, 4150
Flare Poller [candidate] (4.159 ms) : 0, 4159
section profiling
crashtracking [baseline] (1.456 ms) : 0, 1456
crashtracking [candidate] (1.468 ms) : 0, 1468
BytebuddyAgent [baseline] (726.307 ms) : 0, 726307
BytebuddyAgent [candidate] (731.547 ms) : 0, 731547
GlobalTracer [baseline] (220.166 ms) : 0, 220166
GlobalTracer [candidate] (220.863 ms) : 0, 220863
AppSec [baseline] (32.258 ms) : 0, 32258
AppSec [candidate] (33.148 ms) : 0, 33148
Debugger [baseline] (11.468 ms) : 0, 11468
Debugger [candidate] (10.506 ms) : 0, 10506
Remote Config [baseline] (728.464 µs) : 0, 728
Remote Config [candidate] (683.278 µs) : 0, 683
Telemetry [baseline] (11.412 ms) : 0, 11412
Telemetry [candidate] (11.366 ms) : 0, 11366
Flare Poller [baseline] (4.116 ms) : 0, 4116
Flare Poller [candidate] (4.149 ms) : 0, 4149
ProfilingAgent [baseline] (109.498 ms) : 0, 109498
ProfilingAgent [candidate] (109.868 ms) : 0, 109868
Profiling [baseline] (110.139 ms) : 0, 110139
Profiling [candidate] (110.55 ms) : 0, 110550
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 5 performance regressions! Performance is the same for 5 metrics, 12 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section baseline
no_agent (1.206 ms) : 1194, 1218
. : milestone, 1206,
iast (3.274 ms) : 3228, 3319
. : milestone, 3274,
iast_FULL (5.892 ms) : 5833, 5951
. : milestone, 5892,
iast_GLOBAL (3.596 ms) : 3545, 3648
. : milestone, 3596,
profiling (2.007 ms) : 1990, 2024
. : milestone, 2007,
tracing (1.78 ms) : 1765, 1794
. : milestone, 1780,
section candidate
no_agent (1.212 ms) : 1201, 1224
. : milestone, 1212,
iast (3.11 ms) : 3070, 3150
. : milestone, 3110,
iast_FULL (5.69 ms) : 5633, 5746
. : milestone, 5690,
iast_GLOBAL (3.619 ms) : 3564, 3673
. : milestone, 3619,
profiling (2.356 ms) : 2333, 2378
. : milestone, 2356,
tracing (1.858 ms) : 1842, 1873
. : milestone, 1858,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section baseline
no_agent (18.271 ms) : 18083, 18459
. : milestone, 18271,
appsec (19.508 ms) : 19312, 19704
. : milestone, 19508,
code_origins (18.544 ms) : 18354, 18733
. : milestone, 18544,
iast (19.694 ms) : 19498, 19891
. : milestone, 19694,
profiling (20.72 ms) : 20512, 20928
. : milestone, 20720,
tracing (17.602 ms) : 17426, 17778
. : milestone, 17602,
section candidate
no_agent (18.541 ms) : 18347, 18734
. : milestone, 18541,
appsec (20.537 ms) : 20327, 20747
. : milestone, 20537,
code_origins (19.546 ms) : 19349, 19742
. : milestone, 19546,
iast (19.638 ms) : 19442, 19834
. : milestone, 19638,
profiling (20.386 ms) : 20180, 20591
. : milestone, 20386,
tracing (19.256 ms) : 19066, 19447
. : milestone, 19256,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section baseline
no_agent (1.472 ms) : 1460, 1483
. : milestone, 1472,
appsec (2.455 ms) : 2403, 2506
. : milestone, 2455,
iast (2.214 ms) : 2151, 2278
. : milestone, 2214,
iast_GLOBAL (2.251 ms) : 2187, 2315
. : milestone, 2251,
profiling (2.055 ms) : 2003, 2106
. : milestone, 2055,
tracing (2.021 ms) : 1971, 2071
. : milestone, 2021,
section candidate
no_agent (1.476 ms) : 1465, 1488
. : milestone, 1476,
appsec (3.736 ms) : 3517, 3954
. : milestone, 3736,
iast (2.213 ms) : 2149, 2277
. : milestone, 2213,
iast_GLOBAL (2.259 ms) : 2195, 2324
. : milestone, 2259,
profiling (2.045 ms) : 1994, 2096
. : milestone, 2045,
tracing (2.016 ms) : 1967, 2066
. : milestone, 2016,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.55.0-SNAPSHOT~a38383e40d, baseline=1.55.0-SNAPSHOT~d3a0f11a5c
dateFormat X
axisFormat %s
section baseline
no_agent (15.49 s) : 15490000, 15490000
. : milestone, 15490000,
appsec (15.048 s) : 15048000, 15048000
. : milestone, 15048000,
iast (18.566 s) : 18566000, 18566000
. : milestone, 18566000,
iast_GLOBAL (18.129 s) : 18129000, 18129000
. : milestone, 18129000,
profiling (15.602 s) : 15602000, 15602000
. : milestone, 15602000,
tracing (15.172 s) : 15172000, 15172000
. : milestone, 15172000,
section candidate
no_agent (14.883 s) : 14883000, 14883000
. : milestone, 14883000,
appsec (15.194 s) : 15194000, 15194000
. : milestone, 15194000,
iast (18.465 s) : 18465000, 18465000
. : milestone, 18465000,
iast_GLOBAL (18.21 s) : 18210000, 18210000
. : milestone, 18210000,
profiling (15.254 s) : 15254000, 15254000
. : milestone, 15254000,
tracing (15.201 s) : 15201000, 15201000
. : milestone, 15201000,
|
6acbb27 to
4a5648d
Compare
Yeah, I added an alias to keep both for a while, thanks! |
ec39cb6 to
4cc6b68
Compare
34aa1f7 to
7162f2d
Compare
jandro996
left a comment
There was a problem hiding this comment.
Please take a look at the change I suggested since I think it’s relevant. Anyway, I’m approving them so I don’t block you until Wednesday :)
42ede4e to
a38383e
Compare
What Does This Do
Renames the sample rate config variable from
api-security.downstream.request.analysis.sample_ratetoapi-security.downstream.request.body.analysis.sample_rate.Motivation
Due to an issue in the RFC, both variable names were listed. Tracers should use
api-security.downstream.request.body.analysis.sample_rate.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]