Lalit Kumar Bhasin

Documenting LogManager Setup and Best Practices

1

Adding bit more guidance in our docs on setting up and managing loggers, especially since we've got those two paths: `LOGMANAGER_INSTANCE` for the straightforward cases and `LogManagerProvider` for when things...

Ref - https://github.com/microsoft/cpp_client_telemetry/pull/1249 To add two new PiiKind values: - IPv6ScrubLastHextets : A legacy IPV6 Internet address - DropValue: Drop the value. cc @aef123

How to override the DeviceInfo.OsVersion field from client app?

5

### Discussed in https://github.com/microsoft/cpp_client_telemetry/discussions/1092 Originally posted by **nishchith-cp** January 20, 2023 Need to set the `DeviceInfo.OsVersion` from the client side. Currently, the `DeviceInfo.OsVersion` is being populated within the SDK. We...

Ongoing PR, not yet for review. Included changes * Deprecate shared global state (the LogManager instance that is not really an instance). * Return unique_ptr for ILogManager.

Revisit shutdown_signal_provider methods at API level

3

I think we need to revisit `shutdown_*_provider` methods as they are at the API level. The instrumented libraries shouldn't have the method to shutdown the LoggerProvider enabled by the application....

Remove logs_level_enabled feature flag from Log API and SDK

4

Tracking issue to remove `logs_level_enabled` flag introduced in #1147 from Logs API and SDK once the relevant specs PR is merged - https://github.com/open-telemetry/opentelemetry-specification/pull/4020

## Release plan for Logs The components in scope are: - Logs bridge API (`opentelemetry` crate) - Logs SDK (`opentelemetry-sdk` crate) - Logs InMemory Exporter (`opentelemetry-sdk` crate) - Logs OTLP...

[For discussion] Support LogProcessors chaining through mutable reference

4

Fixes #1650 As discussed [here](https://github.com/open-telemetry/opentelemetry-rust/pull/1636#discussion_r1576580577), this PR modifies LogProcessor to take mutable reference to LogData, and further pass it as Owned/Borrowed type to the Exporter. **Existing**: ```rust pub trait LogProcessor:...

[WIP] Suppression of nested logs from dependencies

1

Raising this PR as alternate approach to fix #1171, by extending the Context structure to store the suppression flag. The PR is to discuss this approach. The earlier approach #1315...

Investigate Potential to bypass LogRecord cloning when emitting LogRecord to LogProcessor(s)

3

https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-sdk/src/logs/log_emitter.rs#L210 This clone needs some rethinking. Currently, every processor, even the ones that can stream event directly will have to suffer from this clone cost. I think we should just...