Produits Entr'ouvert » Hobo

Les migrations doivent être jouées sur tous les tenants (et passer ainsi sur le schéma "public"). (et si tu le fais pas c'est pas grave si tu sais que tu ne vas pas sur les tenants). (je zapperais ce ticket).

Mais, tu peux juste appliquer les migrations et que ça s'arrête là ?

Nicolas Roche a écrit :

Là j'ai remplacé les warning du tenant public par ceux de hobo.multitenant (enfin je crois).

(je zapperais ce ticket)

yep, c'est parce que je devenais fou avec toutes ces migrations qui s'accumulaient sans que j'arrive à les appliquer.
(maintenant je sais que c'est normal, ouf)

Je ne vois pas trop ce que tu fais là, tu écrases applied_migrations à chaque tour de ta boucle, je dirai que le plus simple ce serait de vider complètement check_migrations(), ou vraiment si tu veux le faire tourner d'ajouter une option par défaut à False (et dans ce cas je pense que tu pourrais juste utiliser with tenant_context(tenant): super().check_migrations()).

Mais vraiment juste mettre pass ça me parait suffisant.

Frédéric Péters a écrit :

Mais, tu peux juste appliquer les migrations et que ça s'arrête là ?

Je n'utilise pas cette commande, mais pour ceux qui le font, avec beaucoup de tenant ça va couter un peu cher un migrate_schemas à chaque fois non ? Déjà que je trouve que runserver met longtemps à se lancer (certainement à cause de ce check).

Sauf que, j'avais oublié, #31042 et du coup le schéma public n'est plus une base fiable. Mais faudrait écrire ça quelque part. Et idéalement pas trop écrire de code, là il y a toute une part,

            app_labels = []
            for app in apps.get_app_configs():
                if app.name in settings.TENANT_APPS:
                    app_labels.append(app.label)
            loader = MigrationLoader(None)
            loader.load_disk()
            all_migrations = set([(app, migration) for app, migration in loader.disk_migrations if app in app_labels])
            for tenant in TenantMiddleware.get_tenants():
                connection.set_schema(tenant.schema_name, include_public=False)
                applied_migrations = self.get_applied_migrations(app_labels)
                if options.get('fake') or options.get('migration_name') or options.get('app_label'):
                    # never skip migrations if explicit migration actions
                    # are given.
                    applied_migrations = []

qui se trouve déjà dans migrate_schemas.

Est-ce qu'on pourrait arriver à, genre, avoir là :

all_labels, all_migrations = get_all_labels_and_migrations()
for tenant in TenantMiddleware.get_tenants():
    connection.set_schema(tenant.schema_name, include_public=False)
    applied_migrations = self.get_applied_migrations(app_labels)
    (...)

et ici,

all_labels, all_migrations = get_all_labels_and_migrations()
tenants = TenantMiddleware.get_tenants()
if tenants:
    connection.set_schema(tenants[0].schema_name, include_public=False)
    applied_migrations = self.get_applied_migrations(app_labels)
    plan = all_migrations - applied_migrations
    if plan:
        print(...)

un migrate_schemas à chaque fois non

Je parlais de lancer migrate_schemas un coup quand il voit qu'il le message, dans l'idée que ça allait au moins s'inscrire dans public et faire disparaitre le message, avant de regarder attentivement et de voir que public n'était plus utilisé, etc.

(et sans revenir sur l'historique qui nous amène donc ici, ça me va de juste voir commitée la version avec "pass" et ignorer tout ça)