Mikel Blanchard

Results 181 comments of Mikel Blanchard

Related: https://github.com/open-telemetry/opentelemetry-specification/issues/4218

@Kielek Personally I'm not a fan of APIs which rely on magic strings. Could we do something more strongly typed? ```csharp public static class HostResourceBuilderExtensions { public static ResourceBuilder AddHostDetector(this...

So the issue here is the 8.0.0 - 8.0.4 versions of STJ were flagged with security issues. Check out the versions listed on [NuGet](https://www.nuget.org/packages/System.Text.Json) to see what I mean. The...

Hey @LGouellec! I spent some time on this tonight. Here is what I think is going on... * When you dispose your `Meter` we track the `Metric` as deactivated: https://github.com/open-telemetry/opentelemetry-dotnet/blob/9f41eadf03f3dcc5e76c686b61fb39849f046312/src/OpenTelemetry/Metrics/Reader/MetricReaderExt.cs#L29...

Running this: ```csharp public static void Main() { Meter? meter = null; var meterProviderBuilder = Sdk .CreateMeterProviderBuilder() .AddMeter("Test") .SetResourceBuilder( ResourceBuilder.CreateDefault() .AddService(serviceName: "Test")); meterProviderBuilder.AddOtlpExporter((exporterOptions, readerOptions) => { readerOptions.PeriodicExportingMetricReaderOptions.ExportIntervalMilliseconds = 5000; });...

Just FYI for anyone landing on here, as of 1.4.0 SDK named options work: ```csharp services.Configure("traces", opt => opt.Endpoint = myTraceEndpoint); services.Configure("metrics", opt => opt.Endpoint = myMetricEndpoint); tracerProviderBuilder.AddOtlpExporter(name: "traces", configure:...

We've had some talks in .NET but haven't made any decisions yet. We definitely want to support the config file. We also would like to support the standard .NET way...

At the moment .NET doesn't support `OTEL_TRACES_EXPORTER`. The only component-configuring key we support is `OTEL_TRACES_SAMPLER` / `OTEL_TRACES_SAMPLER_ARG` because the SDK happens to have the samplers on hand. That was just...

What's interesting is Logs spec has some language which could be seen as permissive: https://github.com/open-telemetry/opentelemetry-specification/blob/0a78571045ca1dca48621c9648ec3c832c3c541c/specification/logs/sdk.md?plain=1#L289-L291 If we consider changing the concurrency of the simple processor an advanced scenario :smile: I...