Project

General

Profile

Développement #6394

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

Added by Serghei Mihai over 10 years ago. Updated over 10 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 10 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 10 years ago

Je n'ai pas compris non plus.

#4

Updated by Serghei Mihai over 10 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 10 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 10 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 10 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