-
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
[WIP] Handle multi-value tagging for client spans #7943
Closed
Closed
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
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.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.086 s) : 0, 1085976
Total [baseline] (10.447 s) : 0, 10446724
Agent [candidate] (1.089 s) : 0, 1089386
Total [candidate] (10.537 s) : 0, 10536698
section appsec
Agent [baseline] (1.226 s) : 0, 1226112
Total [baseline] (10.737 s) : 0, 10737383
Agent [candidate] (1.229 s) : 0, 1228930
Total [candidate] (10.715 s) : 0, 10714516
section iast
Agent [baseline] (1.214 s) : 0, 1214461
Total [baseline] (10.936 s) : 0, 10935896
Agent [candidate] (1.225 s) : 0, 1224692
Total [candidate] (10.973 s) : 0, 10973014
section profiling
Agent [baseline] (1.29 s) : 0, 1289519
Total [baseline] (10.825 s) : 0, 10825051
Agent [candidate] (1.296 s) : 0, 1296254
Total [candidate] (10.833 s) : 0, 10832762
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (690.684 ms) : 0, 690684
BytebuddyAgent [candidate] (691.624 ms) : 0, 691624
GlobalTracer [baseline] (316.438 ms) : 0, 316438
GlobalTracer [candidate] (317.264 ms) : 0, 317264
AppSec [baseline] (54.509 ms) : 0, 54509
AppSec [candidate] (54.565 ms) : 0, 54565
Remote Config [baseline] (684.2 µs) : 0, 684
Remote Config [candidate] (687.855 µs) : 0, 688
Telemetry [baseline] (9.997 ms) : 0, 9997
Telemetry [candidate] (11.531 ms) : 0, 11531
section appsec
BytebuddyAgent [baseline] (712.264 ms) : 0, 712264
BytebuddyAgent [candidate] (713.582 ms) : 0, 713582
GlobalTracer [baseline] (314.559 ms) : 0, 314559
GlobalTracer [candidate] (316.253 ms) : 0, 316253
AppSec [baseline] (166.871 ms) : 0, 166871
AppSec [candidate] (167.297 ms) : 0, 167297
Remote Config [baseline] (647.964 µs) : 0, 648
Remote Config [candidate] (652.241 µs) : 0, 652
Telemetry [baseline] (7.895 ms) : 0, 7895
Telemetry [candidate] (7.506 ms) : 0, 7506
IAST [baseline] (20.388 ms) : 0, 20388
IAST [candidate] (19.675 ms) : 0, 19675
section iast
BytebuddyAgent [baseline] (807.846 ms) : 0, 807846
BytebuddyAgent [candidate] (815.428 ms) : 0, 815428
GlobalTracer [baseline] (305.855 ms) : 0, 305855
GlobalTracer [candidate] (307.936 ms) : 0, 307936
AppSec [baseline] (57.992 ms) : 0, 57992
AppSec [candidate] (58.112 ms) : 0, 58112
Remote Config [baseline] (620.315 µs) : 0, 620
Remote Config [candidate] (621.17 µs) : 0, 621
Telemetry [baseline] (7.461 ms) : 0, 7461
Telemetry [candidate] (7.552 ms) : 0, 7552
IAST [baseline] (21.003 ms) : 0, 21003
IAST [candidate] (21.188 ms) : 0, 21188
section profiling
BytebuddyAgent [baseline] (686.633 ms) : 0, 686633
BytebuddyAgent [candidate] (691.352 ms) : 0, 691352
GlobalTracer [baseline] (402.657 ms) : 0, 402657
GlobalTracer [candidate] (403.99 ms) : 0, 403990
AppSec [baseline] (55.489 ms) : 0, 55489
AppSec [candidate] (55.415 ms) : 0, 55415
Remote Config [baseline] (672.763 µs) : 0, 673
Remote Config [candidate] (682.668 µs) : 0, 683
Telemetry [baseline] (11.444 ms) : 0, 11444
Telemetry [candidate] (11.288 ms) : 0, 11288
ProfilingAgent [baseline] (93.585 ms) : 0, 93585
ProfilingAgent [candidate] (94.166 ms) : 0, 94166
Profiling [baseline] (93.61 ms) : 0, 93610
Profiling [candidate] (94.191 ms) : 0, 94191
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.094 s) : 0, 1093572
Total [baseline] (8.677 s) : 0, 8676574
Agent [candidate] (1.089 s) : 0, 1088835
Total [candidate] (8.645 s) : 0, 8645149
section iast
Agent [baseline] (1.227 s) : 0, 1227140
Total [baseline] (9.264 s) : 0, 9264349
Agent [candidate] (1.224 s) : 0, 1223988
Total [candidate] (9.183 s) : 0, 9182968
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.216 s) : 0, 1216406
Total [baseline] (9.206 s) : 0, 9206444
Agent [candidate] (1.215 s) : 0, 1214722
Total [candidate] (9.142 s) : 0, 9141601
section iast_TELEMETRY_OFF
Agent [baseline] (1.221 s) : 0, 1221349
Total [baseline] (9.226 s) : 0, 9225597
Agent [candidate] (1.215 s) : 0, 1214818
Total [candidate] (9.188 s) : 0, 9188352
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (696.806 ms) : 0, 696806
BytebuddyAgent [candidate] (693.027 ms) : 0, 693027
GlobalTracer [baseline] (318.172 ms) : 0, 318172
GlobalTracer [candidate] (317.485 ms) : 0, 317485
AppSec [baseline] (54.748 ms) : 0, 54748
AppSec [candidate] (54.582 ms) : 0, 54582
Remote Config [baseline] (703.794 µs) : 0, 704
Remote Config [candidate] (693.359 µs) : 0, 693
Telemetry [baseline] (9.313 ms) : 0, 9313
Telemetry [candidate] (9.297 ms) : 0, 9297
section iast
BytebuddyAgent [baseline] (815.592 ms) : 0, 815592
BytebuddyAgent [candidate] (814.838 ms) : 0, 814838
GlobalTracer [baseline] (309.329 ms) : 0, 309329
GlobalTracer [candidate] (307.489 ms) : 0, 307489
AppSec [baseline] (57.911 ms) : 0, 57911
AppSec [candidate] (56.833 ms) : 0, 56833
IAST [baseline] (22.129 ms) : 0, 22129
IAST [candidate] (22.746 ms) : 0, 22746
Remote Config [baseline] (663.759 µs) : 0, 664
Remote Config [candidate] (624.847 µs) : 0, 625
Telemetry [baseline] (7.7 ms) : 0, 7700
Telemetry [candidate] (7.597 ms) : 0, 7597
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (808.673 ms) : 0, 808673
BytebuddyAgent [candidate] (807.993 ms) : 0, 807993
GlobalTracer [baseline] (306.11 ms) : 0, 306110
GlobalTracer [candidate] (305.973 ms) : 0, 305973
AppSec [baseline] (57.865 ms) : 0, 57865
AppSec [candidate] (56.539 ms) : 0, 56539
IAST [baseline] (21.811 ms) : 0, 21811
IAST [candidate] (22.436 ms) : 0, 22436
Remote Config [baseline] (631.418 µs) : 0, 631
Remote Config [candidate] (622.023 µs) : 0, 622
Telemetry [baseline] (7.579 ms) : 0, 7579
Telemetry [candidate] (7.42 ms) : 0, 7420
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (811.654 ms) : 0, 811654
BytebuddyAgent [candidate] (805.338 ms) : 0, 805338
GlobalTracer [baseline] (307.717 ms) : 0, 307717
GlobalTracer [candidate] (307.704 ms) : 0, 307704
AppSec [baseline] (59.033 ms) : 0, 59033
AppSec [candidate] (59.292 ms) : 0, 59292
IAST [baseline] (20.903 ms) : 0, 20903
IAST [candidate] (20.573 ms) : 0, 20573
Remote Config [baseline] (631.919 µs) : 0, 632
Remote Config [candidate] (628.753 µs) : 0, 629
Telemetry [baseline] (7.587 ms) : 0, 7587
Telemetry [candidate] (7.542 ms) : 0, 7542
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section baseline
no_agent (361.297 µs) : 342, 381
. : milestone, 361,
iast (485.705 µs) : 464, 507
. : milestone, 486,
iast_FULL (645.172 µs) : 624, 667
. : milestone, 645,
iast_GLOBAL (507.309 µs) : 486, 529
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (484.52 µs) : 463, 506
. : milestone, 485,
iast_INACTIVE (441.147 µs) : 420, 462
. : milestone, 441,
iast_TELEMETRY_OFF (477.021 µs) : 456, 498
. : milestone, 477,
tracing (441.907 µs) : 421, 463
. : milestone, 442,
section candidate
no_agent (371.935 µs) : 351, 392
. : milestone, 372,
iast (485.106 µs) : 464, 507
. : milestone, 485,
iast_FULL (644.574 µs) : 623, 666
. : milestone, 645,
iast_GLOBAL (516.792 µs) : 495, 539
. : milestone, 517,
iast_HARDCODED_SECRET_DISABLED (490.624 µs) : 469, 512
. : milestone, 491,
iast_INACTIVE (445.086 µs) : 424, 466
. : milestone, 445,
iast_TELEMETRY_OFF (470.384 µs) : 449, 492
. : milestone, 470,
tracing (444.377 µs) : 423, 466
. : milestone, 444,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section baseline
no_agent (1.343 ms) : 1323, 1363
. : milestone, 1343,
appsec (1.739 ms) : 1714, 1764
. : milestone, 1739,
appsec_no_iast (1.748 ms) : 1724, 1772
. : milestone, 1748,
iast (1.464 ms) : 1442, 1486
. : milestone, 1464,
profiling (1.498 ms) : 1473, 1522
. : milestone, 1498,
tracing (1.472 ms) : 1448, 1497
. : milestone, 1472,
section candidate
no_agent (1.337 ms) : 1317, 1358
. : milestone, 1337,
appsec (1.744 ms) : 1720, 1769
. : milestone, 1744,
appsec_no_iast (1.751 ms) : 1726, 1776
. : milestone, 1751,
iast (1.484 ms) : 1462, 1507
. : milestone, 1484,
profiling (1.535 ms) : 1512, 1559
. : milestone, 1535,
tracing (1.483 ms) : 1458, 1508
. : milestone, 1483,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 5 performance regressions! Performance is the same for 7 metrics, 0 unstable metrics.
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section baseline
no_agent (15.034 s) : 15034000, 15034000
. : milestone, 15034000,
appsec (15.008 s) : 15008000, 15008000
. : milestone, 15008000,
iast (18.89 s) : 18890000, 18890000
. : milestone, 18890000,
iast_GLOBAL (17.576 s) : 17576000, 17576000
. : milestone, 17576000,
profiling (14.747 s) : 14747000, 14747000
. : milestone, 14747000,
tracing (14.899 s) : 14899000, 14899000
. : milestone, 14899000,
section candidate
no_agent (14.932 s) : 14932000, 14932000
. : milestone, 14932000,
appsec (14.965 s) : 14965000, 14965000
. : milestone, 14965000,
iast (18.639 s) : 18639000, 18639000
. : milestone, 18639000,
iast_GLOBAL (18.317 s) : 18317000, 18317000
. : milestone, 18317000,
profiling (14.816 s) : 14816000, 14816000
. : milestone, 14816000,
tracing (14.926 s) : 14926000, 14926000
. : milestone, 14926000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~7573e13ddb, baseline=1.43.0-SNAPSHOT~58de7ff6c6
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1458, 1481
. : milestone, 1469,
appsec (2.338 ms) : 2296, 2379
. : milestone, 2338,
iast (2.087 ms) : 2034, 2140
. : milestone, 2087,
iast_GLOBAL (2.13 ms) : 2078, 2183
. : milestone, 2130,
profiling (1.968 ms) : 1926, 2011
. : milestone, 1968,
tracing (1.924 ms) : 1884, 1964
. : milestone, 1924,
section candidate
no_agent (1.469 ms) : 1458, 1481
. : milestone, 1469,
appsec (3.376 ms) : 3332, 3421
. : milestone, 3376,
iast (3.475 ms) : 3420, 3530
. : milestone, 3475,
iast_GLOBAL (3.465 ms) : 3409, 3521
. : milestone, 3465,
profiling (3.454 ms) : 3407, 3501
. : milestone, 3454,
tracing (3.322 ms) : 3278, 3366
. : milestone, 3322,
|
f54d547
to
a8ade65
Compare
a8ade65
to
570368e
Compare
5 tasks
5 tasks
moved to #7996 |
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.
This PR is getting very large and convoluted. I am going to split it up and start small.
What Does This Do
The goal of this PR is to handle multi-value tagging for client spans.
Motivation
Currently, the HTTP protocol allows splitting the header into multiple lines per value; however, the Java tracer only allows users to extrace the first value of the HTTP header. This PR will now allow extracting multi-valued multi-line headers. This PR would help allow extracting multi-valued multi-line HTTP headers in conjunction with #7943.
Additional Notes
This PR is made in response to this Jira ticket.
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: [PROJ-IDENT]