Skip to content

Commit e421c6a

Browse files
chore(launchpad): create replacement analysis_file_id col in preprod size metrics table
1 parent 7fa5d73 commit e421c6a

File tree

3 files changed

+38
-1
lines changed

3 files changed

+38
-1
lines changed

migrations_lockfile.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ nodestore: 0001_squashed_0002_nodestore_no_dictfield
2121

2222
notifications: 0001_move_notifications_models
2323

24-
preprod: 0007_add_install_file
24+
preprod: 0008_make_preprod_analysis_file_id_in_size_metrics_table
2525

2626
replays: 0006_add_bulk_delete_job
2727

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

src/sentry/preprod/models.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -223,6 +223,9 @@ def as_choices(cls):
223223
min_download_size = BoundedPositiveBigIntegerField(null=True)
224224
max_download_size = BoundedPositiveBigIntegerField(null=True)
225225

226+
# Size analysis wont necessarily be run on every artifact (based on quotas)
227+
analysis_file_id = BoundedBigIntegerField(db_index=True, null=True)
228+
226229
class Meta:
227230
app_label = "preprod"
228231
db_table = "sentry_preprodartifactsizemetrics"

0 commit comments

Comments
 (0)