Development #50828
déployer les tenants dans un sous-répertoire tenants/
0%
Description
Pour le moment c'est directement sous /var/lib/wcs/, par similitude avec les autres modules, ça serait bien d'avoir ça sous /var/lib/wcs/tenants/.
Par compatibilité on chercherait dans les deux répertoires.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier 0001-general-add-support-for-tenants-in-a-tenants-subdire.patch 0001-general-add-support-for-tenants-in-a-tenants-subdire.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
/var/lib/wcs/tenants/XXX utilisé pour les nouveaux déploiements, l'emplacement /var/lib/wcs/ encore effectif pour compatibilité, pas de migration auto (un mv côté système doit être suffisant mais vu comme il a pu y avoir des liens symboliques posés pour par exemple les pièces jointes, je préfère vraiment ne pas automatiser ça).
Mis à jour par Thomas Noël il y a presque 3 ans
Dans wcs/ctl/hobo_notify.py ou wcs/ctl/management/commands/convert_to_sql.py tu as laissé des publisher.set_config() après le publisher.set_tenant... : ça me semble inutile.
Un "git grep set_tenant" montre qu'il reste aussi dans d'autres wcs/ctl (delete_tenant.py, runscript.py, wipe_data.py) où tu as choisi de ne pas jouer avec set_tenant, c'est sans doute voulu ?
(En dehors de ce détail, ça me semble tout très bien)
Mis à jour par Frédéric Péters il y a presque 3 ans
Un "git grep set_tenant" montre qu'il reste aussi dans d'autres wcs/ctl (delete_tenant.py, runscript.py, wipe_data.py) où tu as choisi de ne pas jouer avec set_tenant, c'est sans doute voulu ?
Perdu un moment, mais en fait "git grep set_config" veux-tu dire.
Voilà réduit,
$ git grep set_config [des tests] wcs/ctl/check_hobos.py: pub.set_config(skip_sql=True) wcs/ctl/check_hobos.py: pub.set_config(skip_sql=False) wcs/publisher.py: def set_config(self, request=None, skip_sql=False): wcs/publisher.py: QommonPublisher.set_config(self, request=request) wcs/qommon/publisher.py: def set_config(self, request=None): wcs/qommon/publisher.py: publisher.set_config() wcs/qommon/publisher.py: self.set_config(**kwargs)
ce qui m'a fait noter le skip_sql utilisé là, qui m'amène quelques doutes sur la modification wcs/ctl/check_hobos.py qui désormais commencera par un set_tenant() qui fera le set_config(), sans passer ça. Je pense que c'est ok parce que ça va dire qu'il n'y a pas le tenant et enchainer correctement la création, avec les appels set_config(skip_sql=False puis True) conservés.
Mis à jour par Frédéric Péters il y a presque 3 ans
Mis à jour par Thomas Noël il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
Un "git grep set_tenant" montre qu'il reste aussi dans d'autres wcs/ctl (delete_tenant.py, runscript.py, wipe_data.py) où tu as choisi de ne pas jouer avec set_tenant, c'est sans doute voulu ?
Perdu un moment, mais en fait "git grep set_config" veux-tu dire.
Bien sûr... désolé...
ce qui m'a fait noter le skip_sql utilisé là, qui m'amène quelques doutes sur la modification wcs/ctl/check_hobos.py qui désormais commencera par un set_tenant() qui fera le set_config(), sans passer ça. Je pense que c'est ok parce que ça va dire qu'il n'y a pas le tenant et enchainer correctement la création, avec les appels set_config(skip_sql=False puis True) conservés.
Même analyse. Testé localement quelques déploiements, pas de soucis rencontrés.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 06b7412dd60056df585b3f573ccd99de942d920c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sun May 2 21:59:07 2021 +0200 general: add support for tenants in a tenants subdirectory (#50828)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: add support for tenants in a tenants subdirectory (#50828)