From 2814a127a6308a1c3170870ee5cd9ac5bdd367bc Mon Sep 17 00:00:00 2001 From: anthony sottile <103459774+asottile-sentry@users.noreply.github.com> Date: Fri, 12 Jan 2024 09:57:06 -0500 Subject: [PATCH] ref: replace memcached client with vendored copy (#62755) it is removed in django 4.2 --- self-hosted/sentry.conf.py | 2 +- src/sentry/data/config/sentry.conf.py.default | 2 +- src/sentry/monkey/__init__.py | 16 ---------------- src/sentry/utils/settings.py | 2 +- tests/sentry/utils/test_settings.py | 2 +- 5 files changed, 4 insertions(+), 20 deletions(-) diff --git a/self-hosted/sentry.conf.py b/self-hosted/sentry.conf.py index 470c3d4ef02d03..12d636d7e0dc46 100644 --- a/self-hosted/sentry.conf.py +++ b/self-hosted/sentry.conf.py @@ -115,7 +115,7 @@ memcached_port = env("SENTRY_MEMCACHED_PORT") or "11211" CACHES = { "default": { - "BACKEND": "django.core.cache.backends.memcached.MemcachedCache", + "BACKEND": "sentry.utils.memcached.MemcachedCache", "LOCATION": [memcached + ":" + memcached_port], "TIMEOUT": 3600, } diff --git a/src/sentry/data/config/sentry.conf.py.default b/src/sentry/data/config/sentry.conf.py.default index fb1473f7a6ffef..f1a29e8e1fe452 100644 --- a/src/sentry/data/config/sentry.conf.py.default +++ b/src/sentry/data/config/sentry.conf.py.default @@ -46,7 +46,7 @@ DEBUG = %(debug_flag)s # # CACHES = { # 'default': { -# 'BACKEND': 'django.core.cache.backends.memcached.MemcachedCache', +# 'BACKEND': 'sentry.utils.memcached.MemcachedCache', # 'LOCATION': ['127.0.0.1:11211'], # } # } diff --git a/src/sentry/monkey/__init__.py b/src/sentry/monkey/__init__.py index f554f26aa787d2..3f1ab2cb25b7f8 100644 --- a/src/sentry/monkey/__init__.py +++ b/src/sentry/monkey/__init__.py @@ -25,21 +25,5 @@ def patch_celery_imgcat(): term.imgcat = lambda *a, **kw: b"" -def patch_memcached(): - # Fixes a bug in Django 3.2 - try: - from django.core.cache.backends.memcached import MemcachedCache - except ImportError: - return - - def fixed_delete(self, key, version=None): - key = self.make_key(key, version=version) - self.validate_key(key) - return bool(self._cache.delete(key)) - - MemcachedCache.delete = fixed_delete # type: ignore[method-assign] - - patch_celery_imgcat() patch_pickle_loaders() -patch_memcached() diff --git a/src/sentry/utils/settings.py b/src/sentry/utils/settings.py index dffb5ac1c5150c..e4b0a0c3f263b5 100644 --- a/src/sentry/utils/settings.py +++ b/src/sentry/utils/settings.py @@ -3,7 +3,7 @@ PACKAGES = { "django.db.backends.postgresql_psycopg2": "psycopg2.extensions", "sentry.db.postgres": "psycopg2.extensions", - "django.core.cache.backends.memcached.MemcachedCache": "memcache", + "sentry.utils.memcached.MemcachedCache": "memcache", "django.core.cache.backends.memcached.PyLibMCCache": "pylibmc", } diff --git a/tests/sentry/utils/test_settings.py b/tests/sentry/utils/test_settings.py index 1016b54aec8e6b..b7f42439692373 100644 --- a/tests/sentry/utils/test_settings.py +++ b/tests/sentry/utils/test_settings.py @@ -31,7 +31,7 @@ "django.core.cache.backends.memcached.MemcachedCache", { "default": { - "BACKEND": "django.core.cache.backends.memcached.MemcachedCache", + "BACKEND": "sentry.utils.memcached.MemcachedCache", "LOCATION": "127.0.0.1:11211", } },