Marc Alff

Results 165 comments of Marc Alff

Now that the following is done: * https://github.com/open-telemetry/opentelemetry-proto/pull/503 * https://github.com/open-telemetry/opentelemetry-proto/pull/523 can this item, W3C Trace Context Level 2, be adopted in the specifications ? Looking to implement it for opentelemetry-cpp.

> The Trace Context level 2 is in candidate recommendation phase and the W3C is actively looking for adopters. It would be very helpful to us (w3c working group) if...

Hi @keith Could you merge with a recent main to resolve conflicts, and investigate the remaining CI failures for bazel if any ? Thanks, -- Marc

opentelemetry-cpp needs to support readable / writable span data. Need to investigate how to implement this. Risk is a performance / flexibility tradeoff.

@GeorgViehoever If I understand correctly, you have a subclass of SpanProcessor, and want to: * inspect the span data * possibly modify it ? * possibly discard the Span when...

Per the github runner images: https://github.com/actions/runner-images/blob/ubuntu22/20240414.1/images/ubuntu/Ubuntu2204-Readme.md Available versions in Ubuntu ~24.04~ 22.04 are: `Clang-format: 13.0.1, 14.0.0, 15.0.7` I suggest to use `Clang-format 15`, as it will be available longer than...

> A quick look into the ubuntu24.04 [docker image ](https://hub.docker.com/_/ubuntu)(of dev-brach) shows below versions > > ``` > root@3d32d77d0cd2:/# cat /etc/os-release | grep VERSION > VERSION_ID="24.04" > VERSION="24.04 LTS (Noble...

Related: * https://github.com/actions/runner-images/issues/9691 We sure can wait for April 25, 2024, but will the runner be available in github then ? Most likely not.

Upstream unnecessary permission found, seen with github submodules: * https://github.com/open-telemetry/opentelemetry-proto/issues/540

This PR looks ok. The build failure on MacOS seems unrelated, and to fix separately.