Marc Alff
> I think that we can move kGauge at the end of the enum, to minimize binary compatibility issues and in practice avoid breaks for this case alone. Implemented as...
Please fix the CI for markdown-lint, will merge after that.
Reopen: the test is still unstable, even after using atomics. ``` [ RUN ] BasicCurlHttpTests.FinishInAsyncCallback ext/test/http/curl_http_test.cc:645: Failure Value of: handlers[i]->got_response_.load(std::memory_order_acquire) Actual: false Expected: true [ FAILED ] BasicCurlHttpTests.FinishInAsyncCallback (4 ms)...
Bonjour @olivier-perrault-ibm I am very puzzled by this part of the test failure: ``` /home/fyreuser/repos/ace-tracing/opentelemetry-cpp/sdk/test/trace/tracer_test.cc:1076: Failure Expected: (spans.at(0).get()->GetTraceId()) != (span_first->GetContext().trace_id()), actual: 16-byte object vs 16-byte object ``` According to this...
@dbarker See minor comment about preview options.
Deleted this branch, no longer used: revert-2600-ks/cpp-std (last updated 2024-03-21)
@santiagorr Thanks for volunteering on this. Nobody else is working on this that I know of, this issue is now assigned to you.
Starting with opentelemetry-cpp 1.18.0, this part: > Overall, the build failures looks like the compiler finds multiple abseil implementations, one found externally, and one found in opentelemetry-cpp/api/include/opentelemetry/nostd/internal/absl, which is meant...
Instead of providing a name for a resource detector, should a resource detector be an extension point instead ? This would be aligned with other parts of the configuration provided...