Project

General

Profile

Actions

Développement #6394

closed

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

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

Actions #2

Updated by Frédéric Péters (de retour le 18/5) over 11 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ù ?

Actions #3

Updated by Thomas Noël over 11 years ago

Je n'ai pas compris non plus.

Actions #4

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

Actions #5

Updated by Thomas Noël over 11 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.

Actions #6

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

Actions #7

Updated by Thomas Noël about 11 years ago

  • Status changed from Nouveau to Rejeté

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

Actions

Also available in: Atom PDF