open-telemetry / open-telemetry/opentelemetry-cpp
[TEST] Unit test BasicCurlHttpTests.SendGetRequestAsync is unstable
Open
Nobody has claimed this yet.
bug
Stale
triage/accepted
- Dominant language
- C++
- Stars
- 1.4k
- Forks
- 632
- Avg merge
- 1d 13h
- Merged PRs (30d)
- 75
Description
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)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with BasicCurlHttpTests.SendGetRequestAsync in ext/test/http/curl_http_test.cc at line 531 and run the test repeatedly to observe the intermittent failure. Trace how the test determines that each handler received a response; done means the test no longer reports spurious failures in CI.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- networking, testing-qa
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100