feat(release-health): Sessions should not be default (#67353)

Sessions does not even exist anymore, the metrics pipeline should be
used for release health. Flip the default as all environments have been
cut over.

Author: lynnagara

src/sentry/conf/server.py

@@ -2332,13 +2332,11 @@ def custom_parameter_sort(parameter: dict) -> tuple[str, int]:
 SENTRY_METRICS_INDEXER_ENABLE_SLICED_PRODUCER = False
 
 # Release Health
-SENTRY_RELEASE_HEALTH = "sentry.release_health.sessions.SessionsReleaseHealthBackend"
+SENTRY_RELEASE_HEALTH = "sentry.release_health.metrics.MetricsReleaseHealthBackend"
 SENTRY_RELEASE_HEALTH_OPTIONS: dict[str, Any] = {}
 
 # Release Monitor
-SENTRY_RELEASE_MONITOR = (
-    "sentry.release_health.release_monitor.sessions.SessionReleaseMonitorBackend"
-)
+SENTRY_RELEASE_MONITOR = "sentry.release_health.release_monitor.metrics.MetricReleaseMonitorBackend"
 SENTRY_RELEASE_MONITOR_OPTIONS: dict[str, Any] = {}
 
 # Render charts on the backend. This uses the Chartcuterie external service.

src/sentry/release_health/metrics.py

@@ -5,7 +5,7 @@
 from typing import Any, Literal, TypeVar
 
 from snuba_sdk import Column, Condition, Direction, Op
-from snuba_sdk.expressions import Granularity, Limit
+from snuba_sdk.expressions import Granularity, Limit, Offset
 
 from sentry.models.environment import Environment
 from sentry.models.project import Project
@@ -1716,6 +1716,7 @@ def get_project_releases_by_stability(
             orderby=orderby,
             groupby=groupby,
             granularity=Granularity(LEGACY_SESSIONS_DEFAULT_ROLLUP),
+            offset=Offset(offset) if offset is not None else None,
             limit=Limit(limit) if limit is not None else None,
             include_series=False,
             include_totals=True,

src/sentry/testutils/pytest/metrics.py

@@ -66,7 +66,7 @@ def new_build_results(*args, **kwargs):
             is_performance_metrics = False
             is_metrics = False
             if not isinstance(query, MetricsQuery) and not isinstance(query.match, Join):
-                is_performance_metrics = query.match.name.startswith("generic_metrics")
+                is_performance_metrics = query.match.name.startswith("generic")
                 is_metrics = "metrics" in query.match.name
 
             if is_performance_metrics:
@@ -85,7 +85,7 @@ def new_create_snql_in_snuba(subscription, snuba_query, snql_query, entity_subsc
             is_performance_metrics = False
             is_metrics = False
             if isinstance(query.match, Entity):
-                is_performance_metrics = query.match.name.startswith("generic_metrics")
+                is_performance_metrics = query.match.name.startswith("generic")
                 is_metrics = "metrics" in query.match.name
 
             if is_performance_metrics:

tests/acceptance/test_organization_dashboards.py

@@ -33,6 +33,9 @@
 EDIT_FEATURE = ["organizations:dashboards-edit"]
 
 
+pytestmark = pytest.mark.sentry_metrics
+
+
 @no_silo_test
 class OrganizationDashboardsAcceptanceTest(AcceptanceTestCase):
     def setUp(self):

tests/acceptance/test_organization_switch.py

@@ -1,10 +1,13 @@
+import pytest
 from selenium.common.exceptions import TimeoutException
 from selenium.webdriver.common.by import By
 
 from sentry.testutils.cases import AcceptanceTestCase, SnubaTestCase
 from sentry.testutils.silo import no_silo_test
 from sentry.utils.retries import TimedRetryPolicy
 
+pytestmark = pytest.mark.sentry_metrics
+
 
 @no_silo_test
 class OrganizationSwitchTest(AcceptanceTestCase, SnubaTestCase):

tests/apidocs/endpoints/releases/test_organization_releases.py

@@ -1,11 +1,14 @@
 from datetime import UTC, datetime
 
+import pytest
 from django.test.client import RequestFactory
 from django.urls import reverse
 
 from fixtures.apidocs_test_case import APIDocsTestCase
 from sentry.models.release import Release
 
+pytestmark = pytest.mark.sentry_metrics
+
 
 class OrganizationReleasesDocsTest(APIDocsTestCase):
     def setUp(self):

tests/apidocs/endpoints/releases/test_organization_sessions.py

@@ -1,9 +1,12 @@
+import pytest
 from django.test.client import RequestFactory
 from django.urls import reverse
 
 from fixtures.apidocs_test_case import APIDocsTestCase
 from sentry.testutils.cases import SnubaTestCase
 
+pytestmark = pytest.mark.sentry_metrics
+
 
 class OrganizationSessionsDocsTest(APIDocsTestCase, SnubaTestCase):
     def setUp(self):

tests/sentry/api/endpoints/test_organization_release_details.py

@@ -2,6 +2,7 @@
 from datetime import UTC, datetime, timedelta
 from unittest.mock import patch
 
+import pytest
 from django.urls import reverse
 
 from sentry.api.endpoints.organization_release_details import OrganizationReleaseSerializer
@@ -24,7 +25,7 @@
 from sentry.types.activity import ActivityType
 from sentry.utils.security.orgauthtoken_token import generate_token, hash_token
 
-pytestmark = [requires_snuba]
+pytestmark = [pytest.mark.sentry_metrics, requires_snuba]
 
 
 class ReleaseDetailsTest(APITestCase):

tests/sentry/api/endpoints/test_organization_releases.py

@@ -3,6 +3,7 @@
 from functools import cached_property
 from unittest.mock import patch
 
+import pytest
 from django.urls import reverse
 from django.utils import timezone
 
@@ -36,9 +37,9 @@
 from sentry.silo import SiloMode
 from sentry.testutils.cases import (
     APITestCase,
+    BaseMetricsTestCase,
     ReleaseCommitPatchTest,
     SetRefsTestCase,
-    SnubaTestCase,
     TestCase,
 )
 from sentry.testutils.outbox import outbox_runner
@@ -47,10 +48,10 @@
 from sentry.types.activity import ActivityType
 from sentry.utils.security.orgauthtoken_token import generate_token, hash_token
 
-pytestmark = [requires_snuba]
+pytestmark = [requires_snuba, pytest.mark.sentry_metrics]
 
 
-class OrganizationReleaseListTest(APITestCase, SnubaTestCase):
+class OrganizationReleaseListTest(APITestCase, BaseMetricsTestCase):
     endpoint = "sentry-api-0-organization-releases"
 
     def assert_expected_versions(self, response, expected):

tests/sentry/api/endpoints/test_project_release_details.py

@@ -1,6 +1,7 @@
 import unittest
 from datetime import UTC, datetime
 
+import pytest
 from django.urls import reverse
 
 from sentry.api.serializers.rest_framework.release import ReleaseSerializer
@@ -16,7 +17,7 @@
 from sentry.types.activity import ActivityType
 from sentry.utils.security.orgauthtoken_token import generate_token, hash_token
 
-pytestmark = [requires_snuba]
+pytestmark = [pytest.mark.sentry_metrics, requires_snuba]
 
 
 class ReleaseDetailsTest(APITestCase):