Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Describe OTLP exporter usage #350

Merged
merged 3 commits into from
Feb 8, 2022
Merged

Conversation

pellared
Copy link
Member

@pellared pellared commented Feb 7, 2022

Why

Currently, trying to use the OTLP exporter usually leads to problems.

Related issue #325

Related PR #343

What

Describe the current state of the OTLP exporter support. Proper support of even deperecation of the gRPC protocol can be done later.

Remarks

For .NET Framework, it is probably more complicated. When I added

<PackageReference Include="Grpc.Net.Client" Version="2.32.0" />

to the sample app I got:

Unhandled Exception: System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory 
is corrupt.
   at Grpc.Core.Internal.CompletionQueueSafeHandle.ReleaseHandle()
   at System.Runtime.InteropServices.SafeHandle.InternalDispose()
   at System.Runtime.InteropServices.SafeHandle.Dispose(Boolean disposing)
   at System.Runtime.InteropServices.SafeHandle.Dispose()
   at Grpc.Core.Internal.AsyncCall`2.UnaryCall(TRequest msg)
   at Grpc.Core.DefaultCallInvoker.BlockingUnaryCall[TRequest,TResponse](Method`2 method, String host, CallOptions options, TRequest request)    
   at Grpc.Core.Interceptors.InterceptingCallInvoker.<BlockingUnaryCall>b__3_0[TRequest,TResponse](TRequest req, ClientInterceptorContext`2 ctx) 
   at Grpc.Core.ClientBase.ClientBaseConfiguration.ClientBaseConfigurationInterceptor.BlockingUnaryCall[TRequest,TResponse](TRequest request, ClientInterceptorContext`2 context, BlockingUnaryCallContinuation`2 continuation)
   at Grpc.Core.Interceptors.InterceptingCallInvoker.BlockingUnaryCall[TRequest,TResponse](Method`2 method, String host, CallOptions options, TRequest request)
   at Opentelemetry.Proto.Collector.Trace.V1.TraceService.TraceServiceClient.Export(ExportTraceServiceRequest request, CallOptions options)      
   at Opentelemetry.Proto.Collector.Trace.V1.TraceService.TraceServiceClient.Export(ExportTraceServiceRequest request, Metadata headers, Nullable`1 deadline, CancellationToken cancellationToken)
   at OpenTelemetry.Exporter.OpenTelemetryProtocol.Implementation.ExportClient.OtlpGrpcTraceExportClient.SendExportRequest(ExportTraceServiceRequest request, CancellationToken cancellationToken)
   at OpenTelemetry.Exporter.OtlpTraceExporter.Export(Batch`1& activityBatch)
   at OpenTelemetry.BatchExportProcessor`1.ExporterProc()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state, Boolean preserveSyncCtx)  
   at System.Threading.ExecutionContext.Run(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()
Segmentation fault

@pellared pellared changed the title [WIP] Describe OTLP exporter usage Describe OTLP exporter usage Feb 7, 2022
@pellared pellared marked this pull request as ready for review February 7, 2022 12:14
@pellared pellared requested a review from a team February 7, 2022 12:14
Copy link
Contributor

@pjanotti pjanotti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - just need to be careful with the changes in #351

docs/USAGE.md Show resolved Hide resolved
@pjanotti pjanotti merged commit d741e31 into open-telemetry:main Feb 8, 2022
@pellared pellared deleted the otlp-usage branch February 8, 2022 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants