Project

General

Profile

Development #6394

Le middleware multitenant doit pouvoir sauvegarder la configuration d'un tenant en json

Added by Serghei Mihai over 9 years ago. Updated over 9 years ago.

Status:
Rejeté
Priority:
Normal
Assignee:
Start date:
02 February 2015
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Le middleware devrait pouvoir sauvegarder la configuration d'un tenant, au format JSON, déclarée dans hobo.


Files

History

#2

Updated by Frédéric Péters over 9 years ago

Ça ajoute une fonction qui n'est pas utilisée, elle serait appelée où ?

    def dump_tenant_settings(self, settings):
        path = os.path.join(settings.TENANT_BASE, tenant.schema_name, self.FILENAME)

La variable tenant, elle arrive d'où ?

#3

Updated by Thomas Noël over 9 years ago

Je n'ai pas compris non plus.

#4

Updated by Serghei Mihai over 9 years ago

Mon idée est que les agents de deploiements appelent cette methode en lui passant le nom(le domain plus précisement) du tenant et la liste des variables particulières à ce tenant.
Ainsi ils(les agents) n'ont pas à se soucier de ou sauvegarder ces parametres et dans quel fichier.

#5

Updated by Thomas Noël over 9 years ago

Ok, mais on est d'accord que le code est boggué ? (la variable "tenant" qui n'existe pas). Et pour moi, ça devrait plutôt aller dans la commande deploy.py, j'ai pas trop envie que cette commande deploy fasse appel à de la magie au fond d'un middleware.

Enfin, je propose qu'on attende d'avoir un usage précis d'un settings.json pour en créer un lors d'un deploy.

#6

Updated by Serghei Mihai over 9 years ago

Oui, il manquait le tenant dans les parametres.
Je garde le ticket ouvert jusqu'à ce qu'on décide comment un sauvegarde le settings d'un tenant

#7

Updated by Thomas Noël over 9 years ago

  • Status changed from Nouveau to Rejeté

ce n'est plus d'actualité, cf Hobo pour la gestion de tout ça

Also available in: Atom PDF