-
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
Google PubSub: implement switchable legacy tracing mode #7564
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 50 metrics, 13 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.053 s) : 0, 1053401
Total [baseline] (8.531 s) : 0, 8530689
Agent [candidate] (1.053 s) : 0, 1052922
Total [candidate] (8.474 s) : 0, 8473764
section iast
Agent [baseline] (1.186 s) : 0, 1185934
Total [baseline] (8.989 s) : 0, 8989380
Agent [candidate] (1.176 s) : 0, 1176450
Total [candidate] (8.983 s) : 0, 8983489
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.176 s) : 0, 1175698
Total [baseline] (8.917 s) : 0, 8917084
Agent [candidate] (1.176 s) : 0, 1175757
Total [candidate] (8.964 s) : 0, 8964468
section iast_TELEMETRY_OFF
Agent [baseline] (1.171 s) : 0, 1171077
Total [baseline] (8.959 s) : 0, 8958705
Agent [candidate] (1.171 s) : 0, 1171304
Total [candidate] (8.962 s) : 0, 8961840
gantt
title insecure-bank - break down per module: candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (672.035 ms) : 0, 672035
BytebuddyAgent [candidate] (671.687 ms) : 0, 671687
GlobalTracer [baseline] (308.187 ms) : 0, 308187
GlobalTracer [candidate] (307.809 ms) : 0, 307809
AppSec [baseline] (51.39 ms) : 0, 51390
AppSec [candidate] (51.651 ms) : 0, 51651
Remote Config [baseline] (676.359 µs) : 0, 676
Remote Config [candidate] (663.803 µs) : 0, 664
Telemetry [baseline] (7.505 ms) : 0, 7505
Telemetry [candidate] (7.521 ms) : 0, 7521
section iast
BytebuddyAgent [baseline] (787.238 ms) : 0, 787238
BytebuddyAgent [candidate] (782.708 ms) : 0, 782708
GlobalTracer [baseline] (300.466 ms) : 0, 300466
GlobalTracer [candidate] (297.033 ms) : 0, 297033
AppSec [baseline] (52.615 ms) : 0, 52615
AppSec [candidate] (51.522 ms) : 0, 51522
IAST [baseline] (23.205 ms) : 0, 23205
IAST [candidate] (23.627 ms) : 0, 23627
Remote Config [baseline] (605.659 µs) : 0, 606
Remote Config [candidate] (579.008 µs) : 0, 579
Telemetry [baseline] (8.119 ms) : 0, 8119
Telemetry [candidate] (7.354 ms) : 0, 7354
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (781.815 ms) : 0, 781815
BytebuddyAgent [candidate] (781.617 ms) : 0, 781617
GlobalTracer [baseline] (296.366 ms) : 0, 296366
GlobalTracer [candidate] (296.565 ms) : 0, 296565
AppSec [baseline] (51.643 ms) : 0, 51643
AppSec [candidate] (53.385 ms) : 0, 53385
IAST [baseline] (24.421 ms) : 0, 24421
IAST [candidate] (22.698 ms) : 0, 22698
Remote Config [baseline] (580.738 µs) : 0, 581
Remote Config [candidate] (584.52 µs) : 0, 585
Telemetry [baseline] (7.274 ms) : 0, 7274
Telemetry [candidate] (7.276 ms) : 0, 7276
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (778.129 ms) : 0, 778129
BytebuddyAgent [candidate] (778.662 ms) : 0, 778662
GlobalTracer [baseline] (296.291 ms) : 0, 296291
GlobalTracer [candidate] (296.063 ms) : 0, 296063
AppSec [baseline] (52.907 ms) : 0, 52907
AppSec [candidate] (51.374 ms) : 0, 51374
IAST [baseline] (22.303 ms) : 0, 22303
IAST [candidate] (22.285 ms) : 0, 22285
Remote Config [baseline] (581.058 µs) : 0, 581
Remote Config [candidate] (572.203 µs) : 0, 572
Telemetry [baseline] (7.234 ms) : 0, 7234
Telemetry [candidate] (8.709 ms) : 0, 8709
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.059 s) : 0, 1058746
Total [baseline] (10.383 s) : 0, 10382878
Agent [candidate] (1.052 s) : 0, 1052244
Total [candidate] (10.362 s) : 0, 10361538
section appsec
Agent [baseline] (1.184 s) : 0, 1183741
Total [baseline] (10.618 s) : 0, 10618069
Agent [candidate] (1.19 s) : 0, 1190250
Total [candidate] (10.671 s) : 0, 10670595
section iast
Agent [baseline] (1.186 s) : 0, 1186066
Total [baseline] (10.842 s) : 0, 10841576
Agent [candidate] (1.177 s) : 0, 1177172
Total [candidate] (10.782 s) : 0, 10781597
section profiling
Agent [baseline] (1.245 s) : 0, 1245483
Total [baseline] (10.581 s) : 0, 10581266
Agent [candidate] (1.251 s) : 0, 1250540
Total [candidate] (10.568 s) : 0, 10567889
gantt
title petclinic - break down per module: candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (675.912 ms) : 0, 675912
BytebuddyAgent [candidate] (671.299 ms) : 0, 671299
GlobalTracer [baseline] (309.194 ms) : 0, 309194
GlobalTracer [candidate] (307.851 ms) : 0, 307851
AppSec [baseline] (51.729 ms) : 0, 51729
AppSec [candidate] (51.347 ms) : 0, 51347
Remote Config [baseline] (679.092 µs) : 0, 679
Remote Config [candidate] (657.098 µs) : 0, 657
Telemetry [baseline] (7.56 ms) : 0, 7560
Telemetry [candidate] (7.474 ms) : 0, 7474
section appsec
BytebuddyAgent [baseline] (691.452 ms) : 0, 691452
BytebuddyAgent [candidate] (695.219 ms) : 0, 695219
GlobalTracer [baseline] (300.592 ms) : 0, 300592
GlobalTracer [candidate] (302.017 ms) : 0, 302017
AppSec [baseline] (159.776 ms) : 0, 159776
AppSec [candidate] (159.897 ms) : 0, 159897
IAST [baseline] (19.682 ms) : 0, 19682
IAST [candidate] (22.429 ms) : 0, 22429
Remote Config [baseline] (619.822 µs) : 0, 620
Remote Config [candidate] (626.601 µs) : 0, 627
Telemetry [baseline] (7.878 ms) : 0, 7878
Telemetry [candidate] (7.607 ms) : 0, 7607
section iast
BytebuddyAgent [baseline] (789.232 ms) : 0, 789232
BytebuddyAgent [candidate] (783.273 ms) : 0, 783273
GlobalTracer [baseline] (298.648 ms) : 0, 298648
GlobalTracer [candidate] (296.511 ms) : 0, 296511
AppSec [baseline] (51.236 ms) : 0, 51236
AppSec [candidate] (53.251 ms) : 0, 53251
IAST [baseline] (23.039 ms) : 0, 23039
IAST [candidate] (22.699 ms) : 0, 22699
Remote Config [baseline] (605.121 µs) : 0, 605
Remote Config [candidate] (584.113 µs) : 0, 584
Telemetry [baseline] (9.6 ms) : 0, 9600
Telemetry [candidate] (7.246 ms) : 0, 7246
section profiling
BytebuddyAgent [baseline] (664.144 ms) : 0, 664144
BytebuddyAgent [candidate] (666.571 ms) : 0, 666571
GlobalTracer [baseline] (388.656 ms) : 0, 388656
GlobalTracer [candidate] (390.083 ms) : 0, 390083
AppSec [baseline] (51.851 ms) : 0, 51851
AppSec [candidate] (52.274 ms) : 0, 52274
Remote Config [baseline] (689.953 µs) : 0, 690
Remote Config [candidate] (684.627 µs) : 0, 685
Telemetry [baseline] (7.367 ms) : 0, 7367
Telemetry [candidate] (7.42 ms) : 0, 7420
ProfilingAgent [baseline] (95.008 ms) : 0, 95008
ProfilingAgent [candidate] (95.617 ms) : 0, 95617
Profiling [baseline] (95.031 ms) : 0, 95031
Profiling [candidate] (95.641 ms) : 0, 95641
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 19 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section baseline
no_agent (360.967 µs) : 342, 380
. : milestone, 361,
iast (476.145 µs) : 455, 498
. : milestone, 476,
iast_FULL (549.058 µs) : 528, 570
. : milestone, 549,
iast_GLOBAL (492.299 µs) : 471, 513
. : milestone, 492,
iast_HARDCODED_SECRET_DISABLED (469.035 µs) : 447, 491
. : milestone, 469,
iast_INACTIVE (432.545 µs) : 412, 453
. : milestone, 433,
iast_TELEMETRY_OFF (474.284 µs) : 452, 496
. : milestone, 474,
tracing (441.829 µs) : 421, 463
. : milestone, 442,
section candidate
no_agent (367.423 µs) : 347, 388
. : milestone, 367,
iast (476.527 µs) : 455, 498
. : milestone, 477,
iast_FULL (549.19 µs) : 528, 570
. : milestone, 549,
iast_GLOBAL (501.411 µs) : 480, 523
. : milestone, 501,
iast_HARDCODED_SECRET_DISABLED (478.035 µs) : 456, 500
. : milestone, 478,
iast_INACTIVE (437.091 µs) : 416, 458
. : milestone, 437,
iast_TELEMETRY_OFF (467.352 µs) : 445, 490
. : milestone, 467,
tracing (433.397 µs) : 412, 455
. : milestone, 433,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section baseline
no_agent (1.335 ms) : 1317, 1354
. : milestone, 1335,
appsec (1.727 ms) : 1704, 1750
. : milestone, 1727,
appsec_no_iast (1.749 ms) : 1723, 1776
. : milestone, 1749,
iast (1.463 ms) : 1440, 1486
. : milestone, 1463,
profiling (1.467 ms) : 1443, 1490
. : milestone, 1467,
tracing (1.473 ms) : 1449, 1497
. : milestone, 1473,
section candidate
no_agent (1.346 ms) : 1327, 1366
. : milestone, 1346,
appsec (1.718 ms) : 1694, 1742
. : milestone, 1718,
appsec_no_iast (1.728 ms) : 1704, 1752
. : milestone, 1728,
iast (1.466 ms) : 1443, 1488
. : milestone, 1466,
profiling (1.482 ms) : 1459, 1505
. : milestone, 1482,
tracing (1.472 ms) : 1448, 1496
. : milestone, 1472,
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.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section baseline
no_agent (15.273 s) : 15273000, 15273000
. : milestone, 15273000,
appsec (15.047 s) : 15047000, 15047000
. : milestone, 15047000,
iast (18.81 s) : 18810000, 18810000
. : milestone, 18810000,
iast_GLOBAL (17.652 s) : 17652000, 17652000
. : milestone, 17652000,
profiling (15.382 s) : 15382000, 15382000
. : milestone, 15382000,
tracing (14.826 s) : 14826000, 14826000
. : milestone, 14826000,
section candidate
no_agent (15.065 s) : 15065000, 15065000
. : milestone, 15065000,
appsec (15.092 s) : 15092000, 15092000
. : milestone, 15092000,
iast (18.671 s) : 18671000, 18671000
. : milestone, 18671000,
iast_GLOBAL (18.228 s) : 18228000, 18228000
. : milestone, 18228000,
profiling (15.138 s) : 15138000, 15138000
. : milestone, 15138000,
tracing (15.067 s) : 15067000, 15067000
. : milestone, 15067000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.40.0-SNAPSHOT~4cb14e94e8, baseline=1.40.0-SNAPSHOT~86d01e6f14
dateFormat X
axisFormat %s
section baseline
no_agent (1.457 ms) : 1446, 1468
. : milestone, 1457,
appsec (2.218 ms) : 2183, 2253
. : milestone, 2218,
iast (1.993 ms) : 1950, 2037
. : milestone, 1993,
iast_GLOBAL (2.021 ms) : 1977, 2064
. : milestone, 2021,
profiling (1.86 ms) : 1826, 1895
. : milestone, 1860,
tracing (1.841 ms) : 1808, 1875
. : milestone, 1841,
section candidate
no_agent (1.463 ms) : 1452, 1475
. : milestone, 1463,
appsec (2.223 ms) : 2188, 2258
. : milestone, 2223,
iast (1.961 ms) : 1919, 2004
. : milestone, 1961,
iast_GLOBAL (2.027 ms) : 1983, 2071
. : milestone, 2027,
profiling (2.348 ms) : 2160, 2537
. : milestone, 2348,
tracing (1.832 ms) : 1799, 1864
. : milestone, 1832,
|
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
Google pubsub was lacking of the possibility to turn off the legacy tracing mode. When off, the consumer will use the value of
DD_SERVICE
instead thatgoogle-pubsub
as a service name.The feature can be activated by:
-Ddd.google-pubsub.legacy.tracing.enabled=false
DD_GOOGLE_PUBSUB_LEGACY_TRACING_ENABLED=false
Motivation
see #7209
Additional Notes
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]