Merge pull request #1545 from maykinmedia/swr/bump-sentry-sdk

Bump Sentry to latest version
maykinmedia · Jan 9, 2025 · 4b86009 · 4b86009
2 parents 27bc304 + 31045d1
commit 4b86009
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 5 deletions.
diff --git a/requirements/base.in b/requirements/base.in
@@ -73,7 +73,7 @@ djangorestframework-camel-case
 
 # WSGI servers & monitoring - production oriented
 uwsgi
-sentry-sdk  # error monitoring
+sentry-sdk[django]  # error monitoring
 elastic-apm  # Elastic APM integration
 
 beautifulsoup4

diff --git a/requirements/base.txt b/requirements/base.txt
@@ -146,6 +146,7 @@ django==4.2.16
     #   mozilla-django-oidc
     #   mozilla-django-oidc-db
     #   notifications-api-common
+    #   sentry-sdk
     #   zgw-consumers
     #   zgw-consumers-oas
 django-admin-index==3.1.0
@@ -523,7 +524,7 @@ requests==2.31.0
     #   messagebird
     #   mozilla-django-oidc
     #   zgw-consumers
-sentry-sdk==1.38.0
+sentry-sdk[django]==2.19.2
     # via -r requirements/base.in
 simplejson==3.19.2
     # via mail-parser

diff --git a/requirements/ci.txt b/requirements/ci.txt
@@ -238,6 +238,7 @@ django==4.2.16
     #   mozilla-django-oidc
     #   mozilla-django-oidc-db
     #   notifications-api-common
+    #   sentry-sdk
     #   zgw-consumers
     #   zgw-consumers-oas
 django-admin-index==3.1.0
@@ -992,10 +993,11 @@ requests==2.31.0
     #   zgw-consumers
 requests-mock==1.12.1
     # via -r requirements/test-tools.in
-sentry-sdk==1.38.0
+sentry-sdk[django]==2.19.2
     # via
     #   -c requirements/base.txt
     #   -r requirements/base.txt
+    #   sentry-sdk
 simplejson==3.19.2
     # via
     #   -c requirements/base.txt

diff --git a/requirements/dev.txt b/requirements/dev.txt
@@ -278,6 +278,7 @@ django==4.2.16
     #   mozilla-django-oidc
     #   mozilla-django-oidc-db
     #   notifications-api-common
+    #   sentry-sdk
     #   zgw-consumers
     #   zgw-consumers-oas
 django-admin-index==3.1.0
@@ -1148,10 +1149,11 @@ requests-mock==1.12.1
     #   -r requirements/ci.txt
 roundrobin==0.0.4
     # via locust
-sentry-sdk==1.38.0
+sentry-sdk[django]==2.19.2
     # via
     #   -c requirements/ci.txt
     #   -r requirements/ci.txt
+    #   sentry-sdk
 simplejson==3.19.2
     # via
     #   -c requirements/ci.txt

diff --git a/src/open_inwoner/celery.py b/src/open_inwoner/celery.py
@@ -70,3 +70,9 @@ def worker_shutdown(**_):
 
 
 app.steps["worker"].add(LivenessProbe)
+
+
+@app.task
+def trigger_exception():
+    """Trigger an exception for debugging purposes."""
+    return 1 / 0
diff --git a/src/open_inwoner/conf/utils.py b/src/open_inwoner/conf/utils.py
@@ -46,7 +46,9 @@ def get_sentry_integrations() -> list:
     try:
         from sentry_sdk.integrations import celery
     except DidNotEnable:  # happens if the celery import fails by the integration
-        pass
+        logger.warning(
+            "Unable to initialize Sentry with Celery integration due to failing import"
+        )
     else:
         extra.append(celery.CeleryIntegration())