Project

General

Profile

Development #33873

l'appel à "import_template" via "hobo_deploy" doit spécifier un tenant

Added by Nicolas Roche 3 days ago. Updated about 7 hours ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Category:
agent
Start date:
12 Jun 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

La commande "import_template" se lance via "tenant_command" (#20769).
Aussi, lorsque la commande "hobo_deploy" se fini en invoquant "import_template", elle doit également préciser le tenant.

0001-hobo_deploy-internally-call-import_template-using-te.patch View (6.29 KB) Nicolas Roche, 12 Jun 2019 05:39 PM

History

#1 Updated by Nicolas Roche 3 days ago

les 3 commandes (indépendantes) suivantes sont chaînées : hobo_deploy -> import_template -> import_site

  • par exemple avec passerelle on fait :
    $ passerelle-manage tenant_command import_site /var/lib/passerelle/skeletons/signal-publik.json \
        -d passerelle.dev.publik.love
    
  • qui sinon est appelée de façon générique par :
    $ passerelle-manage tenant_command import_template signal-publik -d passerelle.dev.publik.love
    
  • cette précédente commande étant elle même par ailleurs appelée en fin de 'hobo_deploy'
    $ passerelle-manage hobo_deploy https://passerelle.dev.publik.love/ env.json --ignore-timestamp
    

ce patch permet de réaliser ce troisième point.

#2 Updated by Christophe Siraut 1 day ago

execute_from_command_line ne me semble pas approprié ici, je serais tenté par connection.set_tenant() avant dans lancer call_command, mais ça n'est pas top non-plus.

#3 Updated by Benjamin Dauvergne about 7 hours ago

Christophe Siraut a écrit :

execute_from_command_line ne me semble pas approprié ici, je serais tenté par connection.set_tenant() avant dans lancer call_command, mais ça n'est pas top non-plus.

Il faut éviter les set_tenant() direct et préférer un with tenant_context(tenant) et oui un call_command serait plus joli.

Also available in: Atom PDF