Revert "ref: upgrade datadog to a typed version (#54758)"

This reverts commit 3bfe952.

Co-authored-by: beezz <[email protected]>

Author: getsentry-bot

pyproject.toml

@@ -96,6 +96,7 @@ module = [
     "codeowners.*",
     "confluent_kafka.*",
     "cssselect.*",
+    "datadog.*",
     "django_zero_downtime_migrations.backends.postgres.schema.*",
     "docker.*",
     "email_reply_parser.*",

requirements-base.txt

@@ -9,7 +9,7 @@ click>=8.0.4
 confluent-kafka>=2.1.1
 croniter>=1.3.10
 cssselect>=1.0.3
-datadog>=0.46.0
+datadog>=0.29.3
 django-crispy-forms>=1.14.0
 django-csp>=3.7
 django-pg-zero-downtime-migrations>=0.13

requirements-dev-frozen.txt

@@ -35,7 +35,8 @@ croniter==1.3.10
 cryptography==39.0.1
 cssselect==1.0.3
 cssutils==2.4.0
-datadog==0.46.0
+datadog==0.29.3
+decorator==5.1.1
 dictpath==0.1.3
 distlib==0.3.4
 django==3.2.20

requirements-frozen.txt

@@ -30,7 +30,8 @@ croniter==1.3.10
 cryptography==39.0.1
 cssselect==1.0.3
 cssutils==2.4.0
-datadog==0.46.0
+datadog==0.29.3
+decorator==5.1.1
 django==3.2.20
 django-crispy-forms==1.14.0
 django-csp==3.7

src/sentry/metrics/datadog.py

@@ -2,8 +2,7 @@
 
 from typing import Any, Optional, Union
 
-from datadog import initialize
-from datadog.threadstats.base import ThreadStats
+from datadog import ThreadStats, initialize
 from datadog.util.hostname import get_hostname
 
 from sentry.utils.cache import memoize
@@ -18,7 +17,7 @@ def __init__(self, prefix: Optional[str] = None, **kwargs: Any) -> None:
         if "host" in kwargs:
             self.host = kwargs.pop("host")
         else:
-            self.host = get_hostname(hostname_from_config=True)
+            self.host = get_hostname()
         initialize(**kwargs)
         super().__init__(prefix=prefix)
 

src/sentry/metrics/dogstatsd.py

@@ -1,7 +1,6 @@
 from typing import Any, Optional, Union
 
-from datadog import initialize
-from datadog.dogstatsd.base import statsd
+from datadog import initialize, statsd
 
 from .base import MetricsBackend, Tags
 

tests/sentry/metrics/test_datadog.py

@@ -14,31 +14,19 @@ def setUp(self):
     def test_incr(self, mock_incr):
         self.backend.incr("foo", instance="bar")
         mock_incr.assert_called_once_with(
-            "sentrytest.foo",
-            1,
-            sample_rate=1,
-            tags=["instance:bar"],
-            host=get_hostname(hostname_from_config=True),
+            "sentrytest.foo", 1, sample_rate=1, tags=["instance:bar"], host=get_hostname()
         )
 
     @patch("datadog.threadstats.base.ThreadStats.timing")
     def test_timing(self, mock_timing):
         self.backend.timing("foo", 30, instance="bar")
         mock_timing.assert_called_once_with(
-            "sentrytest.foo",
-            30,
-            sample_rate=1,
-            tags=["instance:bar"],
-            host=get_hostname(hostname_from_config=True),
+            "sentrytest.foo", 30, sample_rate=1, tags=["instance:bar"], host=get_hostname()
         )
 
     @patch("datadog.threadstats.base.ThreadStats.gauge")
     def test_gauge(self, mock_gauge):
         self.backend.gauge("foo", 5, instance="bar")
         mock_gauge.assert_called_once_with(
-            "sentrytest.foo",
-            5,
-            sample_rate=1,
-            tags=["instance:bar"],
-            host=get_hostname(hostname_from_config=True),
+            "sentrytest.foo", 5, sample_rate=1, tags=["instance:bar"], host=get_hostname()
         )