-
Notifications
You must be signed in to change notification settings - Fork 323
Introduce ConfigOrigin.CALCULATED #9371
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
Conversation
|
🎯 Code Coverage 🔗 Commit SHA: 65484b5 | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 45 metrics, 14 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.048 s) : 0, 1047630
Total [baseline] (10.714 s) : 0, 10714036
Agent [candidate] (1.051 s) : 0, 1050617
Total [candidate] (10.677 s) : 0, 10676898
section appsec
Agent [baseline] (1.225 s) : 0, 1225308
Total [baseline] (10.807 s) : 0, 10807161
Agent [candidate] (1.229 s) : 0, 1228765
Total [candidate] (10.776 s) : 0, 10775684
section iast
Agent [baseline] (1.196 s) : 0, 1195567
Total [baseline] (11.037 s) : 0, 11036611
Agent [candidate] (1.189 s) : 0, 1188719
Total [candidate] (10.953 s) : 0, 10953134
section profiling
Agent [baseline] (1.2 s) : 0, 1199694
Total [baseline] (10.902 s) : 0, 10902255
Agent [candidate] (1.2 s) : 0, 1200308
Total [candidate] (10.905 s) : 0, 10904526
gantt
title petclinic - break down per module: candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.442 ms) : 0, 1442
crashtracking [candidate] (1.457 ms) : 0, 1457
BytebuddyAgent [baseline] (732.709 ms) : 0, 732709
BytebuddyAgent [candidate] (734.07 ms) : 0, 734070
GlobalTracer [baseline] (242.632 ms) : 0, 242632
GlobalTracer [candidate] (242.861 ms) : 0, 242861
AppSec [baseline] (30.069 ms) : 0, 30069
AppSec [candidate] (30.152 ms) : 0, 30152
Debugger [baseline] (6.06 ms) : 0, 6060
Debugger [candidate] (6.047 ms) : 0, 6047
Remote Config [baseline] (663.824 µs) : 0, 664
Remote Config [candidate] (657.638 µs) : 0, 658
Telemetry [baseline] (12.995 ms) : 0, 12995
Telemetry [candidate] (14.283 ms) : 0, 14283
section appsec
crashtracking [baseline] (1.459 ms) : 0, 1459
crashtracking [candidate] (1.447 ms) : 0, 1447
BytebuddyAgent [baseline] (756.984 ms) : 0, 756984
BytebuddyAgent [candidate] (759.258 ms) : 0, 759258
GlobalTracer [baseline] (235.756 ms) : 0, 235756
GlobalTracer [candidate] (237.022 ms) : 0, 237022
AppSec [baseline] (170.492 ms) : 0, 170492
AppSec [candidate] (170.582 ms) : 0, 170582
Debugger [baseline] (5.752 ms) : 0, 5752
Debugger [candidate] (5.689 ms) : 0, 5689
Remote Config [baseline] (646.295 µs) : 0, 646
Remote Config [candidate] (636.076 µs) : 0, 636
Telemetry [baseline] (9.333 ms) : 0, 9333
Telemetry [candidate] (8.409 ms) : 0, 8409
IAST [baseline] (23.736 ms) : 0, 23736
IAST [candidate] (23.707 ms) : 0, 23707
section iast
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.469 ms) : 0, 1469
BytebuddyAgent [baseline] (862.721 ms) : 0, 862721
BytebuddyAgent [candidate] (858.335 ms) : 0, 858335
GlobalTracer [baseline] (236.181 ms) : 0, 236181
GlobalTracer [candidate] (234.519 ms) : 0, 234519
AppSec [baseline] (25.658 ms) : 0, 25658
AppSec [candidate] (25.974 ms) : 0, 25974
Debugger [baseline] (8.54 ms) : 0, 8540
Debugger [candidate] (8.447 ms) : 0, 8447
Remote Config [baseline] (627.247 µs) : 0, 627
Remote Config [candidate] (611.848 µs) : 0, 612
Telemetry [baseline] (8.422 ms) : 0, 8422
Telemetry [candidate] (8.335 ms) : 0, 8335
IAST [baseline] (30.681 ms) : 0, 30681
IAST [candidate] (29.76 ms) : 0, 29760
section profiling
crashtracking [baseline] (1.42 ms) : 0, 1420
crashtracking [candidate] (1.417 ms) : 0, 1417
BytebuddyAgent [baseline] (763.915 ms) : 0, 763915
BytebuddyAgent [candidate] (764.781 ms) : 0, 764781
GlobalTracer [baseline] (222.898 ms) : 0, 222898
GlobalTracer [candidate] (222.622 ms) : 0, 222622
AppSec [baseline] (30.22 ms) : 0, 30220
AppSec [candidate] (30.288 ms) : 0, 30288
Debugger [baseline] (7.064 ms) : 0, 7064
Debugger [candidate] (6.339 ms) : 0, 6339
Remote Config [baseline] (709.4 µs) : 0, 709
Remote Config [candidate] (694.296 µs) : 0, 694
Telemetry [baseline] (15.734 ms) : 0, 15734
Telemetry [candidate] (16.128 ms) : 0, 16128
ProfilingAgent [baseline] (108.071 ms) : 0, 108071
ProfilingAgent [candidate] (108.193 ms) : 0, 108193
Profiling [baseline] (108.724 ms) : 0, 108724
Profiling [candidate] (108.849 ms) : 0, 108849
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.048 s) : 0, 1047697
Total [baseline] (8.654 s) : 0, 8653693
Agent [candidate] (1.05 s) : 0, 1050234
Total [candidate] (8.61 s) : 0, 8609903
section iast
Agent [baseline] (1.19 s) : 0, 1190473
Total [baseline] (9.32 s) : 0, 9319798
Agent [candidate] (1.189 s) : 0, 1188663
Total [candidate] (9.358 s) : 0, 9357605
gantt
title insecure-bank - break down per module: candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.459 ms) : 0, 1459
crashtracking [candidate] (1.443 ms) : 0, 1443
BytebuddyAgent [baseline] (732.623 ms) : 0, 732623
BytebuddyAgent [candidate] (734.234 ms) : 0, 734234
GlobalTracer [baseline] (241.826 ms) : 0, 241826
GlobalTracer [candidate] (242.869 ms) : 0, 242869
AppSec [baseline] (30.123 ms) : 0, 30123
AppSec [candidate] (30.342 ms) : 0, 30342
Debugger [baseline] (6.015 ms) : 0, 6015
Debugger [candidate] (6.06 ms) : 0, 6060
Remote Config [baseline] (668.866 µs) : 0, 669
Remote Config [candidate] (672.527 µs) : 0, 673
Telemetry [baseline] (14.016 ms) : 0, 14016
Telemetry [candidate] (13.642 ms) : 0, 13642
section iast
crashtracking [baseline] (1.477 ms) : 0, 1477
crashtracking [candidate] (1.465 ms) : 0, 1465
BytebuddyAgent [baseline] (859.984 ms) : 0, 859984
BytebuddyAgent [candidate] (858.525 ms) : 0, 858525
GlobalTracer [baseline] (234.5 ms) : 0, 234500
GlobalTracer [candidate] (234.237 ms) : 0, 234237
AppSec [baseline] (28.01 ms) : 0, 28010
AppSec [candidate] (27.773 ms) : 0, 27773
Debugger [baseline] (8.301 ms) : 0, 8301
Debugger [candidate] (7.677 ms) : 0, 7677
Remote Config [baseline] (601.048 µs) : 0, 601
Remote Config [candidate] (615.323 µs) : 0, 615
Telemetry [baseline] (8.405 ms) : 0, 8405
Telemetry [candidate] (8.501 ms) : 0, 8501
IAST [baseline] (28.054 ms) : 0, 28054
IAST [candidate] (28.823 ms) : 0, 28823
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 3 performance regressions! Performance is the same for 8 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section baseline
no_agent (36.762 ms) : 36466, 37059
. : milestone, 36762,
appsec (47.203 ms) : 46772, 47633
. : milestone, 47203,
code_origins (44.782 ms) : 44388, 45176
. : milestone, 44782,
iast (44.291 ms) : 43905, 44676
. : milestone, 44291,
profiling (49.219 ms) : 48724, 49714
. : milestone, 49219,
tracing (45.574 ms) : 45182, 45967
. : milestone, 45574,
section candidate
no_agent (36.116 ms) : 35826, 36407
. : milestone, 36116,
appsec (47.792 ms) : 47360, 48225
. : milestone, 47792,
code_origins (45.312 ms) : 44930, 45694
. : milestone, 45312,
iast (44.675 ms) : 44281, 45069
. : milestone, 44675,
profiling (51.618 ms) : 51156, 52080
. : milestone, 51618,
tracing (43.347 ms) : 42969, 43725
. : milestone, 43347,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section baseline
no_agent (4.361 ms) : 4312, 4410
. : milestone, 4361,
iast (9.308 ms) : 9149, 9468
. : milestone, 9308,
iast_FULL (14.385 ms) : 14098, 14672
. : milestone, 14385,
iast_GLOBAL (10.104 ms) : 9929, 10278
. : milestone, 10104,
profiling (8.586 ms) : 8454, 8719
. : milestone, 8586,
tracing (7.431 ms) : 7328, 7535
. : milestone, 7431,
section candidate
no_agent (4.425 ms) : 4373, 4476
. : milestone, 4425,
iast (9.239 ms) : 9081, 9398
. : milestone, 9239,
iast_FULL (14.15 ms) : 13868, 14432
. : milestone, 14150,
iast_GLOBAL (10.303 ms) : 10122, 10483
. : milestone, 10303,
profiling (8.999 ms) : 8858, 9140
. : milestone, 8999,
tracing (7.704 ms) : 7597, 7811
. : milestone, 7704,
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.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section baseline
no_agent (14.991 s) : 14991000, 14991000
. : milestone, 14991000,
appsec (14.972 s) : 14972000, 14972000
. : milestone, 14972000,
iast (18.345 s) : 18345000, 18345000
. : milestone, 18345000,
iast_GLOBAL (18.0 s) : 18000000, 18000000
. : milestone, 18000000,
profiling (15.256 s) : 15256000, 15256000
. : milestone, 15256000,
tracing (14.934 s) : 14934000, 14934000
. : milestone, 14934000,
section candidate
no_agent (15.591 s) : 15591000, 15591000
. : milestone, 15591000,
appsec (15.138 s) : 15138000, 15138000
. : milestone, 15138000,
iast (18.399 s) : 18399000, 18399000
. : milestone, 18399000,
iast_GLOBAL (18.08 s) : 18080000, 18080000
. : milestone, 18080000,
profiling (15.402 s) : 15402000, 15402000
. : milestone, 15402000,
tracing (14.922 s) : 14922000, 14922000
. : milestone, 14922000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.53.0-SNAPSHOT~65484b52b9, baseline=1.53.0-SNAPSHOT~27cd7f43d9
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (3.633 ms) : 3418, 3849
. : milestone, 3633,
iast (2.189 ms) : 2126, 2252
. : milestone, 2189,
iast_GLOBAL (2.23 ms) : 2167, 2293
. : milestone, 2230,
profiling (2.045 ms) : 1994, 2095
. : milestone, 2045,
tracing (2.016 ms) : 1967, 2065
. : milestone, 2016,
section candidate
no_agent (1.472 ms) : 1460, 1483
. : milestone, 1472,
appsec (3.633 ms) : 3418, 3848
. : milestone, 3633,
iast (2.196 ms) : 2133, 2259
. : milestone, 2196,
iast_GLOBAL (2.236 ms) : 2172, 2299
. : milestone, 2236,
profiling (2.056 ms) : 2004, 2107
. : milestone, 2056,
tracing (2.003 ms) : 1954, 2051
. : milestone, 2003,
|
| DEFAULT("default"); | ||
| DEFAULT("default"), | ||
| /** set when the config is calculated from other configs */ | ||
| CALCULATED("calculated"); |
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.
nitpick: I would put this new entry before DEFAULT (i.e. keep DEFAULT as the last entry)
| // If we have already found a non-default origin, the value is calculated from multiple | ||
| // sources. |
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.
this might keep the comment on one line...
| // If we have already found a non-default origin, the value is calculated from multiple | |
| // sources. | |
| // if we already have a non-default origin, the value is calculated from multiple sources |
(I'll avoid adding multiple comments, but the same applies to other occurrences below)
mcculls
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.
nice improvement!
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
What Does This Do
Introduces a new
ConfigOriginenum, "calculated". This origin is used inConfigProvider.getOrderedMapandConfigProvider.getMerged*methods.Motivation
ConfigProvider methods like
getOrderedMapandgetMergedMapwere reporting misleading data to telemetry about configurations that were calculated from multiple sources. This PR changes the behavior such that, if multiple sources are used to calculate a value, the telemetry reports origin "calculated" instead.This is inspired by "calculated" origins that already exist in dd-trace-dotnet and dd-trace-js.
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]