Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.57%. Comparing base (5dcda1d) to head (02380b7).
Report is 30 commits behind head on master.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##           master    #4159   +/-   ##
=======================================
  Coverage   79.57%   79.57%           
=======================================
  Files         141      141           
  Lines       15711    15711           
  Branches     2671     2671           
=======================================
  Hits        12502    12502           
  Misses       2367     2367           
  Partials      842      842           

what is the actual problem and how does this fix it?

This is an experiment.

The problem described in #3753 is roughly like this:

• DataDog is monitoring lock operations
• Sentry's worker thread needs to acquire a lock
• When lock operations occur, DataDog tries to log information
• Logging triggers Sentry's logging integration
• Sentry tries to capture this event
• Capturing an event requires Sentry's worker thread
• The worker thread needs to acquire a lock...

By changing to RLock the given reproduction does not run into a deadlock anymore. But I am not 100% sure that this does not lead to other problems. (I think it does not, but I am not sure)

I wanted to see if any of our test start failing if using RLock.

We are not going to merge this, because it seems an updated version of ddtrace fixes this.