Merge pull request #814 from AlmaLinux/fix-sentry-false-positive

Fix Sentry constantly reporting errors when there are none
AlmaLinux · May 2, 2024 · fa993f7 · fa993f7
2 parents e135498 + 886482c
commit fa993f7
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 2 deletions.
diff --git a/alws/app.py b/alws/app.py
@@ -3,6 +3,7 @@
 
 import sentry_sdk
 from fastapi import FastAPI
+from pika.exceptions import StreamLostError
 from starlette.middleware.exceptions import ExceptionMiddleware
 
 from alws import routers
@@ -29,6 +30,10 @@
         dsn=settings.sentry_dsn,
         traces_sample_rate=settings.sentry_traces_sample_rate,
         environment=settings.sentry_environment,
+        ignore_errors=[
+            ConnectionResetError,
+            StreamLostError,
+        ],
     )
 
 

diff --git a/alws/schemas/build_schema.py b/alws/schemas/build_schema.py
@@ -312,7 +312,7 @@ async def get_module_data_from_beholder(
     try:
         beholder_response = await beholder_client.get(endpoint)
     except Exception:
-        logging.error('Cannot get module info')
+        logging.warning('Cannot get module info')
         return result
     result['devel'] = devel
     result['arch'] = arch

diff --git a/alws/utils/beholder_client.py b/alws/utils/beholder_client.py
@@ -61,7 +61,7 @@ async def iter_endpoints(
                     coro = self.post(endpoint, data)
                 yield await coro
             except Exception:
-                logging.error(
+                logging.warning(
                     "Cannot retrieve beholder info, "
                     "trying next reference platform"
                 )