Browse Source

ref: Use redis clients properly

Uses changes from matt's pr here: https://github.com/getsentry/sentry/pull/15377

And relies on having setting in https://github.com/getsentry/getsentry/pull/3293 merged
Dan Fuller 5 years ago
parent
commit
a5a5679331
1 changed files with 2 additions and 6 deletions
  1. 2 6
      src/sentry/incidents/subscription_processor.py

+ 2 - 6
src/sentry/incidents/subscription_processor.py

@@ -392,9 +392,5 @@ def update_alert_rule_stats(alert_rule, subscription, last_update, alert_counts,
 
 
 def get_redis_client():
-    cluster_key = getattr(settings, "SENTRY_INCIDENT_RULES_REDIS_CLUSTER", None)
-    if cluster_key is None:
-        client = redis.clusters.get("default").get_local_client(0)
-    else:
-        client = redis.redis_clusters.get(cluster_key)
-    return client
+    cluster_key = getattr(settings, "SENTRY_INCIDENT_RULES_REDIS_CLUSTER", "default")
+    return redis.redis_clusters.get(cluster_key)