stormshield-fabs

[`set_parent`](https://docs.rs/tracing-opentelemetry/latest/tracing_opentelemetry/trait.OpenTelemetrySpanExt.html#method.set_parent) is a `tracing-opentelemetry` method that does not interact with the `opentelemetry` structures the way you expect but instead on its own [`OtelData`](https://docs.rs/tracing-opentelemetry/latest/tracing_opentelemetry/struct.OtelData.html). Basically, you're dealing with two instrumentation APIs:...

The problem you're facing is related to inter-process trace propagation (see also [OTel Context Propagation](https://opentelemetry.io/docs/concepts/context-propagation)): when the second instance of the program starts, it has no idea the caller process...

It think the first option is better, especially considering the "self-contained" concern you raised. We could have it take a reference to the `TracerProvider`, avoiding the `Clone` problem. ```rust #[derive(Debug)]...

For context, https://github.com/open-telemetry/opentelemetry-rust/pull/1578 added support for percent-encoded `OTEL_EXPORTER*` environment variables. It was proposed to remove the `urlencoding` crate that indicated it's [looking for a maintainer](https://github.com/kornelski/rust_urlencoding/commit/3cc277fc4e577a0bef99294d7ab9bc85232f0730). The specification states that "these...

From the [`Layer::event_enabled` documentation](https://docs.rs/tracing-subscriber/latest/tracing_subscriber/layer/trait.Layer.html#method.event_enabled): > Layers which do not wish to be notified about certain events but do not wish to globally disable them should ignore those events in their...

Note that for events, [`tracing` defaults to a `target` like `"event file:line"`](https://docs.rs/tracing/latest/tracing/#configuring-attributes), which does not really fit the OTel semantics for `instrumentation_scope.name`.

The CLA is still under review at my company, thanks for the heads up.

The CLA has been signed. I've rebased this PR and updated the changelog.

There were some concerns about having `Arc` in the `library_signal` APIs, but someone also mentioned we want to avoid unnecessary copies. We could hide the `Arc` inside the `InstrumentationLibrary` if...

I've addressed the pending comments and added changelog entries (feel free to edit them!)