Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Conditional variable cv_got_events in otlp_http_exporter_test is never notified #857

Closed
ThomsonTan opened this issue Jun 15, 2021 · 0 comments · Fixed by #858
Closed

Conditional variable cv_got_events in otlp_http_exporter_test is never notified #857

ThomsonTan opened this issue Jun 15, 2021 · 0 comments · Fixed by #858
Assignees
@ThomsonTan
Labels
bug Something isn't working

Comments

@ThomsonTan
Copy link
Contributor

Describe your environment
otlp_http_exporter_test use conditional variable to sync between main thread and HTTP response handling thread, but the conditional variable is never notified even when the HTTP response is completed processing.

Steps to reproduce
Run exporter.otlp.OtlpHttpExporterTestPeer.ExportJsonIntegrationTest.

What is the expected behavior?
The test should pass shortly.

What is the actual behavior?
The test passed, but it also causes timeout.

Additional context
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ThomsonTan ThomsonTan

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix missing notify of conditional variable in OTLP HTTP exporter test ThomsonTan/opentelemetry-cpp
1 participant
@ThomsonTan