-
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
Fall back to ports when we cannot use auto-discovered unix domain sockets #7794
Merged
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 53 metrics, 10 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.079 s) : 0, 1078555
Total [baseline] (8.585 s) : 0, 8584623
Agent [candidate] (1.083 s) : 0, 1083415
Total [candidate] (8.586 s) : 0, 8586208
section iast
Agent [baseline] (1.202 s) : 0, 1202026
Total [baseline] (9.11 s) : 0, 9110279
Agent [candidate] (1.206 s) : 0, 1205566
Total [candidate] (9.145 s) : 0, 9145182
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.216 s) : 0, 1216051
Total [baseline] (9.117 s) : 0, 9116570
Agent [candidate] (1.208 s) : 0, 1208028
Total [candidate] (9.102 s) : 0, 9101742
section iast_TELEMETRY_OFF
Agent [baseline] (1.202 s) : 0, 1201779
Total [baseline] (9.121 s) : 0, 9121432
Agent [candidate] (1.202 s) : 0, 1201699
Total [candidate] (9.122 s) : 0, 9121978
gantt
title insecure-bank - break down per module: candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.1 ms) : 0, 688100
BytebuddyAgent [candidate] (688.606 ms) : 0, 688606
GlobalTracer [baseline] (313.701 ms) : 0, 313701
GlobalTracer [candidate] (316.113 ms) : 0, 316113
AppSec [baseline] (54.061 ms) : 0, 54061
AppSec [candidate] (54.517 ms) : 0, 54517
Remote Config [baseline] (675.234 µs) : 0, 675
Remote Config [candidate] (673.661 µs) : 0, 674
Telemetry [baseline] (8.225 ms) : 0, 8225
Telemetry [candidate] (9.724 ms) : 0, 9724
section iast
BytebuddyAgent [baseline] (800.336 ms) : 0, 800336
BytebuddyAgent [candidate] (802.221 ms) : 0, 802221
GlobalTracer [baseline] (302.481 ms) : 0, 302481
GlobalTracer [candidate] (303.453 ms) : 0, 303453
AppSec [baseline] (57.641 ms) : 0, 57641
AppSec [candidate] (56.31 ms) : 0, 56310
Remote Config [baseline] (594.413 µs) : 0, 594
Remote Config [candidate] (612.523 µs) : 0, 613
Telemetry [baseline] (7.021 ms) : 0, 7021
Telemetry [candidate] (7.092 ms) : 0, 7092
IAST [baseline] (20.194 ms) : 0, 20194
IAST [candidate] (22.035 ms) : 0, 22035
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (809.832 ms) : 0, 809832
BytebuddyAgent [candidate] (804.177 ms) : 0, 804177
GlobalTracer [baseline] (305.714 ms) : 0, 305714
GlobalTracer [candidate] (303.936 ms) : 0, 303936
AppSec [baseline] (57.561 ms) : 0, 57561
AppSec [candidate] (57.853 ms) : 0, 57853
Remote Config [baseline] (595.948 µs) : 0, 596
Remote Config [candidate] (614.38 µs) : 0, 614
Telemetry [baseline] (7.132 ms) : 0, 7132
Telemetry [candidate] (7.066 ms) : 0, 7066
IAST [baseline] (21.26 ms) : 0, 21260
IAST [candidate] (20.525 ms) : 0, 20525
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.649 ms) : 0, 799649
BytebuddyAgent [candidate] (799.372 ms) : 0, 799372
GlobalTracer [baseline] (302.859 ms) : 0, 302859
GlobalTracer [candidate] (303.475 ms) : 0, 303475
AppSec [baseline] (57.638 ms) : 0, 57638
AppSec [candidate] (56.537 ms) : 0, 56537
Remote Config [baseline] (613.11 µs) : 0, 613
Remote Config [candidate] (606.968 µs) : 0, 607
Telemetry [baseline] (7.024 ms) : 0, 7024
Telemetry [candidate] (7.83 ms) : 0, 7830
IAST [baseline] (20.208 ms) : 0, 20208
IAST [candidate] (20.057 ms) : 0, 20057
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.088 s) : 0, 1088172
Total [baseline] (10.423 s) : 0, 10422971
Agent [candidate] (1.081 s) : 0, 1081054
Total [candidate] (10.411 s) : 0, 10410578
section appsec
Agent [baseline] (1.214 s) : 0, 1213907
Total [baseline] (10.635 s) : 0, 10635099
Agent [candidate] (1.219 s) : 0, 1218910
Total [candidate] (10.595 s) : 0, 10594955
section iast
Agent [baseline] (1.206 s) : 0, 1205758
Total [baseline] (10.922 s) : 0, 10922122
Agent [candidate] (1.227 s) : 0, 1227467
Total [candidate] (10.904 s) : 0, 10904303
section profiling
Agent [baseline] (1.274 s) : 0, 1273967
Total [baseline] (10.687 s) : 0, 10687328
Agent [candidate] (1.277 s) : 0, 1277188
Total [candidate] (10.743 s) : 0, 10743166
gantt
title petclinic - break down per module: candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (693.135 ms) : 0, 693135
BytebuddyAgent [candidate] (688.565 ms) : 0, 688565
GlobalTracer [baseline] (316.349 ms) : 0, 316349
GlobalTracer [candidate] (315.534 ms) : 0, 315534
AppSec [baseline] (54.395 ms) : 0, 54395
AppSec [candidate] (54.184 ms) : 0, 54184
Remote Config [baseline] (679.884 µs) : 0, 680
Remote Config [candidate] (665.873 µs) : 0, 666
Telemetry [baseline] (9.724 ms) : 0, 9724
Telemetry [candidate] (8.293 ms) : 0, 8293
section appsec
BytebuddyAgent [baseline] (703.99 ms) : 0, 703990
BytebuddyAgent [candidate] (706.767 ms) : 0, 706767
GlobalTracer [baseline] (311.518 ms) : 0, 311518
GlobalTracer [candidate] (313.244 ms) : 0, 313244
AppSec [baseline] (167.788 ms) : 0, 167788
AppSec [candidate] (165.801 ms) : 0, 165801
Remote Config [baseline] (633.151 µs) : 0, 633
Remote Config [candidate] (648.34 µs) : 0, 648
Telemetry [baseline] (7.026 ms) : 0, 7026
Telemetry [candidate] (7.839 ms) : 0, 7839
IAST [baseline] (18.358 ms) : 0, 18358
IAST [candidate] (21.61 ms) : 0, 21610
section iast
BytebuddyAgent [baseline] (802.963 ms) : 0, 802963
BytebuddyAgent [candidate] (817.739 ms) : 0, 817739
GlobalTracer [baseline] (302.853 ms) : 0, 302853
GlobalTracer [candidate] (308.208 ms) : 0, 308208
AppSec [baseline] (56.235 ms) : 0, 56235
AppSec [candidate] (58.013 ms) : 0, 58013
Remote Config [baseline] (607.478 µs) : 0, 607
Remote Config [candidate] (613.263 µs) : 0, 613
Telemetry [baseline] (7.051 ms) : 0, 7051
Telemetry [candidate] (7.232 ms) : 0, 7232
IAST [baseline] (22.25 ms) : 0, 22250
IAST [candidate] (21.626 ms) : 0, 21626
section profiling
BytebuddyAgent [baseline] (678.94 ms) : 0, 678940
BytebuddyAgent [candidate] (681.347 ms) : 0, 681347
GlobalTracer [baseline] (397.943 ms) : 0, 397943
GlobalTracer [candidate] (398.262 ms) : 0, 398262
AppSec [baseline] (54.418 ms) : 0, 54418
AppSec [candidate] (54.651 ms) : 0, 54651
Remote Config [baseline] (657.49 µs) : 0, 657
Remote Config [candidate] (656.654 µs) : 0, 657
Telemetry [baseline] (14.397 ms) : 0, 14397
Telemetry [candidate] (13.662 ms) : 0, 13662
ProfilingAgent [baseline] (88.777 ms) : 0, 88777
ProfilingAgent [candidate] (89.606 ms) : 0, 89606
Profiling [baseline] (88.801 ms) : 0, 88801
Profiling [candidate] (89.629 ms) : 0, 89629
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section baseline
no_agent (363.919 µs) : 344, 384
. : milestone, 364,
iast (484.252 µs) : 463, 506
. : milestone, 484,
iast_FULL (550.667 µs) : 530, 572
. : milestone, 551,
iast_GLOBAL (506.496 µs) : 485, 528
. : milestone, 506,
iast_HARDCODED_SECRET_DISABLED (481.67 µs) : 461, 503
. : milestone, 482,
iast_INACTIVE (445.341 µs) : 425, 466
. : milestone, 445,
iast_TELEMETRY_OFF (473.017 µs) : 452, 494
. : milestone, 473,
tracing (444.103 µs) : 424, 465
. : milestone, 444,
section candidate
no_agent (370.885 µs) : 351, 391
. : milestone, 371,
iast (484.76 µs) : 463, 506
. : milestone, 485,
iast_FULL (553.792 µs) : 532, 575
. : milestone, 554,
iast_GLOBAL (514.663 µs) : 493, 537
. : milestone, 515,
iast_HARDCODED_SECRET_DISABLED (499.429 µs) : 478, 521
. : milestone, 499,
iast_INACTIVE (447.892 µs) : 427, 469
. : milestone, 448,
iast_TELEMETRY_OFF (470.524 µs) : 449, 492
. : milestone, 471,
tracing (446.865 µs) : 425, 468
. : milestone, 447,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section baseline
no_agent (1.346 ms) : 1327, 1365
. : milestone, 1346,
appsec (1.7 ms) : 1677, 1723
. : milestone, 1700,
appsec_no_iast (1.705 ms) : 1681, 1730
. : milestone, 1705,
iast (1.488 ms) : 1466, 1509
. : milestone, 1488,
profiling (1.485 ms) : 1463, 1508
. : milestone, 1485,
tracing (1.487 ms) : 1463, 1511
. : milestone, 1487,
section candidate
no_agent (1.346 ms) : 1327, 1364
. : milestone, 1346,
appsec (1.723 ms) : 1698, 1748
. : milestone, 1723,
appsec_no_iast (1.742 ms) : 1717, 1767
. : milestone, 1742,
iast (1.468 ms) : 1444, 1491
. : milestone, 1468,
profiling (1.485 ms) : 1460, 1510
. : milestone, 1485,
tracing (1.488 ms) : 1463, 1512
. : milestone, 1488,
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.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section baseline
no_agent (15.143 s) : 15143000, 15143000
. : milestone, 15143000,
appsec (15.037 s) : 15037000, 15037000
. : milestone, 15037000,
iast (18.634 s) : 18634000, 18634000
. : milestone, 18634000,
iast_GLOBAL (18.061 s) : 18061000, 18061000
. : milestone, 18061000,
profiling (15.784 s) : 15784000, 15784000
. : milestone, 15784000,
tracing (15.105 s) : 15105000, 15105000
. : milestone, 15105000,
section candidate
no_agent (14.993 s) : 14993000, 14993000
. : milestone, 14993000,
appsec (15.198 s) : 15198000, 15198000
. : milestone, 15198000,
iast (19.267 s) : 19267000, 19267000
. : milestone, 19267000,
iast_GLOBAL (17.806 s) : 17806000, 17806000
. : milestone, 17806000,
profiling (15.029 s) : 15029000, 15029000
. : milestone, 15029000,
tracing (15.053 s) : 15053000, 15053000
. : milestone, 15053000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.42.0-SNAPSHOT~1461ca588d, baseline=1.42.0-SNAPSHOT~5ba267f941
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (2.32 ms) : 2279, 2362
. : milestone, 2320,
iast (2.066 ms) : 2014, 2118
. : milestone, 2066,
iast_GLOBAL (2.117 ms) : 2064, 2170
. : milestone, 2117,
profiling (1.934 ms) : 1892, 1976
. : milestone, 1934,
tracing (1.912 ms) : 1873, 1952
. : milestone, 1912,
section candidate
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.309 ms) : 2269, 2350
. : milestone, 2309,
iast (2.071 ms) : 2019, 2122
. : milestone, 2071,
iast_GLOBAL (2.117 ms) : 2064, 2170
. : milestone, 2117,
profiling (2.431 ms) : 2243, 2619
. : milestone, 2431,
tracing (1.91 ms) : 1871, 1950
. : milestone, 1910,
|
mcculls
force-pushed
the
mcculls/fallback-from-uds-to-port
branch
2 times, most recently
from
October 21, 2024 09:24
2729acd
to
47683b6
Compare
…kets. This might be due to issues linking the JFFI library or permission problems.
mcculls
force-pushed
the
mcculls/fallback-from-uds-to-port
branch
from
October 21, 2024 12:58
47683b6
to
1461ca5
Compare
amarziali
approved these changes
Oct 21, 2024
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
When using the agent defaults and a unix domain socket is auto-discovered, fall back to using ports if we cannot use the socket. Reasons we might not be able to use the socket include issues linking the JFFI library and permission problems.
Likewise, if the user has explicitly configured unix domain sockets then respect that decision and don't fall back to ports.
Motivation
Helps provide a fall back when the tracer auto-discovers unix domain sockets but is then unable to use them for reasons only apparent when we try to open the socket.
Additional Notes
We don't want to try opening the socket at configuration time because it might be a blocking operation.
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: APMS-13480