Add pathway propagation for SNS#7341
Merged
nayeem-kamal merged 12 commits intomasterfrom Jul 19, 2024
Merged
Conversation
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 petclinicgantt
title petclinic - global startup overhead: candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.043 s) : 0, 1043290
Total [baseline] (10.338 s) : 0, 10338137
Agent [candidate] (1.075 s) : 0, 1074540
Total [candidate] (10.365 s) : 0, 10364971
section appsec
Agent [baseline] (1.164 s) : 0, 1163790
Total [baseline] (10.475 s) : 0, 10475255
Agent [candidate] (1.189 s) : 0, 1189240
Total [candidate] (10.562 s) : 0, 10561614
section iast
Agent [baseline] (1.172 s) : 0, 1171578
Total [baseline] (10.729 s) : 0, 10729444
Agent [candidate] (1.172 s) : 0, 1171568
Total [candidate] (10.743 s) : 0, 10742909
section profiling
Agent [baseline] (1.242 s) : 0, 1241926
Total [baseline] (10.602 s) : 0, 10602385
Agent [candidate] (1.275 s) : 0, 1275371
Total [candidate] (10.612 s) : 0, 10611507
gantt
title petclinic - break down per module: candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.272 ms) : 0, 666272
BytebuddyAgent [candidate] (673.997 ms) : 0, 673997
GlobalTracer [baseline] (305.291 ms) : 0, 305291
GlobalTracer [candidate] (306.804 ms) : 0, 306804
AppSec [baseline] (49.925 ms) : 0, 49925
AppSec [candidate] (50.356 ms) : 0, 50356
Remote Config [baseline] (674.391 µs) : 0, 674
Remote Config [candidate] (677.217 µs) : 0, 677
Telemetry [baseline] (7.652 ms) : 0, 7652
Telemetry [candidate] (7.684 ms) : 0, 7684
section appsec
BytebuddyAgent [baseline] (676.118 ms) : 0, 676118
BytebuddyAgent [candidate] (679.741 ms) : 0, 679741
GlobalTracer [baseline] (299.483 ms) : 0, 299483
GlobalTracer [candidate] (300.813 ms) : 0, 300813
AppSec [baseline] (153.938 ms) : 0, 153938
AppSec [candidate] (154.282 ms) : 0, 154282
Remote Config [baseline] (618.643 µs) : 0, 619
Remote Config [candidate] (621.874 µs) : 0, 622
Telemetry [baseline] (9.559 ms) : 0, 9559
Telemetry [candidate] (8.032 ms) : 0, 8032
IAST [baseline] (21.162 ms) : 0, 21162
IAST [candidate] (21.796 ms) : 0, 21796
section iast
BytebuddyAgent [baseline] (780.07 ms) : 0, 780070
BytebuddyAgent [candidate] (780.088 ms) : 0, 780088
GlobalTracer [baseline] (295.76 ms) : 0, 295760
GlobalTracer [candidate] (295.607 ms) : 0, 295607
AppSec [baseline] (48.35 ms) : 0, 48350
AppSec [candidate] (49.048 ms) : 0, 49048
Remote Config [baseline] (584.176 µs) : 0, 584
Remote Config [candidate] (588.276 µs) : 0, 588
Telemetry [baseline] (7.777 ms) : 0, 7777
Telemetry [candidate] (7.047 ms) : 0, 7047
IAST [baseline] (25.571 ms) : 0, 25571
IAST [candidate] (25.615 ms) : 0, 25615
section profiling
ProfilingAgent [baseline] (94.422 ms) : 0, 94422
ProfilingAgent [candidate] (97.259 ms) : 0, 97259
BytebuddyAgent [baseline] (663.356 ms) : 0, 663356
BytebuddyAgent [candidate] (668.62 ms) : 0, 668620
GlobalTracer [baseline] (388.607 ms) : 0, 388607
GlobalTracer [candidate] (391.096 ms) : 0, 391096
AppSec [baseline] (51.425 ms) : 0, 51425
AppSec [candidate] (52.486 ms) : 0, 52486
Remote Config [baseline] (645.568 µs) : 0, 646
Remote Config [candidate] (661.443 µs) : 0, 661
Telemetry [baseline] (7.37 ms) : 0, 7370
Telemetry [candidate] (7.496 ms) : 0, 7496
Profiling [baseline] (94.447 ms) : 0, 94447
Profiling [candidate] (97.284 ms) : 0, 97284
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.044 s) : 0, 1043891
Total [baseline] (8.524 s) : 0, 8524177
Agent [candidate] (1.063 s) : 0, 1062731
Total [candidate] (8.503 s) : 0, 8502768
section iast
Agent [baseline] (1.17 s) : 0, 1170115
Total [baseline] (8.98 s) : 0, 8979570
Agent [candidate] (1.169 s) : 0, 1168857
Total [candidate] (8.986 s) : 0, 8985916
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.174 s) : 0, 1173945
Total [baseline] (8.96 s) : 0, 8960405
Agent [candidate] (1.171 s) : 0, 1170733
Total [candidate] (8.952 s) : 0, 8951624
section iast_TELEMETRY_OFF
Agent [baseline] (1.165 s) : 0, 1164796
Total [baseline] (8.946 s) : 0, 8946091
Agent [candidate] (1.168 s) : 0, 1168008
Total [candidate] (9.032 s) : 0, 9031591
gantt
title insecure-bank - break down per module: candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.539 ms) : 0, 666539
BytebuddyAgent [candidate] (665.159 ms) : 0, 665159
GlobalTracer [baseline] (305.474 ms) : 0, 305474
GlobalTracer [candidate] (304.694 ms) : 0, 304694
AppSec [baseline] (50.205 ms) : 0, 50205
AppSec [candidate] (49.995 ms) : 0, 49995
Remote Config [baseline] (670.552 µs) : 0, 671
Remote Config [candidate] (672.29 µs) : 0, 672
Telemetry [baseline] (7.578 ms) : 0, 7578
Telemetry [candidate] (7.673 ms) : 0, 7673
section iast
BytebuddyAgent [baseline] (780.594 ms) : 0, 780594
BytebuddyAgent [candidate] (779.693 ms) : 0, 779693
GlobalTracer [baseline] (295.247 ms) : 0, 295247
GlobalTracer [candidate] (295.694 ms) : 0, 295694
AppSec [baseline] (47.566 ms) : 0, 47566
AppSec [candidate] (47.308 ms) : 0, 47308
IAST [baseline] (25.68 ms) : 0, 25680
IAST [candidate] (25.075 ms) : 0, 25075
Remote Config [baseline] (595.048 µs) : 0, 595
Remote Config [candidate] (583.173 µs) : 0, 583
Telemetry [baseline] (7.015 ms) : 0, 7015
Telemetry [candidate] (6.999 ms) : 0, 6999
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (781.37 ms) : 0, 781370
BytebuddyAgent [candidate] (779.512 ms) : 0, 779512
GlobalTracer [baseline] (295.914 ms) : 0, 295914
GlobalTracer [candidate] (294.837 ms) : 0, 294837
AppSec [baseline] (48.363 ms) : 0, 48363
AppSec [candidate] (49.004 ms) : 0, 49004
IAST [baseline] (26.44 ms) : 0, 26440
IAST [candidate] (25.285 ms) : 0, 25285
Remote Config [baseline] (581.863 µs) : 0, 582
Remote Config [candidate] (592.695 µs) : 0, 593
Telemetry [baseline] (7.731 ms) : 0, 7731
Telemetry [candidate] (7.928 ms) : 0, 7928
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (777.593 ms) : 0, 777593
BytebuddyAgent [candidate] (776.526 ms) : 0, 776526
GlobalTracer [baseline] (294.925 ms) : 0, 294925
GlobalTracer [candidate] (294.777 ms) : 0, 294777
AppSec [baseline] (47.223 ms) : 0, 47223
AppSec [candidate] (47.243 ms) : 0, 47243
IAST [baseline] (23.936 ms) : 0, 23936
IAST [candidate] (28.466 ms) : 0, 28466
Remote Config [baseline] (597.907 µs) : 0, 598
Remote Config [candidate] (574.387 µs) : 0, 574
Telemetry [baseline] (7.038 ms) : 0, 7038
Telemetry [candidate] (6.825 ms) : 0, 6825
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 petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section baseline
no_agent (1.322 ms) : 1303, 1341
. : milestone, 1322,
appsec (1.72 ms) : 1698, 1743
. : milestone, 1720,
appsec_no_iast (1.706 ms) : 1681, 1730
. : milestone, 1706,
iast (1.482 ms) : 1459, 1504
. : milestone, 1482,
profiling (1.539 ms) : 1513, 1564
. : milestone, 1539,
tracing (1.447 ms) : 1423, 1471
. : milestone, 1447,
section candidate
no_agent (1.329 ms) : 1309, 1349
. : milestone, 1329,
appsec (1.692 ms) : 1668, 1715
. : milestone, 1692,
appsec_no_iast (1.702 ms) : 1678, 1725
. : milestone, 1702,
iast (1.469 ms) : 1447, 1492
. : milestone, 1469,
profiling (1.486 ms) : 1461, 1510
. : milestone, 1486,
tracing (1.448 ms) : 1424, 1473
. : milestone, 1448,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section baseline
no_agent (361.535 µs) : 341, 382
. : milestone, 362,
iast (473.023 µs) : 452, 494
. : milestone, 473,
iast_FULL (542.158 µs) : 521, 563
. : milestone, 542,
iast_GLOBAL (498.645 µs) : 477, 520
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (479.177 µs) : 458, 500
. : milestone, 479,
iast_INACTIVE (450.291 µs) : 429, 472
. : milestone, 450,
iast_TELEMETRY_OFF (467.125 µs) : 446, 489
. : milestone, 467,
tracing (428.692 µs) : 409, 449
. : milestone, 429,
section candidate
no_agent (356.103 µs) : 337, 375
. : milestone, 356,
iast (478.004 µs) : 457, 499
. : milestone, 478,
iast_FULL (547.067 µs) : 526, 568
. : milestone, 547,
iast_GLOBAL (498.34 µs) : 477, 520
. : milestone, 498,
iast_HARDCODED_SECRET_DISABLED (476.324 µs) : 455, 498
. : milestone, 476,
iast_INACTIVE (451.147 µs) : 430, 473
. : milestone, 451,
iast_TELEMETRY_OFF (466.788 µs) : 446, 488
. : milestone, 467,
tracing (435.184 µs) : 415, 455
. : milestone, 435,
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.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section baseline
no_agent (15.476 s) : 15476000, 15476000
. : milestone, 15476000,
appsec (15.137 s) : 15137000, 15137000
. : milestone, 15137000,
iast (18.967 s) : 18967000, 18967000
. : milestone, 18967000,
iast_GLOBAL (17.911 s) : 17911000, 17911000
. : milestone, 17911000,
profiling (15.331 s) : 15331000, 15331000
. : milestone, 15331000,
tracing (15.252 s) : 15252000, 15252000
. : milestone, 15252000,
section candidate
no_agent (15.325 s) : 15325000, 15325000
. : milestone, 15325000,
appsec (14.838 s) : 14838000, 14838000
. : milestone, 14838000,
iast (18.711 s) : 18711000, 18711000
. : milestone, 18711000,
iast_GLOBAL (17.886 s) : 17886000, 17886000
. : milestone, 17886000,
profiling (15.227 s) : 15227000, 15227000
. : milestone, 15227000,
tracing (15.219 s) : 15219000, 15219000
. : milestone, 15219000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.38.0-SNAPSHOT~36478e6ff7, baseline=1.38.0-SNAPSHOT~bfe00c1b41
dateFormat X
axisFormat %s
section baseline
no_agent (1.462 ms) : 1450, 1473
. : milestone, 1462,
appsec (2.206 ms) : 2171, 2241
. : milestone, 2206,
iast (1.96 ms) : 1919, 2001
. : milestone, 1960,
iast_GLOBAL (2.01 ms) : 1967, 2053
. : milestone, 2010,
profiling (1.854 ms) : 1819, 1889
. : milestone, 1854,
tracing (1.832 ms) : 1798, 1865
. : milestone, 1832,
section candidate
no_agent (1.457 ms) : 1446, 1468
. : milestone, 1457,
appsec (2.22 ms) : 2185, 2255
. : milestone, 2220,
iast (1.951 ms) : 1910, 1993
. : milestone, 1951,
iast_GLOBAL (2.003 ms) : 1960, 2045
. : milestone, 2003,
profiling (2.308 ms) : 2064, 2552
. : milestone, 2308,
tracing (1.832 ms) : 1799, 1865
. : milestone, 1832,
|
added 2 commits
July 17, 2024 14:17
vandonr
reviewed
Jul 18, 2024
Contributor
vandonr
left a comment
There was a problem hiding this comment.
looks good, modulo the added complexity on getting the topic in v2
.../src/main/java/datadog/trace/bootstrap/instrumentation/messaging/DatadogAttributeParser.java
Show resolved
Hide resolved
.../aws-java-sns-2.0/src/main/java/datadog/trace/instrumentation/aws/v2/sns/SnsInterceptor.java
Outdated
Show resolved
Hide resolved
.../aws-java-sns-2.0/src/main/java/datadog/trace/instrumentation/aws/v2/sns/SnsInterceptor.java
Outdated
Show resolved
Hide resolved
added 4 commits
July 18, 2024 09:46
Contributor
vandonr
reviewed
Jul 18, 2024
.../aws-java-sns-2.0/src/main/java/datadog/trace/instrumentation/aws/v2/sns/SnsInterceptor.java
Outdated
Show resolved
Hide resolved
vandonr
approved these changes
Jul 18, 2024
PerfectSlayer
approved these changes
Jul 19, 2024
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.
What Does This Do
Adds support for injecting DSM pathway context into message attributes for SNS
Motivation
Additional Notes
Jira ticket: DSMS-21