Browse Source

ref(migrations): Add hints to old migrations (#27068)

* Add RenameContentType to good ops

* Add hints to all old migrations

* style(lint): Auto commit lint changes

Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Michal Kuffa 3 years ago
parent
commit
665cf48b54

+ 26 - 22
src/sentry/migrations/0001_initial.py

@@ -1,30 +1,31 @@
+import django.db.models.deletion
+import django.utils.timezone
+from django.conf import settings
 from django.db import migrations, models
+
 import bitfield.models
-import sentry.models.scheduledeletion
-import sentry.models.groupshare
-import sentry.db.models.fields.uuid
-import django.utils.timezone
+import sentry.db.mixin
+import sentry.db.models.fields.array
+import sentry.db.models.fields.bounded
 import sentry.db.models.fields.citext
-from django.conf import settings
-import sentry.models.sentryappinstallation
-import sentry.models.apigrant
+import sentry.db.models.fields.encrypted
+import sentry.db.models.fields.foreignkey
 import sentry.db.models.fields.gzippeddict
-import sentry.models.apitoken
-import sentry.models.apiapplication
-import sentry.models.sentryapp
+import sentry.db.models.fields.jsonfield
 import sentry.db.models.fields.node
-import sentry.db.mixin
-import sentry.db.models.fields.bounded
-import sentry.models.useremail
+import sentry.db.models.fields.uuid
+import sentry.models.apiapplication
+import sentry.models.apigrant
+import sentry.models.apitoken
 import sentry.models.broadcast
-import sentry.db.models.fields.array
-import sentry.db.models.fields.jsonfield
+import sentry.models.event
+import sentry.models.groupshare
+import sentry.models.scheduledeletion
+import sentry.models.sentryapp
+import sentry.models.sentryappinstallation
 import sentry.models.servicehook
-import sentry.db.models.fields.foreignkey
-import django.db.models.deletion
 import sentry.models.user
-import sentry.models.event
-import sentry.db.models.fields.encrypted
+import sentry.models.useremail
 
 
 class Migration(migrations.Migration):
@@ -4853,20 +4854,23 @@ class Migration(migrations.Migration):
             end loop;
             end
             $$ language plpgsql;
-        """
+        """,
+            hints={"tables": ["sentry_projectcounter"]},
         ),
         migrations.RunSQL(
             """
         CREATE UNIQUE INDEX sentry_savedsearch_is_global_6793a2f9e1b59b95
         ON sentry_savedsearch USING btree (is_global, name)
         WHERE is_global
-        """
+        """,
+            hints={"tables": ["sentry_savedsearch"]},
         ),
         migrations.RunSQL(
             """
         CREATE UNIQUE INDEX sentry_savedsearch_organization_id_313a24e907cdef99
         ON sentry_savedsearch USING btree (organization_id, name, type)
         WHERE (owner_id IS NULL);
-        """
+        """,
+            hints={"tables": ["sentry_savedsearch"]},
         ),
     ]

+ 7 - 1
src/sentry/migrations/0005_fix_content_types.py

@@ -44,4 +44,10 @@ class Migration(migrations.Migration):
         )
     ]
 
-    operations = [migrations.RunPython(fix_content_types, migrations.RunPython.noop)]
+    operations = [
+        migrations.RunPython(
+            fix_content_types,
+            migrations.RunPython.noop,
+            hints={"tables": ["django_content_type"]},
+        )
+    ]

+ 1 - 0
src/sentry/migrations/0012_remove_pagerdutyservice_service_id.py

@@ -30,6 +30,7 @@ class Migration(migrations.Migration):
                     reverse_sql="""
                         ALTER TABLE "sentry_pagerdutyservice" ADD COLUMN "service_id" varchar(255) NULL;
                         """,
+                    hints={"tables": ["sentry_pagerdutyservice"]},
                 )
             ],
             state_operations=[],

+ 2 - 1
src/sentry/migrations/0015_delete_sentryappwebhookerror_db.py

@@ -26,7 +26,8 @@ class Migration(migrations.Migration):
                 migrations.RunSQL(
                     """
                     DROP TABLE "sentry_sentryappwebhookerror";
-                    """
+                    """,
+                    hints={"tables": ["sentry_sentryappwebhookerror"]},
                 )
             ],
             state_operations=[],

+ 1 - 0
src/sentry/migrations/0016_delete_alert_rule_deprecated_fields.py

@@ -33,6 +33,7 @@ class Migration(migrations.Migration):
                     ALTER TABLE "sentry_alertrule" ADD COLUMN "threshold_type" int NULL;
 
                     """,
+                    hints={"tables": ["sentry_alertrule"]},
                 )
             ],
             state_operations=[],

+ 7 - 1
src/sentry/migrations/0019_auto_20191114_2040.py

@@ -37,4 +37,10 @@ class Migration(migrations.Migration):
         ("sentry", "0018_discoversavedquery_version"),
     ]
 
-    operations = [migrations.RunPython(forwards, migrations.RunPython.noop)]
+    operations = [
+        migrations.RunPython(
+            forwards,
+            migrations.RunPython.noop,
+            hints={"tables": ["sentry_discoversavedquery"]},
+        )
+    ]

+ 7 - 1
src/sentry/migrations/0020_auto_20191125_1420.py

@@ -39,4 +39,10 @@ class Migration(migrations.Migration):
         ("sentry", "0019_auto_20191114_2040"),
     ]
 
-    operations = [migrations.RunPython(forwards, migrations.RunPython.noop)]
+    operations = [
+        migrations.RunPython(
+            forwards,
+            migrations.RunPython.noop,
+            hints={"tables": ["sentry_eventattachment"]},
+        )
+    ]

+ 7 - 1
src/sentry/migrations/0023_hide_environment_none_20191126.py

@@ -33,4 +33,10 @@ class Migration(migrations.Migration):
         ("sentry", "0022_merge"),
     ]
 
-    operations = [migrations.RunPython(hide_environment_none, migrations.RunPython.noop)]
+    operations = [
+        migrations.RunPython(
+            hide_environment_none,
+            migrations.RunPython.noop,
+            hints={"tables": ["sentry_environmentproject"]},
+        )
+    ]

+ 5 - 1
src/sentry/migrations/0024_auto_20191230_2052.py

@@ -144,5 +144,9 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.RunPython(backfill_eventstream, reverse_code=migrations.RunPython.noop),
+        migrations.RunPython(
+            backfill_eventstream,
+            reverse_code=migrations.RunPython.noop,
+            hints={"tables": ["sentry_event", "sentry_groupedmessage", "sentry_project"]},
+        ),
     ]

+ 7 - 3
src/sentry/migrations/0028_user_reports.py

@@ -7,9 +7,9 @@ from django.db import migrations
 from sentry import eventstore
 from sentry.utils.query import RangeQuerySetWrapper
 from sentry.utils.snuba import (
-    SnubaError,
-    QueryOutsideRetentionError,
     QueryOutsideGroupActivityError,
+    QueryOutsideRetentionError,
+    SnubaError,
 )
 
 logger = logging.getLogger(__name__)
@@ -60,5 +60,9 @@ class Migration(migrations.Migration):
     ]
 
     operations = [
-        migrations.RunPython(backfill_user_reports, migrations.RunPython.noop),
+        migrations.RunPython(
+            backfill_user_reports,
+            migrations.RunPython.noop,
+            hints={"tables": ["sentry_userreport"]},
+        ),
     ]

Some files were not shown because too many files changed in this diff