diff --git a/src/OpenTelemetry.Instrumentation.Dependencies/AzureClientsInstrumentation.cs b/src/OpenTelemetry.Instrumentation.Dependencies/AzureClientsInstrumentation.cs deleted file mode 100644 index ef1a717af29..00000000000 --- a/src/OpenTelemetry.Instrumentation.Dependencies/AzureClientsInstrumentation.cs +++ /dev/null @@ -1,49 +0,0 @@ -// -// Copyright The OpenTelemetry Authors -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. -// -using System; - -namespace OpenTelemetry.Instrumentation.Dependencies -{ - /// - /// AzureClients instrumentation. - /// TODO: Azure specific listeners would be moved out of this repo. - /// I believe this was initially put here for quick validation. - /// There were no unit tests covering this feature, so - /// cannot validate after Span is replaced with Activity. - /// - public class AzureClientsInstrumentation : IDisposable - { - private readonly DiagnosticSourceSubscriber diagnosticSourceSubscriber; - - /// - /// Initializes a new instance of the class. - /// - public AzureClientsInstrumentation() - { - this.diagnosticSourceSubscriber = new DiagnosticSourceSubscriber( - name => new AzureSdkDiagnosticListener(name), - listener => listener.Name.StartsWith("Azure."), - null); - this.diagnosticSourceSubscriber.Subscribe(); - } - - /// - public void Dispose() - { - this.diagnosticSourceSubscriber?.Dispose(); - } - } -} diff --git a/src/OpenTelemetry.Instrumentation.Dependencies/AzurePipelineInstrumentation.cs b/src/OpenTelemetry.Instrumentation.Dependencies/AzurePipelineInstrumentation.cs deleted file mode 100644 index 3deb2e204b2..00000000000 --- a/src/OpenTelemetry.Instrumentation.Dependencies/AzurePipelineInstrumentation.cs +++ /dev/null @@ -1,46 +0,0 @@ -// -// Copyright The OpenTelemetry Authors -// -// Licensed under the Apache License, Version 2.0 (the "License"); -// you may not use this file except in compliance with the License. -// You may obtain a copy of the License at -// -// http://www.apache.org/licenses/LICENSE-2.0 -// -// Unless required by applicable law or agreed to in writing, software -// distributed under the License is distributed on an "AS IS" BASIS, -// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. -// See the License for the specific language governing permissions and -// limitations under the License. -// -using System; - -namespace OpenTelemetry.Instrumentation.Dependencies -{ - /// - /// AzurePipeline instrumentation. - /// TODO: Azure specific listeners would be moved out of this repo. - /// I believe this was initially put here for quick validation. - /// There were no unit tests covering this feature, so - /// cannot validate after Span is replaced with Activity. - /// - public class AzurePipelineInstrumentation : IDisposable - { - private readonly DiagnosticSourceSubscriber diagnosticSourceSubscriber; - - /// - /// Initializes a new instance of the class. - /// - public AzurePipelineInstrumentation() - { - this.diagnosticSourceSubscriber = new DiagnosticSourceSubscriber(new AzureSdkDiagnosticListener("Azure.Pipeline"), null); - this.diagnosticSourceSubscriber.Subscribe(); - } - - /// - public void Dispose() - { - this.diagnosticSourceSubscriber?.Dispose(); - } - } -} diff --git a/src/OpenTelemetry.Instrumentation.Dependencies/OpenTelemetryBuilderExtensions.cs b/src/OpenTelemetry.Instrumentation.Dependencies/OpenTelemetryBuilderExtensions.cs index 6a6f5a8277c..73e0d24832d 100644 --- a/src/OpenTelemetry.Instrumentation.Dependencies/OpenTelemetryBuilderExtensions.cs +++ b/src/OpenTelemetry.Instrumentation.Dependencies/OpenTelemetryBuilderExtensions.cs @@ -39,7 +39,6 @@ public static OpenTelemetryBuilder AddDependencyInstrumentation(this OpenTelemet builder.AddHttpClientDependencyInstrumentation(); builder.AddSqlClientDependencyInstrumentation(); - builder.AddAzureClientsDependencyInstrumentation(); #if NETFRAMEWORK builder.AddHttpWebRequestDependencyInstrumentation(); #endif @@ -65,7 +64,6 @@ public static OpenTelemetryBuilder AddDependencyInstrumentation( builder.AddHttpClientDependencyInstrumentation(configureHttpClientInstrumentationOptions); builder.AddSqlClientDependencyInstrumentation(configureSqlClientInstrumentationOptions); - builder.AddAzureClientsDependencyInstrumentation(); #if NETFRAMEWORK builder.AddHttpWebRequestDependencyInstrumentation(); #endif @@ -139,24 +137,6 @@ public static OpenTelemetryBuilder AddSqlClientDependencyInstrumentation( return builder; } - /// - /// Enables instrumentation for Azure clients. - /// - /// being configured. - /// The instance of to chain the calls. - public static OpenTelemetryBuilder AddAzureClientsDependencyInstrumentation( - this OpenTelemetryBuilder builder) - { - if (builder == null) - { - throw new ArgumentNullException(nameof(builder)); - } - - builder.AddActivitySource(AzureSdkDiagnosticListener.ActivitySourceName); - builder.AddInstrumentation((activitySource) => new AzureClientsInstrumentation()); - return builder; - } - #if NETFRAMEWORK /// /// Enables the outgoing requests automatic data collection for .NET Framework HttpWebRequest activity source.