Damien Mathieu

Results 259 comments of Damien Mathieu

otelgin is now deprecated due to a lack of owner: https://github.com/open-telemetry/opentelemetry-go-contrib/issues/6190 While we would be happy to get a new owner for this package, I'm not sure introducing breaking changes...

I have posted about this removal in the mongo community forum. https://www.mongodb.com/community/forums/t/otelmongo-at-risk-of-removal/282989

The [membership requirements](https://github.com/open-telemetry/community/blob/main/guides/contributor/membership.md#requirements) state that new members should have multiple contributions to the project. While Preston's contribution is great, and due to its size, probably counts more than a typo...

I'm also happy to sponsor you once the opt-in environment variable is added to support the latest semconv.

We recommend you to join the next SIG meeting so you can introduce yourself. You can find the date/hour and zoom link for that meeting in the shared calendar: https://github.com/open-telemetry/community/?tab=readme-ov-file#calendar...

Kind of nit, but do the file names need to be prepended with `zap_`? That's not something we're doing in other packages, and this package only holds the zap bridge....

Since this is a published module, we need to ensure we have a deprecated release of the old name, so folks using it now get a warning letting them know...

You can open it and keep it as draft until the release happens.

We just added [WithPublicEndpoint and WithPublicEndpointFn](https://github.com/open-telemetry/opentelemetry-go-contrib/pull/7407) that will set `WithNewRoot()` dynamically already. Every other attribute in the span can be modified with a custom interceptor/middleware within the app.