Project

General

Profile

Development #31042

django-tenant-schemas: ne pas exécuter les migrations sur le schéma public

Added by Benjamin Dauvergne 4 months ago. Updated 3 days ago.

Status:
Solution validée
Priority:
Normal
Category:
multitenant
Start date:
02 Mar 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Nous n'en avons pas l'usage et ça empêche de fait d'utiliser une base unique pour tous les tenants qui pour moi est quelque chose de sensé (sur un hébergement mono-instance ça simplifiera beaucoup les backups).

0003-migrate_schemas-remove-migration-of-public-schema-fi.patch View (1.41 KB) Benjamin Dauvergne, 02 Mar 2019 12:29 PM

0002-migrate_schemas-add-copyright-notice-31042.patch View (1.42 KB) Benjamin Dauvergne, 02 Mar 2019 12:29 PM

0001-migrate_schemas-PEP8ness-remove-useless-imports-3104.patch View (2.03 KB) Benjamin Dauvergne, 02 Mar 2019 12:29 PM

0003-migrate_schemas-remove-migration-of-public-schema-fi.patch View (1.41 KB) Benjamin Dauvergne, 19 Mar 2019 02:21 AM

0002-migrate_schemas-add-copyright-notice-31042.patch View (1.42 KB) Benjamin Dauvergne, 19 Mar 2019 02:21 AM

0001-migrate_schemas-PEP8ness-remove-useless-imports-3104.patch View (2.03 KB) Benjamin Dauvergne, 19 Mar 2019 02:21 AM

0003-migrate_schemas-remove-migration-of-public-schema-fi.patch View (4.72 KB) Benjamin Dauvergne, 06 Jun 2019 07:33 PM

0002-migrate_schemas-add-copyright-notice-31042.patch View (1.42 KB) Benjamin Dauvergne, 06 Jun 2019 07:33 PM

0001-migrate_schemas-PEP8ness-remove-useless-imports-3104.patch View (2.03 KB) Benjamin Dauvergne, 06 Jun 2019 07:33 PM

0003-migrate_schemas-remove-migration-of-public-schema-fi.patch View (4.76 KB) Benjamin Dauvergne, 07 Jun 2019 10:22 PM

0002-migrate_schemas-add-copyright-notice-31042.patch View (1.42 KB) Benjamin Dauvergne, 07 Jun 2019 10:22 PM

0001-migrate_schemas-PEP8ness-remove-useless-imports-3104.patch View (2.03 KB) Benjamin Dauvergne, 07 Jun 2019 10:22 PM

0003-migrate_schemas-remove-migration-of-public-schema-fi.patch View (7.96 KB) Benjamin Dauvergne, 23 Jun 2019 02:22 AM

0002-migrate_schemas-add-copyright-notice-31042.patch View (1.42 KB) Benjamin Dauvergne, 23 Jun 2019 02:22 AM

0001-migrate_schemas-PEP8ness-remove-useless-imports-3104.patch View (2.03 KB) Benjamin Dauvergne, 23 Jun 2019 02:22 AM


Related issues

Related to Hobo - Bug #22883: ne jamais utiliser le schéma public Nouveau 29 Mar 2018

History

#1 Updated by Benjamin Dauvergne 4 months ago

  • Assignee set to Benjamin Dauvergne

#2 Updated by Benjamin Dauvergne 4 months ago

#4 Updated by Benjamin Dauvergne 4 months ago

  • Subject changed from Ne pas exécuter les migrations sur le schéma public to django-tenant-schemas: ne pas exécuter les migrations sur le schéma public
  • Category set to multitenant

#6 Updated by Emmanuel Cazenave 23 days ago

Je pense qu'il faut aller jusqu'au bout du ménage : dans SyncCommon les options --tenant et --shared n'ont plus de raison d'être si je ne m'abuse.

#7 Updated by Benjamin Dauvergne 22 days ago

  • Status changed from Solution proposée to En cours

Ok.

#10 Updated by Christophe Siraut 9 days ago

En soi le patch me semble bien, et c'est cohérent avec la notion de schéma définie par postgresql; mais ça me pose problème de diverger de django-tenant-schema, est-ce qu'on ne devrait pas avoir une discussion plus générale pour une vision long terme :

  • Est-ce qu'on assume de diverger de django-tenant-schema upstream et de maintenir ça ? (Et du coup on oublierait l'alternative mieux maintenue que tu pointais)

#11 Updated by Benjamin Dauvergne 9 days ago

Christophe Siraut a écrit :

En soi le patch me semble bien, et c'est cohérent avec la notion de schéma définie par postgresql; mais ça me pose problème de diverger de django-tenant-schema, est-ce qu'on ne devrait pas avoir une discussion plus générale pour une vision long terme :

  • Est-ce qu'on assume de diverger de django-tenant-schema upstream et de maintenir ça ? (Et du coup on oublierait l'alternative mieux maintenue que tu pointais)

django-tenant-schema n'est plus maintenu, forké par django-tenants1 qui, disons, le remplace, les API ne sont malheureusement pas compatibles (mais pas éloignées non plus) simplement parce que le package s'appelle django_tenants; donc pour l'instant nous sommes seuls.

1 https://github.com/tomturner/django-tenants

#12 Updated by Emmanuel Cazenave 8 days ago

Je rajoute une dernière couche de demande de simplification : je pense qu'il faut jarter la variba@sync_tenant@ qui n'a plus de sens, on va toujours synchroniser un ou plusieurs tenant.

La seule option qui a du sens avec sa variable associée c'est schema_name , soit elle est définie et on migre un seul tenant, soit elle est pas là et on migre tous les tenants.

#13 Updated by Frédéric Péters 6 days ago

  • Related to Bug #22883: ne jamais utiliser le schéma public added

#15 Updated by Emmanuel Cazenave 3 days ago

  • Status changed from Solution proposée to Solution validée

Ok (pour prochain cycle).

Also available in: Atom PDF