-
Notifications
You must be signed in to change notification settings - Fork 318
add request-id to lambda start/end invocation calls #9991
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
base: master
Are you sure you want to change the base?
Conversation
This comment has been minimized.
This comment has been minimized.
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 59 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.098 s) : 0, 1097502
Total [baseline] (10.783 s) : 0, 10782706
Agent [candidate] (1.1 s) : 0, 1100024
Total [candidate] (10.773 s) : 0, 10773146
section appsec
Agent [baseline] (1.281 s) : 0, 1281082
Total [baseline] (11.056 s) : 0, 11056059
Agent [candidate] (1.276 s) : 0, 1276202
Total [candidate] (11.016 s) : 0, 11016484
section iast
Agent [baseline] (1.234 s) : 0, 1233736
Total [baseline] (11.198 s) : 0, 11198237
Agent [candidate] (1.234 s) : 0, 1233904
Total [candidate] (11.097 s) : 0, 11097208
section profiling
Agent [baseline] (1.23 s) : 0, 1230335
Total [baseline] (11.057 s) : 0, 11057423
Agent [candidate] (1.228 s) : 0, 1228182
Total [candidate] (11.025 s) : 0, 11025416
gantt
title petclinic - break down per module: candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.448 ms) : 0, 1448
crashtracking [candidate] (1.449 ms) : 0, 1449
BytebuddyAgent [baseline] (704.152 ms) : 0, 704152
BytebuddyAgent [candidate] (705.937 ms) : 0, 705937
GlobalTracer [baseline] (248.424 ms) : 0, 248424
GlobalTracer [candidate] (248.869 ms) : 0, 248869
AppSec [baseline] (32.231 ms) : 0, 32231
AppSec [candidate] (32.274 ms) : 0, 32274
Debugger [baseline] (64.079 ms) : 0, 64079
Debugger [candidate] (64.174 ms) : 0, 64174
Remote Config [baseline] (632.02 µs) : 0, 632
Remote Config [candidate] (645.815 µs) : 0, 646
Telemetry [baseline] (8.084 ms) : 0, 8084
Telemetry [candidate] (8.202 ms) : 0, 8202
Flare Poller [baseline] (3.671 ms) : 0, 3671
Flare Poller [candidate] (3.686 ms) : 0, 3686
section appsec
crashtracking [baseline] (1.462 ms) : 0, 1462
crashtracking [candidate] (1.45 ms) : 0, 1450
BytebuddyAgent [baseline] (732.311 ms) : 0, 732311
BytebuddyAgent [candidate] (727.609 ms) : 0, 727609
GlobalTracer [baseline] (240.323 ms) : 0, 240323
GlobalTracer [candidate] (239.608 ms) : 0, 239608
IAST [baseline] (24.805 ms) : 0, 24805
IAST [candidate] (24.636 ms) : 0, 24636
AppSec [baseline] (173.692 ms) : 0, 173692
AppSec [candidate] (174.07 ms) : 0, 174070
Debugger [baseline] (60.616 ms) : 0, 60616
Debugger [candidate] (60.952 ms) : 0, 60952
Remote Config [baseline] (698.579 µs) : 0, 699
Remote Config [candidate] (702.019 µs) : 0, 702
Telemetry [baseline] (8.207 ms) : 0, 8207
Telemetry [candidate] (8.319 ms) : 0, 8319
Flare Poller [baseline] (3.814 ms) : 0, 3814
Flare Poller [candidate] (3.928 ms) : 0, 3928
section iast
crashtracking [baseline] (1.444 ms) : 0, 1444
crashtracking [candidate] (1.441 ms) : 0, 1441
BytebuddyAgent [baseline] (825.811 ms) : 0, 825811
BytebuddyAgent [candidate] (826.786 ms) : 0, 826786
GlobalTracer [baseline] (237.224 ms) : 0, 237224
GlobalTracer [candidate] (237.519 ms) : 0, 237519
IAST [baseline] (27.293 ms) : 0, 27293
IAST [candidate] (31.595 ms) : 0, 31595
AppSec [baseline] (34.035 ms) : 0, 34035
AppSec [candidate] (29.471 ms) : 0, 29471
Debugger [baseline] (61.262 ms) : 0, 61262
Debugger [candidate] (60.588 ms) : 0, 60588
Remote Config [baseline] (542.13 µs) : 0, 542
Remote Config [candidate] (556.764 µs) : 0, 557
Telemetry [baseline] (7.673 ms) : 0, 7673
Telemetry [candidate] (7.585 ms) : 0, 7585
Flare Poller [baseline] (3.525 ms) : 0, 3525
Flare Poller [candidate] (3.489 ms) : 0, 3489
section profiling
crashtracking [baseline] (1.438 ms) : 0, 1438
crashtracking [candidate] (1.437 ms) : 0, 1437
BytebuddyAgent [baseline] (731.86 ms) : 0, 731860
BytebuddyAgent [candidate] (730.636 ms) : 0, 730636
GlobalTracer [baseline] (222.294 ms) : 0, 222294
GlobalTracer [candidate] (222.117 ms) : 0, 222117
AppSec [baseline] (32.273 ms) : 0, 32273
AppSec [candidate] (32.159 ms) : 0, 32159
Debugger [baseline] (63.048 ms) : 0, 63048
Debugger [candidate] (62.831 ms) : 0, 62831
Remote Config [baseline] (644.716 µs) : 0, 645
Remote Config [candidate] (647.174 µs) : 0, 647
Telemetry [baseline] (7.892 ms) : 0, 7892
Telemetry [candidate] (7.903 ms) : 0, 7903
Flare Poller [baseline] (3.763 ms) : 0, 3763
Flare Poller [candidate] (3.763 ms) : 0, 3763
ProfilingAgent [baseline] (97.207 ms) : 0, 97207
ProfilingAgent [candidate] (96.992 ms) : 0, 96992
Profiling [baseline] (97.823 ms) : 0, 97823
Profiling [candidate] (97.573 ms) : 0, 97573
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.104 s) : 0, 1104176
Total [baseline] (8.871 s) : 0, 8870879
Agent [candidate] (1.106 s) : 0, 1105531
Total [candidate] (8.849 s) : 0, 8848631
section iast
Agent [baseline] (1.232 s) : 0, 1232330
Total [baseline] (9.517 s) : 0, 9516872
Agent [candidate] (1.233 s) : 0, 1233065
Total [candidate] (9.508 s) : 0, 9507732
gantt
title insecure-bank - break down per module: candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.462 ms) : 0, 1462
crashtracking [candidate] (1.47 ms) : 0, 1470
BytebuddyAgent [baseline] (708.928 ms) : 0, 708928
BytebuddyAgent [candidate] (710.185 ms) : 0, 710185
GlobalTracer [baseline] (249.806 ms) : 0, 249806
GlobalTracer [candidate] (249.879 ms) : 0, 249879
AppSec [baseline] (32.629 ms) : 0, 32629
AppSec [candidate] (32.65 ms) : 0, 32650
Debugger [baseline] (63.857 ms) : 0, 63857
Debugger [candidate] (63.902 ms) : 0, 63902
Remote Config [baseline] (647.44 µs) : 0, 647
Remote Config [candidate] (641.532 µs) : 0, 642
Telemetry [baseline] (8.166 ms) : 0, 8166
Telemetry [candidate] (8.155 ms) : 0, 8155
Flare Poller [baseline] (3.725 ms) : 0, 3725
Flare Poller [candidate] (3.709 ms) : 0, 3709
section iast
crashtracking [baseline] (1.452 ms) : 0, 1452
crashtracking [candidate] (1.441 ms) : 0, 1441
BytebuddyAgent [baseline] (825.903 ms) : 0, 825903
BytebuddyAgent [candidate] (826.149 ms) : 0, 826149
GlobalTracer [baseline] (237.291 ms) : 0, 237291
GlobalTracer [candidate] (237.648 ms) : 0, 237648
IAST [baseline] (28.278 ms) : 0, 28278
IAST [candidate] (28.867 ms) : 0, 28867
AppSec [baseline] (33.18 ms) : 0, 33180
AppSec [candidate] (32.436 ms) : 0, 32436
Debugger [baseline] (59.78 ms) : 0, 59780
Debugger [candidate] (59.986 ms) : 0, 59986
Remote Config [baseline] (542.273 µs) : 0, 542
Remote Config [candidate] (546.638 µs) : 0, 547
Telemetry [baseline] (7.593 ms) : 0, 7593
Telemetry [candidate] (7.724 ms) : 0, 7724
Flare Poller [baseline] (3.422 ms) : 0, 3422
Flare Poller [candidate] (3.492 ms) : 0, 3492
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 2 performance regressions! Performance is the same for 18 metrics, 15 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section baseline
no_agent (19.375 ms) : 19175, 19575
. : milestone, 19375,
appsec (18.355 ms) : 18169, 18541
. : milestone, 18355,
code_origins (17.986 ms) : 17806, 18167
. : milestone, 17986,
iast (17.649 ms) : 17473, 17825
. : milestone, 17649,
profiling (18.69 ms) : 18504, 18876
. : milestone, 18690,
tracing (18.548 ms) : 18359, 18737
. : milestone, 18548,
section candidate
no_agent (18.118 ms) : 17932, 18305
. : milestone, 18118,
appsec (18.545 ms) : 18357, 18732
. : milestone, 18545,
code_origins (17.992 ms) : 17813, 18170
. : milestone, 17992,
iast (17.764 ms) : 17586, 17942
. : milestone, 17764,
profiling (19.131 ms) : 18936, 19326
. : milestone, 19131,
tracing (18.002 ms) : 17824, 18181
. : milestone, 18002,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section baseline
no_agent (1.209 ms) : 1197, 1221
. : milestone, 1209,
iast (3.285 ms) : 3243, 3326
. : milestone, 3285,
iast_FULL (5.889 ms) : 5830, 5949
. : milestone, 5889,
iast_GLOBAL (3.539 ms) : 3490, 3588
. : milestone, 3539,
profiling (2.165 ms) : 2145, 2185
. : milestone, 2165,
tracing (1.809 ms) : 1794, 1824
. : milestone, 1809,
section candidate
no_agent (1.211 ms) : 1200, 1222
. : milestone, 1211,
iast (3.217 ms) : 3172, 3262
. : milestone, 3217,
iast_FULL (6.029 ms) : 5928, 6131
. : milestone, 6029,
iast_GLOBAL (3.748 ms) : 3685, 3812
. : milestone, 3748,
profiling (2.069 ms) : 2051, 2088
. : milestone, 2069,
tracing (1.785 ms) : 1771, 1799
. : milestone, 1785,
DacapoParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 0 unstable metrics.
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section baseline
no_agent (1.485 ms) : 1474, 1497
. : milestone, 1485,
appsec (3.721 ms) : 3502, 3940
. : milestone, 3721,
iast (2.226 ms) : 2161, 2290
. : milestone, 2226,
iast_GLOBAL (2.271 ms) : 2206, 2336
. : milestone, 2271,
profiling (2.072 ms) : 2020, 2124
. : milestone, 2072,
tracing (2.05 ms) : 2000, 2101
. : milestone, 2050,
section candidate
no_agent (1.49 ms) : 1479, 1502
. : milestone, 1490,
appsec (2.489 ms) : 2437, 2541
. : milestone, 2489,
iast (2.224 ms) : 2159, 2288
. : milestone, 2224,
iast_GLOBAL (2.275 ms) : 2210, 2340
. : milestone, 2275,
profiling (2.075 ms) : 2023, 2127
. : milestone, 2075,
tracing (2.046 ms) : 1995, 2096
. : milestone, 2046,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.56.0-SNAPSHOT~3241b0f7c3, baseline=1.56.0-SNAPSHOT~5adec51856
dateFormat X
axisFormat %s
section baseline
no_agent (14.945 s) : 14945000, 14945000
. : milestone, 14945000,
appsec (14.966 s) : 14966000, 14966000
. : milestone, 14966000,
iast (18.537 s) : 18537000, 18537000
. : milestone, 18537000,
iast_GLOBAL (17.885 s) : 17885000, 17885000
. : milestone, 17885000,
profiling (14.749 s) : 14749000, 14749000
. : milestone, 14749000,
tracing (14.879 s) : 14879000, 14879000
. : milestone, 14879000,
section candidate
no_agent (15.583 s) : 15583000, 15583000
. : milestone, 15583000,
appsec (15.349 s) : 15349000, 15349000
. : milestone, 15349000,
iast (18.568 s) : 18568000, 18568000
. : milestone, 18568000,
iast_GLOBAL (17.937 s) : 17937000, 17937000
. : milestone, 17937000,
profiling (15.167 s) : 15167000, 15167000
. : milestone, 15167000,
tracing (14.907 s) : 14907000, 14907000
. : milestone, 14907000,
|
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep it up! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
| RequestBody body = | ||
| RequestBody.create( | ||
| jsonMediaType, | ||
| writeValueAsString(event)); // MAYBE remove CoreTracer parameter since it is not used |
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.
If there's no plan to use it in the near future then I would remove the CoreTracer parameter now
mhlidd
left a comment
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.
LGTM
What Does This Do
tldr: Adds the lambda request ID as a header when making calls to start/end-invocation in lambda datadog extension
Motivation
APMSVLS-274
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: APMSVLS-274