fixup! Introduce DD traces & metrics

permitio · Feb 28, 2024 · 9bd4dd1 · 9bd4dd1
1 parent 0a2068c
commit 9bd4dd1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/packages/opal-common/opal_common/monitoring/apm.py b/packages/opal-common/opal_common/monitoring/apm.py
@@ -26,7 +26,13 @@ def process_trace(self, trace: list[Span]) -> Optional[list[Span]]:
 
                 return trace
 
-        patch(fastapi=True, redis=True, asyncpg=True, aiohttp=True, celery=True)
+        patch(
+            fastapi=True,
+            redis=True,
+            asyncpg=True,
+            aiohttp=True,
+            loguru=True,
+        )
         tracer.configure(
             settings={
                 "FILTERS": [

diff --git a/packages/opal-common/requires.txt b/packages/opal-common/requires.txt
@@ -6,5 +6,5 @@ loguru>=0.6.0,<1
 pyjwt[crypto]>=2.4.0,<3
 python-decouple>=3.6,<4
 tenacity>=8.0.1,<9
-datadog >= 0.44.0, <1
-ddtrace >= 1.6.4,<2
+datadog>=0.44.0, <1
+ddtrace>=2,<3
diff --git a/packages/opal-server/requires.txt b/packages/opal-server/requires.txt
@@ -3,7 +3,6 @@ permit-broadcaster[postgres,redis,kafka]==0.2.5
 gitpython>=3.1.32,<4
 pyjwt[crypto]>=2.1.0,<3
 websockets>=10.3,<11
-ddtrace>=1.1.4,<2
 slowapi>=0.1.5,<1
 # slowapi is stuck on and old `redis`, so fix that and switch from aioredis to redis
 pygit2>=1.13.3,<2