-
Notifications
You must be signed in to change notification settings - Fork 315
Rename request body sample rate config variable #9791
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
Rename request body sample rate config variable #9791
Conversation
jandro996
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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]