Skip to content

Commit 47e8e4a

Browse files
authored
chore(ACI): add event_id column to group open period model (#104549)
We need to link between open periods and events on the frontend. Add an event ID column to the group open period model to accomplish this. Make the field nullable as we do not plan to backfill existing open periods.
1 parent 0674eff commit 47e8e4a

File tree

3 files changed

+35
-1
lines changed

3 files changed

+35
-1
lines changed

migrations_lockfile.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ releases: 0004_cleanup_failed_safe_deletes
3131

3232
replays: 0006_add_bulk_delete_job
3333

34-
sentry: 1011_update_oc_integration_cascade_to_null
34+
sentry: 1012_add_event_id_to_open_period
3535

3636
social_auth: 0003_social_auth_json_field
3737

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
# Generated by Django 5.2.8 on 2025-12-08 22:14
2+
3+
from django.db import migrations, models
4+
5+
from sentry.new_migrations.migrations import CheckedMigration
6+
7+
8+
class Migration(CheckedMigration):
9+
# This flag is used to mark that a migration shouldn't be automatically run in production.
10+
# This should only be used for operations where it's safe to run the migration after your
11+
# code has deployed. So this should not be used for most operations that alter the schema
12+
# of a table.
13+
# Here are some things that make sense to mark as post deployment:
14+
# - Large data migrations. Typically we want these to be run manually so that they can be
15+
# monitored and not block the deploy for a long period of time while they run.
16+
# - Adding indexes to large tables. Since this can take a long time, we'd generally prefer to
17+
# run this outside deployments so that we don't block them. Note that while adding an index
18+
# is a schema change, it's completely safe to run the operation after the code has deployed.
19+
# Once deployed, run these manually via: https://develop.sentry.dev/database-migrations/#migration-deployment
20+
21+
is_post_deployment = False
22+
23+
dependencies = [
24+
("sentry", "1011_update_oc_integration_cascade_to_null"),
25+
]
26+
27+
operations = [
28+
migrations.AddField(
29+
model_name="groupopenperiod",
30+
name="event_id",
31+
field=models.CharField(max_length=32, null=True),
32+
),
33+
]

src/sentry/models/groupopenperiod.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@ class GroupOpenPeriod(DefaultFieldsModel):
5858
date_ended = models.DateTimeField(null=True)
5959

6060
data = models.JSONField(default=dict)
61+
event_id = models.CharField(max_length=32, null=True)
6162

6263
class Meta:
6364
app_label = "sentry"

0 commit comments

Comments
 (0)