opentelemetry-cpp icon indicating copy to clipboard operation
opentelemetry-cpp copied to clipboard

[TEST] Unit test BasicCurlHttpTests.SendGetRequestAsync is unstable

Open marcalff opened this issue 6 months ago • 2 comments

This test fails often in CI.

Failures are spurious (not reproducible).

[ RUN      ] BasicCurlHttpTests.SendGetRequestAsync
ext/test/http/curl_http_test.cc:531: Failure
Value of: handlers[i]->got_response_.load(std::memory_order_acquire)
  Actual: false
Expected: true

[  FAILED  ] BasicCurlHttpTests.SendGetRequestAsync (2 ms)

marcalff avatar Jul 11 '25 20:07 marcalff

This issue was marked as stale due to lack of activity.

github-actions[bot] avatar Sep 10 '25 02:09 github-actions[bot]

Seen again many times, latest today 2025-10-27:

[ 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

This test needs to be fixed (preferable), or at least disabled, since this blocks CI with required builds.

marcalff avatar Oct 27 '25 16:10 marcalff