Browse Source

fix(migrations): Fix migration state (#27042)

Missed updating the migration state in https://github.com/getsentry/sentry/pull/26907
Dan Fuller 3 years ago
parent
commit
8fd96c1645
2 changed files with 41 additions and 1 deletions
  1. 1 1
      migrations_lockfile.txt
  2. 40 0
      src/sentry/migrations/0215_fix_state.py

+ 1 - 1
migrations_lockfile.txt

@@ -7,5 +7,5 @@ will then be regenerated, and you should be able to merge without conflicts.
 
 jira_ac: 0001_initial
 nodestore: 0002_nodestore_no_dictfield
-sentry: 0214_activity_project_type_datetime_idx
+sentry: 0215_fix_state
 social_auth: 0001_initial

+ 40 - 0
src/sentry/migrations/0215_fix_state.py

@@ -0,0 +1,40 @@
+# Generated by Django 1.11.29 on 2021-06-30 21:21
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+    # This flag is used to mark that a migration shouldn't be automatically run in
+    # production. We set this to True for operations that we think are risky and want
+    # someone from ops to run manually and monitor.
+    # General advice is that if in doubt, mark your migration as `is_dangerous`.
+    # Some things you should always mark as dangerous:
+    # - Large data migrations. Typically we want these to be run manually by ops so that
+    #   they can be monitored. Since data migrations will now hold a transaction open
+    #   this is even more important.
+    # - Adding columns to highly active tables, even ones that are NULL.
+    is_dangerous = False
+
+    # This flag is used to decide whether to run this migration in a transaction or not.
+    # By default we prefer to run in a transaction, but for migrations where you want
+    # to `CREATE INDEX CONCURRENTLY` this needs to be set to False. Typically you'll
+    # want to create an index concurrently when adding one to an existing table.
+    # You'll also usually want to set this to `False` if you're writing a data
+    # migration, since we don't want the entire migration to run in one long-running
+    # transaction.
+    atomic = True
+
+    dependencies = [
+        ("sentry", "0214_activity_project_type_datetime_idx"),
+    ]
+
+    operations = [
+        migrations.SeparateDatabaseAndState(
+            state_operations=[
+                migrations.AlterIndexTogether(
+                    name="activity",
+                    index_together={("project", "datetime")},
+                ),
+            ],
+        ),
+    ]