Damien Mathieu

Results 562 comments of Damien Mathieu

Two alternatives could be: ## Introduce a `OpenTelemetry::SDK::DefaultProcessors` array that returns all the default processors. Then, you could do something along the lines of: ```ruby ::OpenTelemetry::SDK.configure do |c| OpenTelemetry::SDK::DefaultProcessors.each do...

Looking again at the [configuration specification](https://github.com/open-telemetry/opentelemetry-specification/tree/main/specification/configuration), there is definitely no mention of manual configuration preventing the additional use of configuration variables. So while it's a breaking change, we could change...

From a quick look: IMHO, the command would be better off in `internal/cmd`. This is not needed as a public tool. Also, your PR currently has the compiled binary committed....

The failures are due to a breaking change with contrib. They need to be fixed in another PR.

> We ended up forking and patching the library and all has been fine since, but preferably we'd use public releases. If you have a fix for this, maybe open...

`WithRouteTag` can probably be deprecated and removed. See https://cloud-native.slack.com/archives/C01NPAXACKT/p1761999249564689

> If this is locking in the api, is that meant as a resolution for #2782 ? This definitely does not resolve that issue. However, making a non-generalist decision at...

See https://github.com/open-telemetry/opentelemetry-go-contrib/issues/7254#issuecomment-3480563031 which recommends stabilization of https://github.com/open-telemetry/semantic-conventions/blob/main/docs/general/recording-errors.md#recording-errors-on-spans as a blocker.

I have created a project to track this, and have started creating issues for the things that can be tackled now. https://github.com/orgs/open-telemetry/projects/173

Something similar is already being done by instrumentation libraries. For example, the `WithPublicEndpoint` option in Go: https://github.com/open-telemetry/opentelemetry-go-contrib/blob/a91e60beace7c1d96302fe359050d705c4723e72/instrumentation/net/http/otelhttp/handler.go#L140-L146 Ruby has this proposal open for the Rack instrumentation: https://github.com/open-telemetry/opentelemetry-ruby-contrib/pull/837 It seems your...