feat(aci): Re-add table to look up groups created by detectors (#95243)

redo of #94972

This table will be used to look up groups that were created by a
detector to display on the monitors page in the UI.

Author: snigdhas

migrations_lockfile.txt

@@ -35,4 +35,4 @@ tempest: 0001_squashed_0002_make_message_type_nullable
 
 uptime: 0001_squashed_0042_extra_uptime_indexes
 
-workflow_engine: 0075_add_index_to_dcg_action
+workflow_engine: 0076_add_detector_group_table

src/sentry/workflow_engine/migrations/0076_add_detector_group_table.py

@@ -0,0 +1,69 @@
+# Generated by Django 5.2.1 on 2025-07-09 17:43
+
+import django.db.models.deletion
+from django.db import migrations, models
+
+import sentry.db.models.fields.bounded
+import sentry.db.models.fields.foreignkey
+from sentry.new_migrations.migrations import CheckedMigration
+from sentry.new_migrations.monkey.special import SafeRunSQL
+
+
+class Migration(CheckedMigration):
+    # This flag is used to mark that a migration shouldn't be automatically run in production.
+    # This should only be used for operations where it's safe to run the migration after your
+    # code has deployed. So this should not be used for most operations that alter the schema
+    # of a table.
+    # Here are some things that make sense to mark as post deployment:
+    # - Large data migrations. Typically we want these to be run manually so that they can be
+    #   monitored and not block the deploy for a long period of time while they run.
+    # - Adding indexes to large tables. Since this can take a long time, we'd generally prefer to
+    #   run this outside deployments so that we don't block them. Note that while adding an index
+    #   is a schema change, it's completely safe to run the operation after the code has deployed.
+    # Once deployed, run these manually via: https://develop.sentry.dev/database-migrations/#migration-deployment
+
+    is_post_deployment = False
+
+    dependencies = [
+        ("sentry", "0943_create_data_access_grant"),
+        ("workflow_engine", "0075_add_index_to_dcg_action"),
+    ]
+
+    operations = [
+        SafeRunSQL(
+            """DROP TABLE IF EXISTS "workflow_engine_detectorgroup";""",
+            reverse_sql=migrations.RunSQL.noop,
+            hints={"tables": ["workflow_engine_detectorgroup"]},
+        ),  # this migration was successfully run in S4S and DE, failed in US
+        migrations.CreateModel(
+            name="DetectorGroup",
+            fields=[
+                (
+                    "id",
+                    sentry.db.models.fields.bounded.BoundedBigAutoField(
+                        primary_key=True, serialize=False
+                    ),
+                ),
+                ("date_updated", models.DateTimeField(auto_now=True)),
+                ("date_added", models.DateTimeField(auto_now_add=True)),
+                (
+                    "detector",
+                    sentry.db.models.fields.foreignkey.FlexibleForeignKey(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        to="workflow_engine.detector",
+                    ),
+                ),
+                (
+                    "group",
+                    sentry.db.models.fields.foreignkey.FlexibleForeignKey(
+                        db_constraint=False,
+                        on_delete=django.db.models.deletion.CASCADE,
+                        to="sentry.group",
+                    ),
+                ),
+            ],
+            options={
+                "db_table": "workflow_engine_detectorgroup",
+            },
+        ),
+    ]

src/sentry/workflow_engine/models/__init__.py

@@ -12,6 +12,7 @@
     "DataSource",
     "DataSourceDetector",
     "Detector",
+    "DetectorGroup",
     "DetectorState",
     "DetectorWorkflow",
     "IncidentGroupOpenPeriod",
@@ -32,6 +33,7 @@
 from .data_source_detector import DataSourceDetector
 from .datacondition_alertruletrigger import DataConditionAlertRuleTrigger
 from .detector import Detector
+from .detector_group import DetectorGroup
 from .detector_state import DetectorState
 from .detector_workflow import DetectorWorkflow
 from .incident_groupopenperiod import IncidentGroupOpenPeriod

src/sentry/workflow_engine/models/detector_group.py

@@ -0,0 +1,20 @@
+from django.db import models
+
+from sentry.backup.scopes import RelocationScope
+from sentry.db.models import DefaultFieldsModel, FlexibleForeignKey, region_silo_model
+
+
+@region_silo_model
+class DetectorGroup(DefaultFieldsModel):
+    """
+    A model to represent the relationship between a detector and a group.
+    """
+
+    __relocation_scope__ = RelocationScope.Excluded
+
+    detector = FlexibleForeignKey("workflow_engine.Detector", on_delete=models.CASCADE)
+    group = FlexibleForeignKey("sentry.Group", on_delete=models.CASCADE, db_constraint=False)
+
+    class Meta:
+        db_table = "workflow_engine_detectorgroup"
+        app_label = "workflow_engine"