awslabs / amazon-kinesis-video-streams-pic

Apache License 2.0
49 stars 47 forks source link

Enable tsan #236

Closed hassanctech closed 8 months ago

hassanctech commented 8 months ago

Issue #, if available:

Description of changes:

Enable TSAN in CI. Resolve some TSAN failures / cleanup. Fix test which was failing TSAN due to bad test setup.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

codecov-commenter commented 8 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b067814) 78.34% compared to head (0fee37e) 78.34%.

:exclamation: Current head 0fee37e differs from pull request most recent head 58be03b. Consider uploading reports for the commit 58be03b to get more accurate results

Files Patch % Lines
src/utils/src/Threadpool.c 71.42% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #236 +/- ## ======================================== Coverage 78.34% 78.34% ======================================== Files 52 52 Lines 10066 10066 ======================================== Hits 7886 7886 Misses 2180 2180 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.