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

[Discuss] Enable OpenTelemetryBridgeEnabled by default. #818

Open
Mpdreamz opened this issue Jul 18, 2023 · 1 comment
Open

[Discuss] Enable OpenTelemetryBridgeEnabled by default. #818

Mpdreamz opened this issue Jul 18, 2023 · 1 comment

Comments

@Mpdreamz
Copy link
Member

Today the default is false:

https://www.elastic.co/guide/en/apm/agent/dotnet/current/config-core.html#config-opentelemetry-bridge-enabled

I am coming up short to find downsides of enabling this by default very keen to learn if others know of any.

@gregkalapos
Copy link
Contributor

This is also planned for Node.js: elastic/apm-agent-nodejs#3447 (the issue is about making it GA, but as part of it, we'd also make this config true by default).

I think there is no known downsides - so I'd just go for it.

Also: most agents work a bit differently, they basically inject their own tracer, and don't even have this config. I think this config is only relevant for .NET and Node.js.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants