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

Add a top-level UseDebugExporter() convenience method. #5671

Open
martinjt opened this issue Jun 5, 2024 · 2 comments
Open

Add a top-level UseDebugExporter() convenience method. #5671

martinjt opened this issue Jun 5, 2024 · 2 comments
Labels
enhancement New feature or request pkg:OpenTelemetry.Exporter.Console Issues related to OpenTelemetry.Exporter.Console NuGet package

Comments

@martinjt
Copy link
Member

martinjt commented Jun 5, 2024

Package

OpenTelemetry.Exporter.Console

Is your feature request related to a problem?

No response

What is the expected behavior?

I think we should have a new API for adding debug exporting across all signals.

builder.Services.AddOpenTelemetry()
    .UseDebugExporter()
    

This would add the Console exporter for each of the signals.

I think this also makes it clear that this is for "debug" and not a console output of data.

It's also inline with the new UseOtlpExporter() that was added at the top level.

Which alternative solutions or features have you considered?

We can keep doing what we have been, and using the individual console exporter lines, this is mainly a convenience and also something to bring it inline with the OTLP exporter as the 2 most commonly used for both production and getting started usecases.

Additional context

No response

@martinjt martinjt added the enhancement New feature or request label Jun 5, 2024
@github-actions github-actions bot added the pkg:OpenTelemetry.Exporter.Console Issues related to OpenTelemetry.Exporter.Console NuGet package label Jun 5, 2024
@KennethHoff
Copy link

Shouldn't this be called UseConsoleExporter? It's presumably not using the Debug.* API.

@CodeBlanch
Copy link
Member

I am supportive of a cross-cutting UseConsoleExporter extension! I intended to add a lot more cross-cutting things but my efforts have been stalled by #5325. Without a way for .NET Framework/non-host users to use those extensions it didn't seem like a great investment. But it looks like we'll be able to get that API done soon (just waiting for 1.9.0 to go out before adding anything).

Vision: #5262 (comment)

On there it is called "AddConsoleExporter" but that was before we settled on "Use" for the OTLP one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request pkg:OpenTelemetry.Exporter.Console Issues related to OpenTelemetry.Exporter.Console NuGet package
Projects
None yet
Development

No branches or pull requests

3 participants