fix: except LockRetry instead of LockError

[joseph-sentry]

Fixes: https://codecov.sentry.io/issues/5057432400/?project=-1&query=is%3Aunresolved&referrer=issue-stream&statsPeriod=24h&stream_index=1&utc=true

We should be excepting a LockRetry here because that is what LockManager throws when it hits an error when trying to acquire the lock

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: tasks/notify.py

Function	Unhandled Issue
run_impl	LockRetry: 143 app.tasks.notify.Notify Event Count: 1.0k

---

_{Did you find this useful? React with a 👍 or 👎}

[codecov-notifications]

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found ☺️

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         393      393           
  Lines       32900    32901    +1     
=======================================
+ Hits        32094    32095    +1     
  Misses        806      806           

Flag	Coverage Δ
integration	97.55% <100.00%> (+<0.01%)	⬆️
latest-uploader-overall	97.55% <100.00%> (+<0.01%)	⬆️
unit	97.55% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.91% <100.00%> (ø)
OutsideTasks	97.78% <ø> (ø)

Files	Coverage Δ
tasks/notify.py	98.50% <100.00%> (ø)
tasks/tests/unit/test_notify_task.py	100.00% <100.00%> (ø)

[codecov-qa]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.55%. Comparing base (7d9c198) to head (65eb83f).

✅ All tests successful. No failed tests found ☺️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         393      393           
  Lines       32900    32901    +1     
=======================================
+ Hits        32094    32095    +1     
  Misses        806      806           

Flag	Coverage Δ
integration	97.55% <100.00%> (+<0.01%)	⬆️
latest-uploader-overall	97.55% <100.00%> (+<0.01%)	⬆️
unit	97.55% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.91% <100.00%> (ø)
OutsideTasks	97.78% <ø> (ø)

Files	Coverage Δ
tasks/notify.py	98.50% <100.00%> (ø)
tasks/tests/unit/test_notify_task.py	100.00% <100.00%> (ø)

[codecov-public-qa]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.55%. Comparing base (7d9c198) to head (65eb83f).

✅ All tests successful. No failed tests found ☺️

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   97.55%   97.55%           
=======================================
  Files         393      393           
  Lines       32900    32901    +1     
=======================================
+ Hits        32094    32095    +1     
  Misses        806      806           

Flag	Coverage Δ
integration	97.55% <100.00%> (+<0.01%)	⬆️
latest-uploader-overall	97.55% <100.00%> (+<0.01%)	⬆️
unit	97.55% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.91% <100.00%> (ø)
OutsideTasks	97.78% <ø> (ø)

Files	Coverage Δ
tasks/notify.py	98.50% <100.00%> (ø)
tasks/tests/unit/test_notify_task.py	100.00% <100.00%> (ø)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.56%. Comparing base (7d9c198) to head (65eb83f).

Changes have been made to critical files, which contain lines commonly executed in production. Learn more

✅ All tests successful. No failed tests found ☺️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #372   +/-   ##
=======================================
  Coverage   97.56%   97.56%           
=======================================
  Files         424      424           
  Lines       33600    33601    +1     
=======================================
+ Hits        32782    32783    +1     
  Misses        818      818           

Flag	Coverage Δ
integration	97.55% <100.00%> (+<0.01%)	⬆️
latest-uploader-overall	97.55% <100.00%> (+<0.01%)	⬆️
unit	97.55% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
NonTestCode	94.91% <100.00%> (ø)
OutsideTasks	97.78% <ø> (ø)

Files	Coverage Δ
tasks/notify.py Critical	98.50% <100.00%> (ø)
tasks/tests/unit/test_notify_task.py	100.00% <100.00%> (ø)

Related Entrypoints
run/app.tasks.notify.Notify

[giovanni-guidini]

❤️