nicer logs

posthog/management/commands/test_ch_migrations_are_safe.py

@@ -33,14 +33,14 @@ def run_and_check_migration(new_migration):
             old_migrations.sort()
 
             for index, name in old_migrations:
-                logger.warn(f"old ClickHouse migration with index {index} and name {name} found")
+                logger.info(f"old ClickHouse migration with index {index} and name {name} found")
 
             try:
                 should_fail = False
                 app, index, name = re.findall(
                     r"([a-z]+)\/clickhouse\/migrations\/([0-9]+)_([a-zA-Z_0-9]+)\.py", new_migration
                 )[0]
-                logger.warn(f"new ClickHouse migration for app {app} with index {index} and name {name} found")
+                logger.info(f"new ClickHouse migration for app {app} with index {index} and name {name} found")
 
                 matching_migration_indexes = []
                 for old_index, old_name in old_migrations:
@@ -52,6 +52,7 @@ def run_and_check_migration(new_migration):
                     logger.error("Colliding migrations are:")
                     for old_index, old_name in matching_migration_indexes:
                         logger.error(f"  - {old_index}_{old_name}")
+                    logger.error(f"  - {index}_{name}")
                     logger.error(
                         "Please manually resolve this conflict and ensure all migrations are monotonically increasing"
                     )