Robert Pająk

Refactor Nuke tasks

1

The current build implementation using Nuke uses different APIs for defining and ordering dependencies, making it hard to maintain. We should document and use some guidelines to make it more...

This one is the most frequently failing flaky test - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/runs/7546662258?check_suite_focus=true

IntegrationTests.SqlClientTests.SubmitTraces [FAIL] Microsoft.Data.SqlClient.SqlException (0x80131904): Login failed for user 'sa'.

2

https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/runs/7444225072?check_suite_focus=true Flaky? Is it a problem with the generated password?

IntegrationTests.SmokeTests.PrometheusExporter [FAIL] Did not expect any exceptions after 30s, but found System.Net.Http.HttpRequestException with message "Connection refused"

2

Potentially flaky test - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/runs/7500128904?check_suite_focus=true - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/runs/7513623116?check_suite_focus=true

Examples: - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/actions/runs/2755073455 - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/actions/runs/2755126398 - https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/actions/runs/2749693766

What is our plan regarding bytecode instrumentation for metrics? For example, how we would add bytecode metrics instrumentation for GraphQL? _Originally posted by @pellared in https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/pull/693#discussion_r880128909_

Use OTel SDK's self-diagnostics - managed code

1

We should use [OTel SDK's self-diagnostics](https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/README.md#self-diagnostics) for internal logging.

Use OTel SDK's self-diagnostics - native code

2

We should take advantage of [OTel SDK's self-diagnostics](https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry/README.md#self-diagnostics) for internal logging and try to adopt it in the native code. We could do it by emitting EventSource events from the...

I mean only to ignore it for `src` but still have dependabot working for `test-applications` _Originally posted by @pellared in https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/issues/961#issuecomment-1191123515_

Update the AspNetCoreMvc example (or remove it) once https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/pull/979 is merged _Originally posted by @pellared in https://github.com/open-telemetry/opentelemetry-dotnet-instrumentation/issues/984#issuecomment-1195493892_