Remove AgentScope.source() from instrumentation API#8539
Merged
Conversation
amarziali
approved these changes
Mar 12, 2025
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 60 metrics, 3 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.045 s) : 0, 1044627
Total [baseline] (10.456 s) : 0, 10455856
Agent [candidate] (1.045 s) : 0, 1044662
Total [candidate] (10.454 s) : 0, 10454229
section appsec
Agent [baseline] (1.182 s) : 0, 1181504
Total [baseline] (10.725 s) : 0, 10724538
Agent [candidate] (1.194 s) : 0, 1193791
Total [candidate] (10.808 s) : 0, 10808473
section iast
Agent [baseline] (1.178 s) : 0, 1178404
Total [baseline] (10.956 s) : 0, 10956247
Agent [candidate] (1.182 s) : 0, 1182246
Total [candidate] (11.001 s) : 0, 11000922
section profiling
Agent [baseline] (1.274 s) : 0, 1273605
Total [baseline] (10.887 s) : 0, 10887375
Agent [candidate] (1.263 s) : 0, 1262557
Total [candidate] (10.812 s) : 0, 10812045
gantt
title petclinic - break down per module: candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (721.81 ms) : 0, 721810
BytebuddyAgent [candidate] (718.546 ms) : 0, 718546
GlobalTracer [baseline] (240.506 ms) : 0, 240506
GlobalTracer [candidate] (240.551 ms) : 0, 240551
AppSec [baseline] (55.302 ms) : 0, 55302
AppSec [candidate] (55.636 ms) : 0, 55636
Remote Config [baseline] (709.567 µs) : 0, 710
Remote Config [candidate] (694.931 µs) : 0, 695
Telemetry [baseline] (11.386 ms) : 0, 11386
Telemetry [candidate] (14.392 ms) : 0, 14392
section appsec
BytebuddyAgent [baseline] (734.678 ms) : 0, 734678
BytebuddyAgent [candidate] (743.671 ms) : 0, 743671
GlobalTracer [baseline] (236.13 ms) : 0, 236130
GlobalTracer [candidate] (238.344 ms) : 0, 238344
AppSec [baseline] (176.296 ms) : 0, 176296
AppSec [candidate] (176.944 ms) : 0, 176944
Remote Config [baseline] (659.256 µs) : 0, 659
Remote Config [candidate] (670.557 µs) : 0, 671
Telemetry [baseline] (8.262 ms) : 0, 8262
Telemetry [candidate] (8.34 ms) : 0, 8340
IAST [baseline] (21.299 ms) : 0, 21299
IAST [candidate] (21.725 ms) : 0, 21725
section iast
BytebuddyAgent [baseline] (843.58 ms) : 0, 843580
BytebuddyAgent [candidate] (845.731 ms) : 0, 845731
GlobalTracer [baseline] (230.907 ms) : 0, 230907
GlobalTracer [candidate] (231.897 ms) : 0, 231897
AppSec [baseline] (56.685 ms) : 0, 56685
AppSec [candidate] (57.01 ms) : 0, 57010
Remote Config [baseline] (613.581 µs) : 0, 614
Remote Config [candidate] (613.289 µs) : 0, 613
Telemetry [baseline] (8.687 ms) : 0, 8687
Telemetry [candidate] (8.837 ms) : 0, 8837
IAST [baseline] (22.952 ms) : 0, 22952
IAST [candidate] (23.165 ms) : 0, 23165
section profiling
ProfilingAgent [baseline] (97.841 ms) : 0, 97841
ProfilingAgent [candidate] (96.76 ms) : 0, 96760
BytebuddyAgent [baseline] (716.478 ms) : 0, 716478
BytebuddyAgent [candidate] (710.498 ms) : 0, 710498
GlobalTracer [baseline] (353.426 ms) : 0, 353426
GlobalTracer [candidate] (351.109 ms) : 0, 351109
AppSec [baseline] (55.337 ms) : 0, 55337
AppSec [candidate] (54.21 ms) : 0, 54210
Remote Config [baseline] (692.952 µs) : 0, 693
Remote Config [candidate] (701.605 µs) : 0, 702
Telemetry [baseline] (9.09 ms) : 0, 9090
Telemetry [candidate] (8.971 ms) : 0, 8971
Profiling [baseline] (97.865 ms) : 0, 97865
Profiling [candidate] (96.784 ms) : 0, 96784
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.042 s) : 0, 1041570
Total [baseline] (8.665 s) : 0, 8665081
Agent [candidate] (1.046 s) : 0, 1046120
Total [candidate] (8.69 s) : 0, 8689890
section iast
Agent [baseline] (1.177 s) : 0, 1177163
Total [baseline] (9.274 s) : 0, 9274382
Agent [candidate] (1.174 s) : 0, 1174299
Total [candidate] (9.276 s) : 0, 9275702
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.181 s) : 0, 1180793
Total [baseline] (9.233 s) : 0, 9233011
Agent [candidate] (1.169 s) : 0, 1169462
Total [candidate] (9.157 s) : 0, 9157190
section iast_TELEMETRY_OFF
Agent [baseline] (1.166 s) : 0, 1165954
Total [baseline] (9.259 s) : 0, 9258525
Agent [candidate] (1.168 s) : 0, 1167500
Total [candidate] (9.21 s) : 0, 9209753
gantt
title insecure-bank - break down per module: candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (719.285 ms) : 0, 719285
BytebuddyAgent [candidate] (723.511 ms) : 0, 723511
GlobalTracer [baseline] (239.718 ms) : 0, 239718
GlobalTracer [candidate] (241.296 ms) : 0, 241296
AppSec [baseline] (54.902 ms) : 0, 54902
AppSec [candidate] (55.498 ms) : 0, 55498
Remote Config [baseline] (686.43 µs) : 0, 686
Remote Config [candidate] (701.634 µs) : 0, 702
Telemetry [baseline] (12.137 ms) : 0, 12137
Telemetry [candidate] (10.105 ms) : 0, 10105
section iast
BytebuddyAgent [baseline] (841.892 ms) : 0, 841892
BytebuddyAgent [candidate] (839.618 ms) : 0, 839618
GlobalTracer [baseline] (231.013 ms) : 0, 231013
GlobalTracer [candidate] (230.782 ms) : 0, 230782
AppSec [baseline] (56.842 ms) : 0, 56842
AppSec [candidate] (56.529 ms) : 0, 56529
Remote Config [baseline] (611.188 µs) : 0, 611
Remote Config [candidate] (611.173 µs) : 0, 611
Telemetry [baseline] (8.793 ms) : 0, 8793
Telemetry [candidate] (8.824 ms) : 0, 8824
IAST [baseline] (23.083 ms) : 0, 23083
IAST [candidate] (23.019 ms) : 0, 23019
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (844.398 ms) : 0, 844398
BytebuddyAgent [candidate] (835.972 ms) : 0, 835972
GlobalTracer [baseline] (231.75 ms) : 0, 231750
GlobalTracer [candidate] (230.208 ms) : 0, 230208
AppSec [baseline] (56.935 ms) : 0, 56935
AppSec [candidate] (56.43 ms) : 0, 56430
Remote Config [baseline] (616.034 µs) : 0, 616
Remote Config [candidate] (603.308 µs) : 0, 603
Telemetry [baseline] (8.737 ms) : 0, 8737
Telemetry [candidate] (8.62 ms) : 0, 8620
IAST [baseline] (23.333 ms) : 0, 23333
IAST [candidate] (22.759 ms) : 0, 22759
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (833.252 ms) : 0, 833252
BytebuddyAgent [candidate] (834.635 ms) : 0, 834635
GlobalTracer [baseline] (229.872 ms) : 0, 229872
GlobalTracer [candidate] (230.076 ms) : 0, 230076
AppSec [baseline] (56.465 ms) : 0, 56465
AppSec [candidate] (56.64 ms) : 0, 56640
Remote Config [baseline] (612.641 µs) : 0, 613
Remote Config [candidate] (608.388 µs) : 0, 608
Telemetry [baseline] (8.602 ms) : 0, 8602
Telemetry [candidate] (8.546 ms) : 0, 8546
IAST [baseline] (22.359 ms) : 0, 22359
IAST [candidate] (22.153 ms) : 0, 22153
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section baseline
no_agent (382.027 µs) : 361, 403
. : milestone, 382,
iast (516.576 µs) : 494, 539
. : milestone, 517,
iast_FULL (726.716 µs) : 705, 749
. : milestone, 727,
iast_GLOBAL (562.832 µs) : 541, 585
. : milestone, 563,
iast_HARDCODED_SECRET_DISABLED (509.371 µs) : 488, 531
. : milestone, 509,
iast_INACTIVE (463.98 µs) : 442, 486
. : milestone, 464,
iast_TELEMETRY_OFF (501.695 µs) : 480, 523
. : milestone, 502,
tracing (461.03 µs) : 440, 482
. : milestone, 461,
section candidate
no_agent (375.274 µs) : 355, 395
. : milestone, 375,
iast (509.179 µs) : 488, 531
. : milestone, 509,
iast_FULL (728.18 µs) : 706, 750
. : milestone, 728,
iast_GLOBAL (559.029 µs) : 537, 581
. : milestone, 559,
iast_HARDCODED_SECRET_DISABLED (510.672 µs) : 489, 533
. : milestone, 511,
iast_INACTIVE (463.735 µs) : 442, 485
. : milestone, 464,
iast_TELEMETRY_OFF (500.582 µs) : 479, 522
. : milestone, 501,
tracing (461.765 µs) : 441, 483
. : milestone, 462,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section baseline
no_agent (1.362 ms) : 1343, 1382
. : milestone, 1362,
appsec (1.718 ms) : 1694, 1743
. : milestone, 1718,
appsec_no_iast (1.735 ms) : 1711, 1759
. : milestone, 1735,
code_origins (1.689 ms) : 1662, 1716
. : milestone, 1689,
iast (1.533 ms) : 1509, 1557
. : milestone, 1533,
profiling (1.511 ms) : 1487, 1536
. : milestone, 1511,
tracing (1.492 ms) : 1468, 1516
. : milestone, 1492,
section candidate
no_agent (1.342 ms) : 1323, 1362
. : milestone, 1342,
appsec (1.73 ms) : 1706, 1754
. : milestone, 1730,
appsec_no_iast (1.749 ms) : 1725, 1772
. : milestone, 1749,
code_origins (1.687 ms) : 1661, 1713
. : milestone, 1687,
iast (1.528 ms) : 1504, 1552
. : milestone, 1528,
profiling (1.553 ms) : 1529, 1578
. : milestone, 1553,
tracing (1.483 ms) : 1458, 1508
. : milestone, 1483,
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.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section baseline
no_agent (15.041 s) : 15041000, 15041000
. : milestone, 15041000,
appsec (15.214 s) : 15214000, 15214000
. : milestone, 15214000,
iast (18.403 s) : 18403000, 18403000
. : milestone, 18403000,
iast_GLOBAL (17.573 s) : 17573000, 17573000
. : milestone, 17573000,
profiling (15.282 s) : 15282000, 15282000
. : milestone, 15282000,
tracing (14.636 s) : 14636000, 14636000
. : milestone, 14636000,
section candidate
no_agent (15.504 s) : 15504000, 15504000
. : milestone, 15504000,
appsec (15.094 s) : 15094000, 15094000
. : milestone, 15094000,
iast (18.653 s) : 18653000, 18653000
. : milestone, 18653000,
iast_GLOBAL (17.77 s) : 17770000, 17770000
. : milestone, 17770000,
profiling (14.956 s) : 14956000, 14956000
. : milestone, 14956000,
tracing (14.644 s) : 14644000, 14644000
. : milestone, 14644000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.48.0-SNAPSHOT~b2ee479713, baseline=1.48.0-SNAPSHOT~11015eec16
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.365 ms) : 2321, 2409
. : milestone, 2365,
iast (2.137 ms) : 2081, 2193
. : milestone, 2137,
iast_GLOBAL (2.194 ms) : 2138, 2251
. : milestone, 2194,
profiling (1.98 ms) : 1936, 2024
. : milestone, 1980,
tracing (1.962 ms) : 1919, 2004
. : milestone, 1962,
section candidate
no_agent (1.485 ms) : 1473, 1496
. : milestone, 1485,
appsec (2.364 ms) : 2320, 2408
. : milestone, 2364,
iast (2.132 ms) : 2077, 2188
. : milestone, 2132,
iast_GLOBAL (2.18 ms) : 2124, 2236
. : milestone, 2180,
profiling (2.008 ms) : 1963, 2054
. : milestone, 2008,
tracing (1.978 ms) : 1934, 2021
. : milestone, 1978,
|
PerfectSlayer
approved these changes
Mar 12, 2025
Contributor
PerfectSlayer
left a comment
There was a problem hiding this comment.
Nice moving logic away from tracer itself 👍
This file contains hidden or 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
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.
Motivation
No reason to leak this into the instrumentation API
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: APMAPI-960