Development #6467
Renommer/deplacer la commande de deploiement afin de permettre à l'application de la surcharger
0%
Description
Renommer la commande implementée dans le ticket #6340 pour ne pas avoir de conflit des noms des commandes des applications.
Cette commande pourrait être utilisée par une application afin d'implementer la logique de deploiement du tenant
Files
History
Updated by Serghei Mihai over 9 years ago
- File 0001-re-usable-tenant-deploy-command.patch 0001-re-usable-tenant-deploy-command.patch added
- Patch proposed changed from No to Yes
Placée dans le __init__, la classe peut être héritée par les commandes de deploiement des applications
Updated by Frédéric Péters over 9 years ago
Il me semble que ce patch supprime la commande, ça ne me semble pas l'affaire souhaitée. Par ailleurs je ne vois pas pourquoi une commande tierce ne pourrait pas hériter de la classe définie dans deploy.py, quel est le soucis ?
Updated by Serghei Mihai over 9 years ago
En effet, ça supprime la commande de l'application tenant_schemas et laisse la possibilité à une autre application, par exemple authentic, de la rajouter.
Cela m'évite à avoir à gerer l'ordre dans lequel les commandes des applications sont appelées.
Updated by Frédéric Péters over 9 years ago
Mais l'idée c'était d'avoir dans python-entrouvert une commande de base, satisfaisant les besoins communs (combo, passerelle, etc.), ce que tu retires, non ?
Updated by Serghei Mihai over 9 years ago
Cette commande de base ne fait que créer le tenant(schéma + répertoire).
Ensuite chaque application (combo, passerrelle, ...) implemente ses particularités de deploiement, comme par exemple l'ajout d'authentic en tant qu'idp, via la méthode deploy_tenant
Updated by Frédéric Péters over 9 years ago
On va donc avoir dans passerelle, dans combo, dans corbo, dans etc. le même code, qui va faire un create_tenant puis ajouter la conf django-mellon ? Je ne trouve pas que ça soit adéquat, et vraiment dommage quand la seule raison donnée soit de ne pas avoir à s'interroger sur l'ordre de chargement des applications dans authentic.
Updated by Serghei Mihai over 9 years ago
- Status changed from Nouveau to Rejeté
Je pensais que chaque application avait plus de configs à faire au niveau de deploy_command.
On peut ignorer ce patch.