Open gleocadie opened 4 days ago
The following differences have been observed in committed snapshots. It is meant to help the reviewer. The diff is simplistic, so please check some files anyway while we improve it.
2 occurrences of :
+ "language": "dotnet",
5947 occurrences of :
+ language: dotnet,
1 occurrences of :
- {
- TraceId: Id_1,
- SpanId: Id_2,
- Name: Manual-1.Initial,
- Resource: Manual-1.Initial,
- Service: Samples.ManualInstrumentation,
- Tags: {
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_1,
- SpanId: Id_3,
- Name: Manual-1.Initial.HttpClient,
- Resource: Manual-1.Initial.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_2,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_4,
- SpanId: Id_5,
- Name: Manual-1.Initial.HttpListener,
- Resource: Manual-1.Initial.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_6,
- SpanId: Id_7,
- Name: Manual-2.Reconfigured,
- Resource: Manual-2.Reconfigured,
- Service: updated-name,
- Tags: {
- env: updated-env,
- language: dotnet,
- runtime-id: Guid_1,
- Updated-key: Updated Value
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_6,
- SpanId: Id_8,
- Name: Manual-2.Reconfigured.HttpClient,
- Resource: Manual-2.Reconfigured.HttpClient,
- Service: updated-name,
- ParentId: Id_7,
- Tags: {
- env: updated-env,
- language: dotnet,
- Updated-key: Updated Value
- }
- },
- {
- TraceId: Id_9,
- SpanId: Id_10,
- Name: Manual-2.Reconfigured.HttpListener,
- Resource: Manual-2.Reconfigured.HttpListener,
- Service: updated-name,
- Tags: {
- content: PONG,
- env: updated-env,
- language: dotnet,
- runtime-id: Guid_1,
- Updated-key: Updated Value
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_11,
- SpanId: Id_12,
- Name: Manual-3.HttpDisabled,
- Resource: Manual-3.HttpDisabled,
- Service: Samples.ManualInstrumentation,
- Tags: {
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_11,
- SpanId: Id_13,
- Name: Manual-3.HttpDisabled.HttpClient,
- Resource: Manual-3.HttpDisabled.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_12,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_14,
- SpanId: Id_15,
- Name: Manual-3.HttpDisabled.HttpListener,
- Resource: Manual-3.HttpDisabled.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_16,
- SpanId: Id_17,
- Name: Manual-4.DefaultsReinstated,
- Resource: Manual-4.DefaultsReinstated,
- Service: Samples.ManualInstrumentation,
- Tags: {
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_16,
- SpanId: Id_18,
- Name: Manual-4.DefaultsReinstated.HttpClient,
- Resource: Manual-4.DefaultsReinstated.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_17,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_19,
- SpanId: Id_20,
- Name: Manual-4.DefaultsReinstated.HttpListener,
- Resource: Manual-4.DefaultsReinstated.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_21,
- SpanId: Id_22,
- Name: Manual-5.Ext.HttpListener,
- Resource: Manual-5.Ext.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_23,
- SpanId: Id_24,
- Name: Manual-5.Ext.Outer,
- Resource: Manual-5.Ext.Outer,
- Service: Samples.ManualInstrumentation,
- Tags: {
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1,
- usr.email: test@example.com,
- usr.id: my-id,
- usr.name: Bits,
- usr.role: Mascot,
- usr.scope: test-scope,
- usr.session_id: abc123
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 2.0
- }
- },
- {
- TraceId: Id_23,
- SpanId: Id_25,
- Name: Manual-5.Ext.Inner,
- Resource: Manual-5.Ext.Inner,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_24,
- Error: 1,
- Tags: {
- Custom: Some-Value,
- env: integration_tests,
- error.msg: Exception of type 'CustomException' was thrown.,
- error.stack:
[...]
- error.type: CustomException,
- language: dotnet
- },
- Metrics: {
- Some-Number: 123.0
- }
- },
- {
- TraceId: Id_23,
- SpanId: Id_26,
- Name: Manual-5.Ext.HttpClient,
- Resource: Manual-5.Ext.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_25,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_27,
- SpanId: Id_28,
- Name: Manual-6.EventSdk.Custom.Outer,
- Resource: Manual-6.EventSdk.Custom.Outer,
- Service: Samples.ManualInstrumentation,
- Tags: {
- appsec.events.custom-event-meta.key-1: val-1,
- appsec.events.custom-event-meta.key-2: val-2,
- appsec.events.custom-event-meta.track: true,
- appsec.events.custom-event.track: true,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1,
- _dd.appsec.events.custom-event-meta.sdk: true,
- _dd.appsec.events.custom-event.sdk: true
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_27,
- SpanId: Id_29,
- Name: Manual-6.EventSdk.Custom.Inner,
- Resource: Manual-6.EventSdk.Custom.Inner,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_28,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_27,
- SpanId: Id_30,
- Name: Manual-6.Ext.HttpClient,
- Resource: Manual-6.Ext.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_29,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_31,
- SpanId: Id_32,
- Name: Manual-6.Ext.HttpListener,
- Resource: Manual-6.Ext.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_33,
- SpanId: Id_34,
- Name: Manual-7.EventSdk.Success.Outer,
- Resource: Manual-7.EventSdk.Success.Outer,
- Service: Samples.ManualInstrumentation,
- Tags: {
- appsec.events.users.login.success.key-1: val-1,
- appsec.events.users.login.success.key-2: val-2,
- appsec.events.users.login.success.track: true,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1,
- usr.id: my-id,
- _dd.appsec.events.users.login.success.sdk: true
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_33,
- SpanId: Id_35,
- Name: Manual-7.EventSdk.Success.Inner,
- Resource: Manual-7.EventSdk.Success.Inner,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_34,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_33,
- SpanId: Id_36,
- Name: Manual-7.Ext.HttpClient,
- Resource: Manual-7.Ext.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_35,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_37,
- SpanId: Id_38,
- Name: Manual-7.Ext.HttpListener,
- Resource: Manual-7.Ext.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_39,
- SpanId: Id_40,
- Name: Manual-8.EventSdk.Failure.Outer,
- Resource: Manual-8.EventSdk.Failure.Outer,
- Service: Samples.ManualInstrumentation,
- Tags: {
- appsec.events.users.login.failure.key-1: val-1,
- appsec.events.users.login.failure.key-2: val-2,
- appsec.events.users.login.failure.track: true,
- appsec.events.users.login.failure.usr.exists: true,
- appsec.events.users.login.failure.usr.id: my-id,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1,
- _dd.appsec.events.users.login.failure.sdk: true
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_39,
- SpanId: Id_41,
- Name: Manual-8.EventSdk.Failure.Inner,
- Resource: Manual-8.EventSdk.Failure.Inner,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_40,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_39,
- SpanId: Id_42,
- Name: Manual-8.Ext.HttpClient,
- Resource: Manual-8.Ext.HttpClient,
- Service: Samples.ManualInstrumentation,
- ParentId: Id_41,
- Tags: {
- env: integration_tests,
- language: dotnet
- }
- },
- {
- TraceId: Id_43,
- SpanId: Id_44,
- Name: Manual-8.Ext.HttpListener,
- Resource: Manual-8.Ext.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- },
- {
- TraceId: Id_45,
- SpanId: Id_46,
- Name: Manual-9.CustomContext.HttpListener,
- Resource: Manual-9.CustomContext.HttpListener,
- Service: Samples.ManualInstrumentation,
- Tags: {
- content: PONG,
- env: integration_tests,
- language: dotnet,
- runtime-id: Guid_1
- },
- Metrics: {
- process_id: 0,
- _dd.top_level: 1.0,
- _dd.tracer_kr: 1.0,
- _sampling_priority_v1: 1.0
- }
- }
Branch report: gleocadie/override-native-call-for-wrapped-functions
Commit report: 2930bfd
Test service: dd-trace-dotnet
:x: 11 Failed (0 Known Flaky), 259476 Passed, 1294 Skipped, 12h 14m 44.36s Total Time
This report shows up to 5 failed tests.
CheckCpuLimit
-Datadog.Profiler.IntegrationTests.SmokeTests.CpuLimitTest
- Details
<details>
<summary>Expand for error</summary>
```
Expected agent.NbCallsOnProfilingEndpoint to be greater than 0, but found 0.
```
</details>
CheckCpuLimit
- Datadog.Profiler.IntegrationTests.SmokeTests.CpuLimitTest
- Details
DoesNotInstrumentDotnetBuild
- Datadog.Trace.ClrProfiler.IntegrationTests.InstrumentationTests
- Details
InstrumentsDotNetRun
- Datadog.Trace.ClrProfiler.IntegrationTests.InstrumentationTests
- Details
InstrumentsDotNetTest
- Datadog.Trace.ClrProfiler.IntegrationTests.InstrumentationTests
- Details
Execution-time results for samples comparing the following branches/commits:
Execution-time benchmarks measure the whole time it takes to execute a program. And are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are shown in red. The following thresholds were used for comparing the execution times:
Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard.
Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph).
gantt
title Execution time (ms) FakeDbCommand (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (73ms) : 65, 81
. : milestone, 73,
master - mean (73ms) : 65, 82
. : milestone, 73,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (903ms) : 879, 928
. : milestone, 903,
master - mean (992ms) : 971, 1012
. : milestone, 992,
gantt
title Execution time (ms) FakeDbCommand (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (110ms) : 106, 114
. : milestone, 110,
master - mean (110ms) : 106, 113
. : milestone, 110,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (636ms) : 614, 658
. : milestone, 636,
master - mean (706ms) : 681, 731
. : milestone, 706,
gantt
title Execution time (ms) FakeDbCommand (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (93ms) : 91, 96
. : milestone, 93,
master - mean (93ms) : 90, 96
. : milestone, 93,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (592ms) : 575, 610
. : milestone, 592,
master - mean (658ms) : 634, 682
. : milestone, 658,
gantt
title Execution time (ms) HttpMessageHandler (.NET Framework 4.6.2)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (191ms) : 186, 195
. : milestone, 191,
master - mean (192ms) : 187, 197
. : milestone, 192,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (1,000ms) : 976, 1024
. : milestone, 1000,
master - mean (1,090ms) : 1061, 1118
. : milestone, 1090,
gantt
title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (276ms) : 270, 282
. : milestone, 276,
master - mean (278ms) : 273, 283
. : milestone, 278,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (829ms) : 800, 858
. : milestone, 829,
master - mean (878ms) : 854, 901
. : milestone, 878,
gantt
title Execution time (ms) HttpMessageHandler (.NET 6)
dateFormat X
axisFormat %s
todayMarker off
section Baseline
This PR (5749) - mean (265ms) : 261, 269
. : milestone, 265,
master - mean (266ms) : 262, 270
. : milestone, 266,
section CallTarget+Inlining+NGEN
This PR (5749) - mean (811ms) : 782, 841
. : milestone, 811,
master - mean (862ms) : 838, 886
. : milestone, 862,
Benchmarks for #5749 compared to master:
The following thresholds were used for comparing the benchmark speeds:
Allocation changes below 0.5% are ignored.
Summary of changes
Reason for change
Implementation details
Test coverage
Other details