-
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
Widen catch blocks to make agent discovery more tolerant #7796
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
…g errors, like UnsatisfiedLinkError
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 petclinicgantt
title petclinic - global startup overhead: candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.077 s) : 0, 1077442
Total [baseline] (10.385 s) : 0, 10385096
Agent [candidate] (1.079 s) : 0, 1079213
Total [candidate] (10.347 s) : 0, 10347270
section appsec
Agent [baseline] (1.212 s) : 0, 1212334
Total [baseline] (10.559 s) : 0, 10559170
Agent [candidate] (1.215 s) : 0, 1214546
Total [candidate] (10.575 s) : 0, 10574684
section iast
Agent [baseline] (1.207 s) : 0, 1207001
Total [baseline] (10.846 s) : 0, 10846364
Agent [candidate] (1.198 s) : 0, 1198331
Total [candidate] (10.865 s) : 0, 10865325
section profiling
Agent [baseline] (1.267 s) : 0, 1267301
Total [baseline] (10.654 s) : 0, 10654069
Agent [candidate] (1.272 s) : 0, 1271541
Total [candidate] (10.76 s) : 0, 10759946
gantt
title petclinic - break down per module: candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (685.807 ms) : 0, 685807
BytebuddyAgent [candidate] (685.802 ms) : 0, 685802
GlobalTracer [baseline] (312.485 ms) : 0, 312485
GlobalTracer [candidate] (313.53 ms) : 0, 313530
AppSec [baseline] (53.864 ms) : 0, 53864
AppSec [candidate] (53.779 ms) : 0, 53779
Remote Config [baseline] (681.292 µs) : 0, 681
Remote Config [candidate] (664.71 µs) : 0, 665
Telemetry [baseline] (11.006 ms) : 0, 11006
Telemetry [candidate] (11.786 ms) : 0, 11786
section appsec
BytebuddyAgent [baseline] (703.279 ms) : 0, 703279
BytebuddyAgent [candidate] (704.28 ms) : 0, 704280
GlobalTracer [baseline] (311.03 ms) : 0, 311030
GlobalTracer [candidate] (312.279 ms) : 0, 312279
AppSec [baseline] (166.598 ms) : 0, 166598
AppSec [candidate] (165.397 ms) : 0, 165397
Remote Config [baseline] (632.925 µs) : 0, 633
Remote Config [candidate] (635.92 µs) : 0, 636
Telemetry [baseline] (7.066 ms) : 0, 7066
Telemetry [candidate] (7.471 ms) : 0, 7471
IAST [baseline] (20.143 ms) : 0, 20143
IAST [candidate] (21.603 ms) : 0, 21603
section iast
BytebuddyAgent [baseline] (805.19 ms) : 0, 805190
BytebuddyAgent [candidate] (798.225 ms) : 0, 798225
GlobalTracer [baseline] (302.666 ms) : 0, 302666
GlobalTracer [candidate] (300.782 ms) : 0, 300782
AppSec [baseline] (57.563 ms) : 0, 57563
AppSec [candidate] (56.85 ms) : 0, 56850
Remote Config [baseline] (594.317 µs) : 0, 594
Remote Config [candidate] (608.471 µs) : 0, 608
Telemetry [baseline] (7.066 ms) : 0, 7066
Telemetry [candidate] (7.064 ms) : 0, 7064
IAST [baseline] (20.185 ms) : 0, 20185
IAST [candidate] (21.188 ms) : 0, 21188
section profiling
BytebuddyAgent [baseline] (676.127 ms) : 0, 676127
BytebuddyAgent [candidate] (678.708 ms) : 0, 678708
GlobalTracer [baseline] (394.394 ms) : 0, 394394
GlobalTracer [candidate] (395.42 ms) : 0, 395420
AppSec [baseline] (54.165 ms) : 0, 54165
AppSec [candidate] (54.523 ms) : 0, 54523
Remote Config [baseline] (648.824 µs) : 0, 649
Remote Config [candidate] (647.431 µs) : 0, 647
Telemetry [baseline] (14.354 ms) : 0, 14354
Telemetry [candidate] (14.423 ms) : 0, 14423
ProfilingAgent [baseline] (89.053 ms) : 0, 89053
ProfilingAgent [candidate] (89.143 ms) : 0, 89143
Profiling [baseline] (89.076 ms) : 0, 89076
Profiling [candidate] (89.166 ms) : 0, 89166
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.077 s) : 0, 1077072
Total [baseline] (8.562 s) : 0, 8562127
Agent [candidate] (1.074 s) : 0, 1073792
Total [candidate] (8.559 s) : 0, 8558712
section iast
Agent [baseline] (1.21 s) : 0, 1209504
Total [baseline] (9.087 s) : 0, 9087186
Agent [candidate] (1.197 s) : 0, 1197179
Total [candidate] (9.082 s) : 0, 9081592
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.199 s) : 0, 1198644
Total [baseline] (9.063 s) : 0, 9062976
Agent [candidate] (1.197 s) : 0, 1196645
Total [candidate] (9.083 s) : 0, 9082580
section iast_TELEMETRY_OFF
Agent [baseline] (1.194 s) : 0, 1194347
Total [baseline] (9.068 s) : 0, 9068356
Agent [candidate] (1.197 s) : 0, 1197402
Total [candidate] (9.097 s) : 0, 9096924
gantt
title insecure-bank - break down per module: candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.991 ms) : 0, 686991
BytebuddyAgent [candidate] (682.029 ms) : 0, 682029
GlobalTracer [baseline] (313.033 ms) : 0, 313033
GlobalTracer [candidate] (311.841 ms) : 0, 311841
AppSec [baseline] (53.916 ms) : 0, 53916
AppSec [candidate] (53.413 ms) : 0, 53413
Remote Config [baseline] (657.61 µs) : 0, 658
Remote Config [candidate] (662.851 µs) : 0, 663
Telemetry [baseline] (8.827 ms) : 0, 8827
Telemetry [candidate] (12.269 ms) : 0, 12269
section iast
BytebuddyAgent [baseline] (806.537 ms) : 0, 806537
BytebuddyAgent [candidate] (797.581 ms) : 0, 797581
GlobalTracer [baseline] (303.782 ms) : 0, 303782
GlobalTracer [candidate] (300.694 ms) : 0, 300694
AppSec [baseline] (56.8 ms) : 0, 56800
AppSec [candidate] (56.523 ms) : 0, 56523
Remote Config [baseline] (614.448 µs) : 0, 614
Remote Config [candidate] (622.034 µs) : 0, 622
Telemetry [baseline] (7.044 ms) : 0, 7044
Telemetry [candidate] (7.054 ms) : 0, 7054
IAST [baseline] (20.996 ms) : 0, 20996
IAST [candidate] (21.073 ms) : 0, 21073
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (798.204 ms) : 0, 798204
BytebuddyAgent [candidate] (796.586 ms) : 0, 796586
GlobalTracer [baseline] (301.071 ms) : 0, 301071
GlobalTracer [candidate] (300.996 ms) : 0, 300996
AppSec [baseline] (56.854 ms) : 0, 56854
AppSec [candidate] (57.79 ms) : 0, 57790
Remote Config [baseline] (605.409 µs) : 0, 605
Remote Config [candidate] (587.059 µs) : 0, 587
Telemetry [baseline] (7.067 ms) : 0, 7067
Telemetry [candidate] (7.024 ms) : 0, 7024
IAST [baseline] (21.225 ms) : 0, 21225
IAST [candidate] (20.072 ms) : 0, 20072
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (794.925 ms) : 0, 794925
BytebuddyAgent [candidate] (796.841 ms) : 0, 796841
GlobalTracer [baseline] (300.955 ms) : 0, 300955
GlobalTracer [candidate] (301.983 ms) : 0, 301983
AppSec [baseline] (56.568 ms) : 0, 56568
AppSec [candidate] (57.499 ms) : 0, 57499
Remote Config [baseline] (604.433 µs) : 0, 604
Remote Config [candidate] (595.703 µs) : 0, 596
Telemetry [baseline] (7.729 ms) : 0, 7729
Telemetry [candidate] (6.943 ms) : 0, 6943
IAST [baseline] (19.962 ms) : 0, 19962
IAST [candidate] (19.9 ms) : 0, 19900
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 petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (1.362 ms) : 1343, 1381
. : milestone, 1362,
appsec (1.71 ms) : 1686, 1735
. : milestone, 1710,
appsec_no_iast (1.733 ms) : 1709, 1757
. : milestone, 1733,
iast (1.494 ms) : 1471, 1516
. : milestone, 1494,
profiling (1.549 ms) : 1526, 1573
. : milestone, 1549,
tracing (1.482 ms) : 1458, 1505
. : milestone, 1482,
section candidate
no_agent (1.331 ms) : 1311, 1350
. : milestone, 1331,
appsec (1.724 ms) : 1699, 1748
. : milestone, 1724,
appsec_no_iast (1.732 ms) : 1708, 1756
. : milestone, 1732,
iast (1.478 ms) : 1455, 1501
. : milestone, 1478,
profiling (1.505 ms) : 1483, 1528
. : milestone, 1505,
tracing (1.488 ms) : 1464, 1513
. : milestone, 1488,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (366.148 µs) : 347, 385
. : milestone, 366,
iast (488.285 µs) : 467, 510
. : milestone, 488,
iast_FULL (565.378 µs) : 544, 587
. : milestone, 565,
iast_GLOBAL (502.377 µs) : 481, 524
. : milestone, 502,
iast_HARDCODED_SECRET_DISABLED (488.44 µs) : 467, 510
. : milestone, 488,
iast_INACTIVE (449.233 µs) : 429, 470
. : milestone, 449,
iast_TELEMETRY_OFF (470.398 µs) : 450, 491
. : milestone, 470,
tracing (449.172 µs) : 427, 471
. : milestone, 449,
section candidate
no_agent (371.019 µs) : 352, 390
. : milestone, 371,
iast (486.686 µs) : 466, 508
. : milestone, 487,
iast_FULL (556.167 µs) : 535, 578
. : milestone, 556,
iast_GLOBAL (523.313 µs) : 500, 547
. : milestone, 523,
iast_HARDCODED_SECRET_DISABLED (487.689 µs) : 466, 509
. : milestone, 488,
iast_INACTIVE (457.074 µs) : 436, 478
. : milestone, 457,
iast_TELEMETRY_OFF (481.959 µs) : 460, 504
. : milestone, 482,
tracing (449.941 µs) : 429, 471
. : milestone, 450,
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 tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (1.473 ms) : 1461, 1484
. : milestone, 1473,
appsec (2.35 ms) : 2308, 2392
. : milestone, 2350,
iast (2.084 ms) : 2032, 2136
. : milestone, 2084,
iast_GLOBAL (2.133 ms) : 2080, 2187
. : milestone, 2133,
profiling (1.952 ms) : 1910, 1993
. : milestone, 1952,
tracing (1.93 ms) : 1890, 1970
. : milestone, 1930,
section candidate
no_agent (1.481 ms) : 1470, 1493
. : milestone, 1481,
appsec (2.347 ms) : 2305, 2388
. : milestone, 2347,
iast (2.101 ms) : 2049, 2153
. : milestone, 2101,
iast_GLOBAL (2.145 ms) : 2092, 2198
. : milestone, 2145,
profiling (1.956 ms) : 1914, 1999
. : milestone, 1956,
tracing (1.932 ms) : 1892, 1973
. : milestone, 1932,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.42.0-SNAPSHOT~58ab6ef562, baseline=1.42.0-SNAPSHOT~9320a05d02
dateFormat X
axisFormat %s
section baseline
no_agent (15.142 s) : 15142000, 15142000
. : milestone, 15142000,
appsec (15.344 s) : 15344000, 15344000
. : milestone, 15344000,
iast (18.92 s) : 18920000, 18920000
. : milestone, 18920000,
iast_GLOBAL (17.882 s) : 17882000, 17882000
. : milestone, 17882000,
profiling (15.119 s) : 15119000, 15119000
. : milestone, 15119000,
tracing (14.895 s) : 14895000, 14895000
. : milestone, 14895000,
section candidate
no_agent (15.454 s) : 15454000, 15454000
. : milestone, 15454000,
appsec (15.222 s) : 15222000, 15222000
. : milestone, 15222000,
iast (19.205 s) : 19205000, 19205000
. : milestone, 19205000,
iast_GLOBAL (18.129 s) : 18129000, 18129000
. : milestone, 18129000,
profiling (15.168 s) : 15168000, 15168000
. : milestone, 15168000,
tracing (15.185 s) : 15185000, 15185000
. : milestone, 15185000,
|
amarziali
approved these changes
Oct 18, 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
Widen catch blocks to make agent discovery more tolerant of underlying errors, like
UnsatisfiedLinkError
Motivation
Otherwise if an underlying transport throws an non I/O related error, such as JFFI throwing
UnsatisfiedLinkError
when it fails to load native components, then the exception will bubble up and installation of the tracer will be cancelled.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