-
Notifications
You must be signed in to change notification settings - Fork 292
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
Disable smap RSS parsing by default #8342
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.
Could you also remove the logging in SmapEntryFactory.registerEvents()
when dcmd is not found, unless it is at debug level?
With those events disabled by default this initialization will be executed at startup but the functionality will be disabled.
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.049 s) : 0, 1049495
Total [baseline] (10.5 s) : 0, 10499894
Agent [candidate] (1.043 s) : 0, 1042558
Total [candidate] (10.537 s) : 0, 10536513
section appsec
Agent [baseline] (1.189 s) : 0, 1189029
Total [baseline] (10.745 s) : 0, 10744817
Agent [candidate] (1.199 s) : 0, 1199220
Total [candidate] (10.766 s) : 0, 10766339
section iast
Agent [baseline] (1.174 s) : 0, 1173847
Total [baseline] (10.943 s) : 0, 10943083
Agent [candidate] (1.179 s) : 0, 1179078
Total [candidate] (11.008 s) : 0, 11008441
section profiling
Agent [baseline] (1.28 s) : 0, 1279906
Total [baseline] (10.916 s) : 0, 10915958
Agent [candidate] (1.26 s) : 0, 1259650
Total [candidate] (10.877 s) : 0, 10877296
gantt
title petclinic - break down per module: candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (720.369 ms) : 0, 720369
BytebuddyAgent [candidate] (715.248 ms) : 0, 715248
GlobalTracer [baseline] (244.26 ms) : 0, 244260
GlobalTracer [candidate] (243.297 ms) : 0, 243297
AppSec [baseline] (55.331 ms) : 0, 55331
AppSec [candidate] (55.107 ms) : 0, 55107
Remote Config [baseline] (727.812 µs) : 0, 728
Remote Config [candidate] (712.622 µs) : 0, 713
Telemetry [baseline] (13.629 ms) : 0, 13629
Telemetry [candidate] (12.988 ms) : 0, 12988
section appsec
BytebuddyAgent [baseline] (736.35 ms) : 0, 736350
BytebuddyAgent [candidate] (743.763 ms) : 0, 743763
GlobalTracer [baseline] (240.925 ms) : 0, 240925
GlobalTracer [candidate] (242.923 ms) : 0, 242923
AppSec [baseline] (176.511 ms) : 0, 176511
AppSec [candidate] (176.819 ms) : 0, 176819
Remote Config [baseline] (661.498 µs) : 0, 661
Remote Config [candidate] (666.563 µs) : 0, 667
Telemetry [baseline] (8.288 ms) : 0, 8288
Telemetry [candidate] (8.664 ms) : 0, 8664
IAST [baseline] (21.993 ms) : 0, 21993
IAST [candidate] (22.109 ms) : 0, 22109
section iast
BytebuddyAgent [baseline] (835.556 ms) : 0, 835556
BytebuddyAgent [candidate] (839.61 ms) : 0, 839610
GlobalTracer [baseline] (233.624 ms) : 0, 233624
GlobalTracer [candidate] (234.822 ms) : 0, 234822
AppSec [baseline] (57.199 ms) : 0, 57199
AppSec [candidate] (56.971 ms) : 0, 56971
Remote Config [baseline] (629.905 µs) : 0, 630
Remote Config [candidate] (621.027 µs) : 0, 621
Telemetry [baseline] (8.719 ms) : 0, 8719
Telemetry [candidate] (8.726 ms) : 0, 8726
IAST [baseline] (22.867 ms) : 0, 22867
IAST [candidate] (23.011 ms) : 0, 23011
section profiling
ProfilingAgent [baseline] (98.478 ms) : 0, 98478
ProfilingAgent [candidate] (95.37 ms) : 0, 95370
BytebuddyAgent [baseline] (715.811 ms) : 0, 715811
BytebuddyAgent [candidate] (704.689 ms) : 0, 704689
GlobalTracer [baseline] (357.267 ms) : 0, 357267
GlobalTracer [candidate] (353.71 ms) : 0, 353710
AppSec [baseline] (56.029 ms) : 0, 56029
AppSec [candidate] (54.2 ms) : 0, 54200
Remote Config [baseline] (714.613 µs) : 0, 715
Remote Config [candidate] (719.433 µs) : 0, 719
Telemetry [baseline] (9.022 ms) : 0, 9022
Telemetry [candidate] (8.866 ms) : 0, 8866
Profiling [baseline] (98.502 ms) : 0, 98502
Profiling [candidate] (95.394 ms) : 0, 95394
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.047 s) : 0, 1046624
Total [baseline] (8.628 s) : 0, 8627729
Agent [candidate] (1.048 s) : 0, 1047521
Total [candidate] (8.63 s) : 0, 8629864
section iast
Agent [baseline] (1.173 s) : 0, 1172995
Total [baseline] (9.188 s) : 0, 9187672
Agent [candidate] (1.174 s) : 0, 1173884
Total [candidate] (9.2 s) : 0, 9199500
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.18 s) : 0, 1179613
Total [baseline] (9.209 s) : 0, 9208998
Agent [candidate] (1.172 s) : 0, 1172141
Total [candidate] (9.177 s) : 0, 9177317
section iast_TELEMETRY_OFF
Agent [baseline] (1.176 s) : 0, 1176364
Total [baseline] (9.194 s) : 0, 9193934
Agent [candidate] (1.17 s) : 0, 1169572
Total [candidate] (9.231 s) : 0, 9230665
gantt
title insecure-bank - break down per module: candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (717.853 ms) : 0, 717853
BytebuddyAgent [candidate] (721.044 ms) : 0, 721044
GlobalTracer [baseline] (243.817 ms) : 0, 243817
GlobalTracer [candidate] (243.382 ms) : 0, 243382
AppSec [baseline] (55.335 ms) : 0, 55335
AppSec [candidate] (55.06 ms) : 0, 55060
Remote Config [baseline] (706.926 µs) : 0, 707
Remote Config [candidate] (723.26 µs) : 0, 723
Telemetry [baseline] (13.671 ms) : 0, 13671
Telemetry [candidate] (12.142 ms) : 0, 12142
section iast
BytebuddyAgent [baseline] (834.926 ms) : 0, 834926
BytebuddyAgent [candidate] (835.696 ms) : 0, 835696
GlobalTracer [baseline] (233.81 ms) : 0, 233810
GlobalTracer [candidate] (234.233 ms) : 0, 234233
IAST [baseline] (22.697 ms) : 0, 22697
IAST [candidate] (22.765 ms) : 0, 22765
AppSec [baseline] (56.974 ms) : 0, 56974
AppSec [candidate] (56.771 ms) : 0, 56771
Remote Config [baseline] (605.892 µs) : 0, 606
Remote Config [candidate] (608.926 µs) : 0, 609
Telemetry [baseline] (8.721 ms) : 0, 8721
Telemetry [candidate] (8.649 ms) : 0, 8649
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (839.725 ms) : 0, 839725
BytebuddyAgent [candidate] (833.948 ms) : 0, 833948
GlobalTracer [baseline] (234.92 ms) : 0, 234920
GlobalTracer [candidate] (233.926 ms) : 0, 233926
IAST [baseline] (23.219 ms) : 0, 23219
IAST [candidate] (22.802 ms) : 0, 22802
AppSec [baseline] (56.957 ms) : 0, 56957
AppSec [candidate] (56.86 ms) : 0, 56860
Remote Config [baseline] (617.222 µs) : 0, 617
Remote Config [candidate] (627.609 µs) : 0, 628
Telemetry [baseline] (8.679 ms) : 0, 8679
Telemetry [candidate] (8.705 ms) : 0, 8705
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (837.57 ms) : 0, 837570
BytebuddyAgent [candidate] (832.382 ms) : 0, 832382
GlobalTracer [baseline] (234.453 ms) : 0, 234453
GlobalTracer [candidate] (233.661 ms) : 0, 233661
IAST [baseline] (24.905 ms) : 0, 24905
IAST [candidate] (26.344 ms) : 0, 26344
AppSec [baseline] (54.678 ms) : 0, 54678
AppSec [candidate] (52.815 ms) : 0, 52815
Remote Config [baseline] (627.661 µs) : 0, 628
Remote Config [candidate] (627.943 µs) : 0, 628
Telemetry [baseline] (8.629 ms) : 0, 8629
Telemetry [candidate] (8.519 ms) : 0, 8519
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section baseline
no_agent (376.802 µs) : 357, 397
. : milestone, 377,
iast (504.957 µs) : 483, 527
. : milestone, 505,
iast_FULL (740.484 µs) : 719, 762
. : milestone, 740,
iast_GLOBAL (545.986 µs) : 524, 568
. : milestone, 546,
iast_HARDCODED_SECRET_DISABLED (501.076 µs) : 479, 523
. : milestone, 501,
iast_INACTIVE (450.931 µs) : 429, 473
. : milestone, 451,
iast_TELEMETRY_OFF (492.688 µs) : 470, 515
. : milestone, 493,
tracing (451.994 µs) : 431, 473
. : milestone, 452,
section candidate
no_agent (374.438 µs) : 354, 395
. : milestone, 374,
iast (502.666 µs) : 481, 525
. : milestone, 503,
iast_FULL (735.94 µs) : 714, 758
. : milestone, 736,
iast_GLOBAL (546.872 µs) : 525, 569
. : milestone, 547,
iast_HARDCODED_SECRET_DISABLED (507.879 µs) : 486, 530
. : milestone, 508,
iast_INACTIVE (452.068 µs) : 431, 473
. : milestone, 452,
iast_TELEMETRY_OFF (497.282 µs) : 476, 519
. : milestone, 497,
tracing (451.557 µs) : 430, 473
. : milestone, 452,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section baseline
no_agent (1.341 ms) : 1321, 1361
. : milestone, 1341,
appsec (1.735 ms) : 1712, 1759
. : milestone, 1735,
appsec_no_iast (1.739 ms) : 1715, 1763
. : milestone, 1739,
iast (1.511 ms) : 1486, 1535
. : milestone, 1511,
profiling (1.544 ms) : 1520, 1568
. : milestone, 1544,
tracing (1.493 ms) : 1469, 1517
. : milestone, 1493,
section candidate
no_agent (1.357 ms) : 1336, 1377
. : milestone, 1357,
appsec (1.743 ms) : 1720, 1766
. : milestone, 1743,
appsec_no_iast (1.75 ms) : 1726, 1773
. : milestone, 1750,
iast (1.508 ms) : 1484, 1532
. : milestone, 1508,
profiling (1.543 ms) : 1519, 1567
. : milestone, 1543,
tracing (1.5 ms) : 1476, 1523
. : milestone, 1500,
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~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section baseline
no_agent (15.451 s) : 15451000, 15451000
. : milestone, 15451000,
appsec (15.2 s) : 15200000, 15200000
. : milestone, 15200000,
iast (18.409 s) : 18409000, 18409000
. : milestone, 18409000,
iast_GLOBAL (17.959 s) : 17959000, 17959000
. : milestone, 17959000,
profiling (15.207 s) : 15207000, 15207000
. : milestone, 15207000,
tracing (15.017 s) : 15017000, 15017000
. : milestone, 15017000,
section candidate
no_agent (15.228 s) : 15228000, 15228000
. : milestone, 15228000,
appsec (15.072 s) : 15072000, 15072000
. : milestone, 15072000,
iast (18.335 s) : 18335000, 18335000
. : milestone, 18335000,
iast_GLOBAL (18.33 s) : 18330000, 18330000
. : milestone, 18330000,
profiling (15.05 s) : 15050000, 15050000
. : milestone, 15050000,
tracing (14.843 s) : 14843000, 14843000
. : milestone, 14843000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.47.0-SNAPSHOT~a53514a155, baseline=1.47.0-SNAPSHOT~1083178bc0
dateFormat X
axisFormat %s
section baseline
no_agent (1.464 ms) : 1452, 1475
. : milestone, 1464,
appsec (2.348 ms) : 2304, 2391
. : milestone, 2348,
iast (2.101 ms) : 2046, 2155
. : milestone, 2101,
iast_GLOBAL (2.14 ms) : 2085, 2195
. : milestone, 2140,
profiling (1.994 ms) : 1948, 2041
. : milestone, 1994,
tracing (1.943 ms) : 1901, 1985
. : milestone, 1943,
section candidate
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.352 ms) : 2309, 2395
. : milestone, 2352,
iast (2.095 ms) : 2040, 2149
. : milestone, 2095,
iast_GLOBAL (2.143 ms) : 2088, 2198
. : milestone, 2143,
profiling (1.978 ms) : 1933, 2023
. : milestone, 1978,
tracing (1.936 ms) : 1894, 1978
. : milestone, 1936,
|
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.
LGTM
What Does This Do
This disables the parsing of RSS from
/proc/pid/smaps
by default to be mindful of heavily resource-constrained systems.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: PROF-10699