Browse Source

Revert "perf(alerts): Add noop handler for load test subscriptions (#29428)" (#29987)

This reverts commit ce71b2b53badd2f426700968342ad6b1be1f99cd.
Ahmed Etefy 3 years ago
parent
commit
6fd2793e91
1 changed files with 0 additions and 9 deletions
  1. 0 9
      src/sentry/incidents/tasks.py

+ 0 - 9
src/sentry/incidents/tasks.py

@@ -24,7 +24,6 @@ from sentry.utils.http import absolute_uri
 logger = logging.getLogger(__name__)
 
 INCIDENTS_SNUBA_SUBSCRIPTION_TYPE = "incidents"
-SUBSCRIPTION_LOAD_TEST_SUBSCRIPTION_TYPE = "load_test_sessions_subscription"
 INCIDENT_SNAPSHOT_BATCH_SIZE = 50
 
 
@@ -115,14 +114,6 @@ def handle_snuba_query_update(subscription_update, subscription):
         SubscriptionProcessor(subscription).process_update(subscription_update)
 
 
-@register_subscriber(SUBSCRIPTION_LOAD_TEST_SUBSCRIPTION_TYPE)
-def handle_load_test_snuba_fake_query_update(subscription_update, subscription):
-    # ToDo(ahmed): Remove this handler. This is only added as part of a load test for crash rate
-    #  alerts. In this case, we don't really care about the actual subscriptions as much as the
-    #  load crash rate alerts will have on our infra, and so we just drop the updates here
-    metrics.incr("incidents.crash_rate_alerts.load_test")
-
-
 @instrumented_task(
     name="sentry.incidents.tasks.handle_trigger_action",
     queue="incidents",