-
Notifications
You must be signed in to change notification settings - Fork 279
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
Trace JMS Queue and Topic producers when destination is explicit #7266
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.
Good catch! The existing call-depth checks should avoid duplicate spans
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 46 metrics, 17 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1063541
Total [baseline] (10.382 s) : 0, 10381657
Agent [candidate] (1.071 s) : 0, 1071450
Total [candidate] (10.376 s) : 0, 10375782
section appsec
Agent [baseline] (1.19 s) : 0, 1190404
Total [baseline] (10.492 s) : 0, 10491677
Agent [candidate] (1.184 s) : 0, 1184012
Total [candidate] (10.543 s) : 0, 10542601
section iast
Agent [baseline] (1.178 s) : 0, 1177872
Total [baseline] (10.839 s) : 0, 10838697
Agent [candidate] (1.173 s) : 0, 1173090
Total [candidate] (10.784 s) : 0, 10783592
section profiling
Agent [baseline] (1.265 s) : 0, 1265076
Total [baseline] (10.545 s) : 0, 10545037
Agent [candidate] (1.268 s) : 0, 1268194
Total [candidate] (10.658 s) : 0, 10658017
gantt
title petclinic - break down per module: candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (665.692 ms) : 0, 665692
BytebuddyAgent [candidate] (672.211 ms) : 0, 672211
GlobalTracer [baseline] (304.679 ms) : 0, 304679
GlobalTracer [candidate] (305.731 ms) : 0, 305731
AppSec [baseline] (50.298 ms) : 0, 50298
AppSec [candidate] (50.275 ms) : 0, 50275
Remote Config [baseline] (720.867 µs) : 0, 721
Remote Config [candidate] (728.189 µs) : 0, 728
Telemetry [baseline] (7.61 ms) : 0, 7610
Telemetry [candidate] (7.573 ms) : 0, 7573
section appsec
BytebuddyAgent [baseline] (680.528 ms) : 0, 680528
BytebuddyAgent [candidate] (676.474 ms) : 0, 676474
GlobalTracer [baseline] (300.601 ms) : 0, 300601
GlobalTracer [candidate] (298.871 ms) : 0, 298871
AppSec [baseline] (154.571 ms) : 0, 154571
AppSec [candidate] (154.13 ms) : 0, 154130
Remote Config [baseline] (638.135 µs) : 0, 638
Remote Config [candidate] (636.942 µs) : 0, 637
Telemetry [baseline] (9.351 ms) : 0, 9351
Telemetry [candidate] (8.787 ms) : 0, 8787
IAST [baseline] (19.858 ms) : 0, 19858
IAST [candidate] (21.143 ms) : 0, 21143
section iast
BytebuddyAgent [baseline] (783.314 ms) : 0, 783314
BytebuddyAgent [candidate] (781.239 ms) : 0, 781239
GlobalTracer [baseline] (297.719 ms) : 0, 297719
GlobalTracer [candidate] (295.916 ms) : 0, 295916
AppSec [baseline] (47.842 ms) : 0, 47842
AppSec [candidate] (47.495 ms) : 0, 47495
Remote Config [baseline] (601.488 µs) : 0, 601
Remote Config [candidate] (646.356 µs) : 0, 646
Telemetry [baseline] (7.964 ms) : 0, 7964
Telemetry [candidate] (7.844 ms) : 0, 7844
IAST [baseline] (26.953 ms) : 0, 26953
IAST [candidate] (26.494 ms) : 0, 26494
section profiling
BytebuddyAgent [baseline] (663.314 ms) : 0, 663314
BytebuddyAgent [candidate] (664.808 ms) : 0, 664808
GlobalTracer [baseline] (388.506 ms) : 0, 388506
GlobalTracer [candidate] (389.887 ms) : 0, 389887
AppSec [baseline] (51.759 ms) : 0, 51759
AppSec [candidate] (51.878 ms) : 0, 51878
Remote Config [baseline] (645.27 µs) : 0, 645
Remote Config [candidate] (650.405 µs) : 0, 650
Telemetry [baseline] (7.452 ms) : 0, 7452
Telemetry [candidate] (7.395 ms) : 0, 7395
ProfilingAgent [baseline] (96.312 ms) : 0, 96312
ProfilingAgent [candidate] (96.431 ms) : 0, 96431
Profiling [baseline] (96.336 ms) : 0, 96336
Profiling [candidate] (96.455 ms) : 0, 96455
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.068 s) : 0, 1067526
Total [baseline] (8.568 s) : 0, 8567721
Agent [candidate] (1.064 s) : 0, 1064461
Total [candidate] (8.567 s) : 0, 8566615
section iast
Agent [baseline] (1.184 s) : 0, 1184117
Total [baseline] (9.072 s) : 0, 9071878
Agent [candidate] (1.172 s) : 0, 1172137
Total [candidate] (9.008 s) : 0, 9008431
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.172 s) : 0, 1171517
Total [baseline] (8.972 s) : 0, 8972300
Agent [candidate] (1.172 s) : 0, 1172017
Total [candidate] (9.062 s) : 0, 9062080
section iast_TELEMETRY_OFF
Agent [baseline] (1.178 s) : 0, 1177650
Total [baseline] (9.036 s) : 0, 9036381
Agent [candidate] (1.169 s) : 0, 1169196
Total [candidate] (9.035 s) : 0, 9035200
gantt
title insecure-bank - break down per module: candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (668.206 ms) : 0, 668206
BytebuddyAgent [candidate] (666.299 ms) : 0, 666299
GlobalTracer [baseline] (305.835 ms) : 0, 305835
GlobalTracer [candidate] (304.994 ms) : 0, 304994
AppSec [baseline] (50.542 ms) : 0, 50542
AppSec [candidate] (50.39 ms) : 0, 50390
Remote Config [baseline] (743.728 µs) : 0, 744
Remote Config [candidate] (721.396 µs) : 0, 721
Telemetry [baseline] (7.658 ms) : 0, 7658
Telemetry [candidate] (7.621 ms) : 0, 7621
section iast
BytebuddyAgent [baseline] (787.79 ms) : 0, 787790
BytebuddyAgent [candidate] (781.383 ms) : 0, 781383
GlobalTracer [baseline] (299.253 ms) : 0, 299253
GlobalTracer [candidate] (294.769 ms) : 0, 294769
AppSec [baseline] (48.034 ms) : 0, 48034
AppSec [candidate] (47.523 ms) : 0, 47523
IAST [baseline] (26.947 ms) : 0, 26947
IAST [candidate] (27.27 ms) : 0, 27270
Remote Config [baseline] (629.439 µs) : 0, 629
Remote Config [candidate] (605.305 µs) : 0, 605
Telemetry [baseline] (7.961 ms) : 0, 7961
Telemetry [candidate] (7.135 ms) : 0, 7135
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (779.486 ms) : 0, 779486
BytebuddyAgent [candidate] (779.959 ms) : 0, 779959
GlobalTracer [baseline] (295.068 ms) : 0, 295068
GlobalTracer [candidate] (295.633 ms) : 0, 295633
AppSec [baseline] (47.388 ms) : 0, 47388
AppSec [candidate] (47.41 ms) : 0, 47410
IAST [baseline] (26.201 ms) : 0, 26201
IAST [candidate] (28.036 ms) : 0, 28036
Remote Config [baseline] (604.719 µs) : 0, 605
Remote Config [candidate] (591.246 µs) : 0, 591
Telemetry [baseline] (9.316 ms) : 0, 9316
Telemetry [candidate] (6.948 ms) : 0, 6948
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (784.061 ms) : 0, 784061
BytebuddyAgent [candidate] (777.758 ms) : 0, 777758
GlobalTracer [baseline] (296.886 ms) : 0, 296886
GlobalTracer [candidate] (294.761 ms) : 0, 294761
AppSec [baseline] (47.372 ms) : 0, 47372
AppSec [candidate] (47.161 ms) : 0, 47161
IAST [baseline] (26.087 ms) : 0, 26087
IAST [candidate] (27.651 ms) : 0, 27651
Remote Config [baseline] (647.065 µs) : 0, 647
Remote Config [candidate] (652.884 µs) : 0, 653
Telemetry [baseline] (9.095 ms) : 0, 9095
Telemetry [candidate] (7.771 ms) : 0, 7771
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 17 unstable metrics.
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section baseline
no_agent (366.584 µs) : 347, 386
. : milestone, 367,
iast (486.428 µs) : 465, 508
. : milestone, 486,
iast_FULL (550.785 µs) : 529, 572
. : milestone, 551,
iast_GLOBAL (509.196 µs) : 487, 531
. : milestone, 509,
iast_HARDCODED_SECRET_DISABLED (482.064 µs) : 461, 503
. : milestone, 482,
iast_INACTIVE (452.001 µs) : 431, 473
. : milestone, 452,
iast_TELEMETRY_OFF (473.065 µs) : 451, 495
. : milestone, 473,
tracing (440.482 µs) : 420, 461
. : milestone, 440,
section candidate
no_agent (368.323 µs) : 349, 388
. : milestone, 368,
iast (485.886 µs) : 464, 508
. : milestone, 486,
iast_FULL (551.244 µs) : 530, 573
. : milestone, 551,
iast_GLOBAL (509.417 µs) : 487, 532
. : milestone, 509,
iast_HARDCODED_SECRET_DISABLED (480.382 µs) : 459, 501
. : milestone, 480,
iast_INACTIVE (456.054 µs) : 434, 478
. : milestone, 456,
iast_TELEMETRY_OFF (471.024 µs) : 450, 493
. : milestone, 471,
tracing (435.656 µs) : 415, 456
. : milestone, 436,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section baseline
no_agent (1.336 ms) : 1317, 1356
. : milestone, 1336,
appsec (1.718 ms) : 1693, 1743
. : milestone, 1718,
appsec_no_iast (1.716 ms) : 1691, 1740
. : milestone, 1716,
iast (1.467 ms) : 1444, 1489
. : milestone, 1467,
profiling (1.56 ms) : 1535, 1585
. : milestone, 1560,
tracing (1.469 ms) : 1445, 1493
. : milestone, 1469,
section candidate
no_agent (1.339 ms) : 1318, 1359
. : milestone, 1339,
appsec (1.71 ms) : 1686, 1734
. : milestone, 1710,
appsec_no_iast (1.708 ms) : 1683, 1732
. : milestone, 1708,
iast (1.473 ms) : 1450, 1496
. : milestone, 1473,
profiling (1.481 ms) : 1456, 1507
. : milestone, 1481,
tracing (1.465 ms) : 1440, 1489
. : milestone, 1465,
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.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section baseline
no_agent (1.457 ms) : 1446, 1469
. : milestone, 1457,
appsec (2.228 ms) : 2191, 2264
. : milestone, 2228,
iast (1.991 ms) : 1948, 2035
. : milestone, 1991,
iast_GLOBAL (2.022 ms) : 1979, 2065
. : milestone, 2022,
profiling (1.86 ms) : 1824, 1896
. : milestone, 1860,
tracing (1.851 ms) : 1817, 1885
. : milestone, 1851,
section candidate
no_agent (1.463 ms) : 1452, 1475
. : milestone, 1463,
appsec (2.224 ms) : 2188, 2260
. : milestone, 2224,
iast (1.98 ms) : 1938, 2023
. : milestone, 1980,
iast_GLOBAL (2.023 ms) : 1980, 2067
. : milestone, 2023,
profiling (1.864 ms) : 1829, 1899
. : milestone, 1864,
tracing (1.835 ms) : 1802, 1868
. : milestone, 1835,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.37.0-SNAPSHOT~7d53edba58, baseline=1.37.0-SNAPSHOT~a9dde4fd66
dateFormat X
axisFormat %s
section baseline
no_agent (15.446 s) : 15446000, 15446000
. : milestone, 15446000,
appsec (14.757 s) : 14757000, 14757000
. : milestone, 14757000,
iast (19.011 s) : 19011000, 19011000
. : milestone, 19011000,
iast_GLOBAL (17.772 s) : 17772000, 17772000
. : milestone, 17772000,
profiling (15.675 s) : 15675000, 15675000
. : milestone, 15675000,
tracing (15.042 s) : 15042000, 15042000
. : milestone, 15042000,
section candidate
no_agent (15.356 s) : 15356000, 15356000
. : milestone, 15356000,
appsec (14.959 s) : 14959000, 14959000
. : milestone, 14959000,
iast (18.863 s) : 18863000, 18863000
. : milestone, 18863000,
iast_GLOBAL (17.847 s) : 17847000, 17847000
. : milestone, 17847000,
profiling (15.472 s) : 15472000, 15472000
. : milestone, 15472000,
tracing (15.202 s) : 15202000, 15202000
. : milestone, 15202000,
|
What Does This Do
The java agent today is not instrumenting jms send for signatures
QueueProducer.send(Queue, Message, ..)
orTopicPublisher.send(Topic, Message)
.Instead, we trace
MessageProducer.send(Destination, Message, ..)
only.This is generally OK when specialized classes are delegating to superclasses. In fact for ActiveMQ we have
But it's not OK when the specialized class does not delegate to super. Hence we lost the tracing on the producer.
Motivation
Additional Notes
Jira ticket: [PROJ-IDENT]