Skip to content
Snippets Groups Projects

Revert "Ignore noisy GroupCoordinator timeouts from SCiMMA"

Merged Deep Chatterjee requested to merge deep.chatterjee/gwcelery:revert-before-send into main

This reverts commit 1b8addc4.

So on emfollow test I can see that with this change new issues are getting logged in Sentry. The capture_message in the gwcelery shell is also getting logged. What is not logged is something like raise RuntimeError while in the shell. However doing the following logs it here though not with the intended behavior since the task is not registered

In [2]: @app.task
   ...: def raise_a_type_error(*args, **kwargs):
   ...:     raise TypeError("Raising a type error")
   ...: 

In [3]: r = raise_a_type_error.delay()

Merge request reports

Approval is optional

Merged by Leo Pound SingerLeo Pound Singer 2 years ago (Oct 18, 2022 1:36pm UTC)

Merge details

  • Changes merged into with 546eecac.
  • Did not delete the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading