From 0b17dd813929658bead9a921d368e6623bbd71f7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Piotr=20Kie=C5=82kowicz?= Date: Fri, 3 Mar 2023 16:53:32 +0100 Subject: [PATCH] Remove references to Jaeger exporter (#1060) --- examples/AspNet/Examples.AspNet.csproj | 1 - examples/AspNet/Global.asax.cs | 7 ------- examples/AspNet/Web.config | 2 -- src/OpenTelemetry.Instrumentation.AspNet/README.md | 6 +++--- .../README.md | 4 ++-- src/OpenTelemetry.Instrumentation.MassTransit/README.md | 4 ++-- src/OpenTelemetry.Instrumentation.Quartz/README.md | 6 +++--- 7 files changed, 10 insertions(+), 20 deletions(-) diff --git a/examples/AspNet/Examples.AspNet.csproj b/examples/AspNet/Examples.AspNet.csproj index 9a71f598b7..0d181ef44c 100644 --- a/examples/AspNet/Examples.AspNet.csproj +++ b/examples/AspNet/Examples.AspNet.csproj @@ -83,7 +83,6 @@ - diff --git a/examples/AspNet/Global.asax.cs b/examples/AspNet/Global.asax.cs index b32f6dfaf3..db50f5bd43 100644 --- a/examples/AspNet/Global.asax.cs +++ b/examples/AspNet/Global.asax.cs @@ -42,13 +42,6 @@ protected void Application_Start() switch (ConfigurationManager.AppSettings["UseExporter"].ToLowerInvariant()) { - case "jaeger": - builder.AddJaegerExporter(jaegerOptions => - { - jaegerOptions.AgentHost = ConfigurationManager.AppSettings["JaegerHost"]; - jaegerOptions.AgentPort = int.Parse(ConfigurationManager.AppSettings["JaegerPort"]); - }); - break; case "zipkin": builder.AddZipkinExporter(zipkinOptions => { diff --git a/examples/AspNet/Web.config b/examples/AspNet/Web.config index d5496c456d..9eebb5c685 100644 --- a/examples/AspNet/Web.config +++ b/examples/AspNet/Web.config @@ -7,8 +7,6 @@ - - diff --git a/src/OpenTelemetry.Instrumentation.AspNet/README.md b/src/OpenTelemetry.Instrumentation.AspNet/README.md index 46586814be..0ba136ba1d 100644 --- a/src/OpenTelemetry.Instrumentation.AspNet/README.md +++ b/src/OpenTelemetry.Instrumentation.AspNet/README.md @@ -57,8 +57,8 @@ following shows changes required to your `Web.config` when using IIS web server. ASP.NET instrumentation must be enabled at application startup. This is typically done in the `Global.asax.cs` as shown below. This example also sets up -the OpenTelemetry Jaeger exporter, which requires adding the package -[`OpenTelemetry.Exporter.Jaeger`](https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry.Exporter.Jaeger/README.md) +the OpenTelemetry OTLP exporter, which requires adding the package +[`OpenTelemetry.Exporter.OpenTelemetryProtocol`](https://github.com/open-telemetry/opentelemetry-dotnet/blob/main/src/OpenTelemetry.Exporter.OpenTelemetryProtocol/README.md) to the application. ```csharp @@ -72,7 +72,7 @@ public class WebApiApplication : HttpApplication { this.tracerProvider = Sdk.CreateTracerProviderBuilder() .AddAspNetInstrumentation() - .AddJaegerExporter() + .AddOtlpExporter() .Build(); } protected void Application_End() diff --git a/src/OpenTelemetry.Instrumentation.ElasticsearchClient/README.md b/src/OpenTelemetry.Instrumentation.ElasticsearchClient/README.md index 1c5734e509..9c2d8c555e 100644 --- a/src/OpenTelemetry.Instrumentation.ElasticsearchClient/README.md +++ b/src/OpenTelemetry.Instrumentation.ElasticsearchClient/README.md @@ -23,8 +23,8 @@ ASP.NET Core instrumentation example: services.AddOpenTelemetry().WithTracing(x => { x.AddElasticsearchClientInstrumentation(); - x.UseJaegerExporter(config => { - // Configure Jaeger + x.AddOtlpExporter(config => { + // Configure OTLP }); }); ``` diff --git a/src/OpenTelemetry.Instrumentation.MassTransit/README.md b/src/OpenTelemetry.Instrumentation.MassTransit/README.md index 06300e2d1d..159adc9558 100644 --- a/src/OpenTelemetry.Instrumentation.MassTransit/README.md +++ b/src/OpenTelemetry.Instrumentation.MassTransit/README.md @@ -52,8 +52,8 @@ services.AddMassTransitHostedService(); services.AddOpenTelemetrySdk(x => { x.AddMassTransitInstrumentation(); - x.UseJaegerExporter(config => { - // Configure Jaeger + x.AddOtlpExporter(config => { + // Configure OTLP }); }); ``` diff --git a/src/OpenTelemetry.Instrumentation.Quartz/README.md b/src/OpenTelemetry.Instrumentation.Quartz/README.md index 0e7edb1e2d..06653ecda6 100644 --- a/src/OpenTelemetry.Instrumentation.Quartz/README.md +++ b/src/OpenTelemetry.Instrumentation.Quartz/README.md @@ -9,7 +9,7 @@ Automatically instruments the Quartz jobs from ## Supported Frameworks QuartzNET Instrumentation is only supported when using .NET Framework >= -`net472` and netstandard >= `netstandard2.0`. Quartz`net461` support for +`net472` and .NET Standard >= `netstandard2.0`. Quartz`net461` support for activity sources has been removed, more information can be found [here](https://www.quartz-scheduler.net/2021/04/07/quartznet-3-3-released/). @@ -45,8 +45,8 @@ public void ConfigureServices(IServiceCollection services) services.AddOpenTelemetry().WithTracing(x => { x.AddQuartzInstrumentation(); - x.UseJaegerExporter(config => { - // Configure Jaeger + x.AddOtlpExporter(config => { + // Configure OTLP }); }); ```