-
Notifications
You must be signed in to change notification settings - Fork 293
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
Remove remote configuration for API Security sampling rate #8486
Merged
+3
−111
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 9 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.041 s) : 0, 1041382
Total [baseline] (8.672 s) : 0, 8672188
Agent [candidate] (1.049 s) : 0, 1049336
Total [candidate] (8.693 s) : 0, 8693031
section iast
Agent [baseline] (1.178 s) : 0, 1178368
Total [baseline] (9.242 s) : 0, 9242428
Agent [candidate] (1.172 s) : 0, 1171576
Total [candidate] (9.235 s) : 0, 9235259
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.191 s) : 0, 1190616
Total [baseline] (9.274 s) : 0, 9273573
Agent [candidate] (1.184 s) : 0, 1183874
Total [candidate] (9.232 s) : 0, 9232045
section iast_TELEMETRY_OFF
Agent [baseline] (1.164 s) : 0, 1163811
Total [baseline] (9.235 s) : 0, 9235076
Agent [candidate] (1.167 s) : 0, 1167221
Total [candidate] (9.24 s) : 0, 9240335
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (718.713 ms) : 0, 718713
BytebuddyAgent [candidate] (722.086 ms) : 0, 722086
GlobalTracer [baseline] (239.795 ms) : 0, 239795
GlobalTracer [candidate] (241.075 ms) : 0, 241075
AppSec [baseline] (55.845 ms) : 0, 55845
AppSec [candidate] (55.502 ms) : 0, 55502
Remote Config [baseline] (694.179 µs) : 0, 694
Remote Config [candidate] (701.873 µs) : 0, 702
Telemetry [baseline] (11.396 ms) : 0, 11396
Telemetry [candidate] (14.915 ms) : 0, 14915
section iast
BytebuddyAgent [baseline] (842.647 ms) : 0, 842647
BytebuddyAgent [candidate] (837.259 ms) : 0, 837259
GlobalTracer [baseline] (231.454 ms) : 0, 231454
GlobalTracer [candidate] (230.508 ms) : 0, 230508
AppSec [baseline] (56.957 ms) : 0, 56957
AppSec [candidate] (56.863 ms) : 0, 56863
Remote Config [baseline] (619.027 µs) : 0, 619
Remote Config [candidate] (613.784 µs) : 0, 614
Telemetry [baseline] (8.769 ms) : 0, 8769
Telemetry [candidate] (8.651 ms) : 0, 8651
IAST [baseline] (22.893 ms) : 0, 22893
IAST [candidate] (22.769 ms) : 0, 22769
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (850.219 ms) : 0, 850219
BytebuddyAgent [candidate] (845.996 ms) : 0, 845996
GlobalTracer [baseline] (233.401 ms) : 0, 233401
GlobalTracer [candidate] (233.025 ms) : 0, 233025
AppSec [baseline] (58.226 ms) : 0, 58226
AppSec [candidate] (56.987 ms) : 0, 56987
Remote Config [baseline] (645.944 µs) : 0, 646
Remote Config [candidate] (616.526 µs) : 0, 617
Telemetry [baseline] (9.055 ms) : 0, 9055
Telemetry [candidate] (8.699 ms) : 0, 8699
IAST [baseline] (23.924 ms) : 0, 23924
IAST [candidate] (23.351 ms) : 0, 23351
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (831.194 ms) : 0, 831194
BytebuddyAgent [candidate] (833.255 ms) : 0, 833255
GlobalTracer [baseline] (229.371 ms) : 0, 229371
GlobalTracer [candidate] (230.191 ms) : 0, 230191
AppSec [baseline] (53.658 ms) : 0, 53658
AppSec [candidate] (53.946 ms) : 0, 53946
Remote Config [baseline] (613.19 µs) : 0, 613
Remote Config [candidate] (633.041 µs) : 0, 633
Telemetry [baseline] (8.586 ms) : 0, 8586
Telemetry [candidate] (8.72 ms) : 0, 8720
IAST [baseline] (25.507 ms) : 0, 25507
IAST [candidate] (25.527 ms) : 0, 25527
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.041 s) : 0, 1040728
Total [baseline] (10.475 s) : 0, 10475213
Agent [candidate] (1.046 s) : 0, 1046275
Total [candidate] (10.556 s) : 0, 10555819
section appsec
Agent [baseline] (1.187 s) : 0, 1187001
Total [baseline] (10.782 s) : 0, 10781773
Agent [candidate] (1.191 s) : 0, 1190831
Total [candidate] (10.781 s) : 0, 10780657
section iast
Agent [baseline] (1.18 s) : 0, 1180141
Total [baseline] (11.039 s) : 0, 11038642
Agent [candidate] (1.181 s) : 0, 1180773
Total [candidate] (11.037 s) : 0, 11036686
section profiling
Agent [baseline] (1.27 s) : 0, 1270351
Total [baseline] (10.909 s) : 0, 10908554
Agent [candidate] (1.26 s) : 0, 1260326
Total [candidate] (10.926 s) : 0, 10925931
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (716.904 ms) : 0, 716904
BytebuddyAgent [candidate] (723.104 ms) : 0, 723104
GlobalTracer [baseline] (239.075 ms) : 0, 239075
GlobalTracer [candidate] (241.344 ms) : 0, 241344
AppSec [baseline] (55.638 ms) : 0, 55638
AppSec [candidate] (55.402 ms) : 0, 55402
Remote Config [baseline] (681.406 µs) : 0, 681
Remote Config [candidate] (690.101 µs) : 0, 690
Telemetry [baseline] (13.532 ms) : 0, 13532
Telemetry [candidate] (10.738 ms) : 0, 10738
section appsec
BytebuddyAgent [baseline] (737.66 ms) : 0, 737660
BytebuddyAgent [candidate] (741.146 ms) : 0, 741146
GlobalTracer [baseline] (237.059 ms) : 0, 237059
GlobalTracer [candidate] (237.475 ms) : 0, 237475
AppSec [baseline] (177.179 ms) : 0, 177179
AppSec [candidate] (177.375 ms) : 0, 177375
Remote Config [baseline] (665.804 µs) : 0, 666
Remote Config [candidate] (667.723 µs) : 0, 668
Telemetry [baseline] (8.62 ms) : 0, 8620
Telemetry [candidate] (8.311 ms) : 0, 8311
IAST [baseline] (21.743 ms) : 0, 21743
IAST [candidate] (21.608 ms) : 0, 21608
section iast
BytebuddyAgent [baseline] (842.719 ms) : 0, 842719
BytebuddyAgent [candidate] (844.076 ms) : 0, 844076
GlobalTracer [baseline] (232.169 ms) : 0, 232169
GlobalTracer [candidate] (232.203 ms) : 0, 232203
AppSec [baseline] (57.777 ms) : 0, 57777
AppSec [candidate] (56.985 ms) : 0, 56985
Remote Config [baseline] (625.958 µs) : 0, 626
Remote Config [candidate] (611.693 µs) : 0, 612
Telemetry [baseline] (8.777 ms) : 0, 8777
Telemetry [candidate] (8.683 ms) : 0, 8683
IAST [baseline] (23.078 ms) : 0, 23078
IAST [candidate] (23.1 ms) : 0, 23100
section profiling
ProfilingAgent [baseline] (97.366 ms) : 0, 97366
ProfilingAgent [candidate] (95.53 ms) : 0, 95530
BytebuddyAgent [baseline] (714.276 ms) : 0, 714276
BytebuddyAgent [candidate] (709.126 ms) : 0, 709126
GlobalTracer [baseline] (352.361 ms) : 0, 352361
GlobalTracer [candidate] (351.341 ms) : 0, 351341
AppSec [baseline] (55.974 ms) : 0, 55974
AppSec [candidate] (54.185 ms) : 0, 54185
Remote Config [baseline] (687.73 µs) : 0, 688
Remote Config [candidate] (690.202 µs) : 0, 690
Telemetry [baseline] (9.058 ms) : 0, 9058
Telemetry [candidate] (9.04 ms) : 0, 9040
Profiling [baseline] (97.391 ms) : 0, 97391
Profiling [candidate] (95.557 ms) : 0, 95557
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 14 metrics, 16 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section baseline
no_agent (1.354 ms) : 1334, 1373
. : milestone, 1354,
appsec (1.726 ms) : 1702, 1749
. : milestone, 1726,
appsec_no_iast (1.728 ms) : 1703, 1752
. : milestone, 1728,
code_origins (1.675 ms) : 1642, 1709
. : milestone, 1675,
iast (1.513 ms) : 1488, 1537
. : milestone, 1513,
profiling (1.539 ms) : 1514, 1565
. : milestone, 1539,
tracing (1.472 ms) : 1447, 1497
. : milestone, 1472,
section candidate
no_agent (1.346 ms) : 1326, 1365
. : milestone, 1346,
appsec (1.732 ms) : 1709, 1755
. : milestone, 1732,
appsec_no_iast (1.716 ms) : 1690, 1742
. : milestone, 1716,
code_origins (1.698 ms) : 1666, 1731
. : milestone, 1698,
iast (1.506 ms) : 1482, 1531
. : milestone, 1506,
profiling (1.532 ms) : 1507, 1557
. : milestone, 1532,
tracing (1.499 ms) : 1474, 1524
. : milestone, 1499,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section baseline
no_agent (377.012 µs) : 357, 397
. : milestone, 377,
iast (508.512 µs) : 487, 530
. : milestone, 509,
iast_FULL (730.171 µs) : 708, 752
. : milestone, 730,
iast_GLOBAL (563.017 µs) : 540, 586
. : milestone, 563,
iast_HARDCODED_SECRET_DISABLED (509.201 µs) : 487, 531
. : milestone, 509,
iast_INACTIVE (461.428 µs) : 440, 482
. : milestone, 461,
iast_TELEMETRY_OFF (495.571 µs) : 473, 518
. : milestone, 496,
tracing (457.728 µs) : 437, 479
. : milestone, 458,
section candidate
no_agent (375.966 µs) : 355, 396
. : milestone, 376,
iast (509.824 µs) : 487, 533
. : milestone, 510,
iast_FULL (727.349 µs) : 705, 749
. : milestone, 727,
iast_GLOBAL (575.422 µs) : 552, 599
. : milestone, 575,
iast_HARDCODED_SECRET_DISABLED (505.631 µs) : 484, 527
. : milestone, 506,
iast_INACTIVE (462.658 µs) : 441, 484
. : milestone, 463,
iast_TELEMETRY_OFF (497.544 µs) : 476, 519
. : milestone, 498,
tracing (454.123 µs) : 433, 475
. : milestone, 454,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section baseline
no_agent (15.27 s) : 15270000, 15270000
. : milestone, 15270000,
appsec (14.934 s) : 14934000, 14934000
. : milestone, 14934000,
iast (18.309 s) : 18309000, 18309000
. : milestone, 18309000,
iast_GLOBAL (17.805 s) : 17805000, 17805000
. : milestone, 17805000,
profiling (15.579 s) : 15579000, 15579000
. : milestone, 15579000,
tracing (14.935 s) : 14935000, 14935000
. : milestone, 14935000,
section candidate
no_agent (14.913 s) : 14913000, 14913000
. : milestone, 14913000,
appsec (14.938 s) : 14938000, 14938000
. : milestone, 14938000,
iast (18.689 s) : 18689000, 18689000
. : milestone, 18689000,
iast_GLOBAL (17.704 s) : 17704000, 17704000
. : milestone, 17704000,
profiling (14.959 s) : 14959000, 14959000
. : milestone, 14959000,
tracing (15.002 s) : 15002000, 15002000
. : milestone, 15002000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~5ba0e37c58, baseline=1.47.0-SNAPSHOT~7787c9ddf9
dateFormat X
axisFormat %s
section baseline
no_agent (1.468 ms) : 1457, 1480
. : milestone, 1468,
appsec (2.332 ms) : 2288, 2376
. : milestone, 2332,
iast (2.109 ms) : 2054, 2165
. : milestone, 2109,
iast_GLOBAL (2.156 ms) : 2100, 2211
. : milestone, 2156,
profiling (1.974 ms) : 1931, 2018
. : milestone, 1974,
tracing (1.936 ms) : 1895, 1978
. : milestone, 1936,
section candidate
no_agent (1.471 ms) : 1460, 1482
. : milestone, 1471,
appsec (2.342 ms) : 2298, 2386
. : milestone, 2342,
iast (2.114 ms) : 2059, 2169
. : milestone, 2114,
iast_GLOBAL (2.164 ms) : 2108, 2220
. : milestone, 2164,
profiling (1.974 ms) : 1930, 2018
. : milestone, 1974,
tracing (1.94 ms) : 1898, 1982
. : milestone, 1940,
|
sezen-datadog
approved these changes
Mar 5, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What Does This Do
Remove the old system to configure API Security sampling rate via remote config. This is currently not exposed in the UI, and the backend is being decomisioned. It was never in operation in production.
Motivation
Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: APPSEC-54874