https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-06-12T15:56:53ZRedmine Entr’ouvertHobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1786202019-06-12T15:56:53ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/35205">0001-hobo_deploy-internally-call-import_template-using-te.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/35205/0001-hobo_deploy-internally-call-import_template-using-te.patch">0001-hobo_deploy-internally-call-import_template-using-te.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>les 3 commandes (indépendantes) suivantes sont chaînées : hobo_deploy -> import_template -> import_site</p>
<ul>
<li>par exemple avec passerelle on fait :<br /><pre>
$ passerelle-manage tenant_command import_site /var/lib/passerelle/skeletons/signal-publik.json \
-d passerelle.dev.publik.love
</pre></li>
<li>qui sinon est appelée de façon générique par : <br /><pre>
$ passerelle-manage tenant_command import_template signal-publik -d passerelle.dev.publik.love
</pre></li>
</ul>
<ul>
<li>cette précédente commande étant elle même par ailleurs appelée en fin de 'hobo_deploy'<br /><pre>
$ passerelle-manage hobo_deploy https://passerelle.dev.publik.love/ env.json --ignore-timestamp
</pre></li>
</ul>
<p>ce patch permet de réaliser ce troisième point.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1791002019-06-14T15:47:42ZChristophe Siraut
<ul></ul><p>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.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1791212019-06-15T14:07:23ZBenjamin Dauvergne
<ul></ul><p>Christophe Siraut a écrit :</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>Il faut éviter les set_tenant() direct et préférer un <code>with tenant_context(tenant)</code> et oui un call_command serait plus joli.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1793242019-06-17T13:47:20ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/35334">0001-hobo_deploy-internally-call-import_template-using-te.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/35334/0001-hobo_deploy-internally-call-import_template-using-te.patch">0001-hobo_deploy-internally-call-import_template-using-te.patch</a> ajouté</li></ul><p>Merci pour les précisions.</p>
<p>Pour mémoire, la première version avait le mérite d'éviter d'être obligé d'exécuter le test <em>hobo/tests_schemas/test_hobo_deploy.py::test_import_template</em> avant les autres tests qui patchent <code>get_commands</code> ou <code>call_command</code>.</p>
<p>C'est un effet de bord lié au fait que la fonction <code>get_commands</code> de Django est mise en cache et qui risque d'être pénalisant par la suite.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1811702019-06-27T16:52:44ZChristophe Siraut
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1813122019-06-28T10:12:11ZNicolas Roche (absent jusqu'au 3 avril)
<ul></ul><p>merci,<br />pour info j'ai aussi essayé :<br /><pre>
call_command('tenant_command', 'import_template', me['template_name'],
'-d', tenant.domain)
</pre> <br />mais je bute comme pour <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: tests : ne pas utiliser call_command('tenant_command', 'autre commande', ...) (Fermé)" href="https://dev.entrouvert.org/issues/25657">#25657</a></p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1813222019-06-28T10:31:17ZBenjamin Dauvergne
<ul></ul><p><code>domain=tenant.comain</code> je pense, call_command prend les arguments déjà parsés.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1813492019-06-28T11:23:08ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Oui mais de toute façon non, <a class="external" href="https://github.com/bernardopires/django-tenant-schemas/issues/495">https://github.com/bernardopires/django-tenant-schemas/issues/495</a> (positional arguments ko).</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1813572019-06-28T11:45:04ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
commit 43c082bf3bbfd4d49738e45ba2c3079a2efd1d07
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date: Wed Jun 12 16:22:45 2019 +0200
hobo_deploy: internally call import_template using tenant_command (#33873)
</pre> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1813802019-06-28T12:31:22ZBenjamin Dauvergne
<ul></ul><p>De toute façon je suis pour ne pas utiliser tenant_command dans des scripts, tenant_context() fait très bien le job.</p> Hobo - Development #33873: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenanthttps://dev.entrouvert.org/issues/33873?journal_id=1821582019-07-03T21:15:35ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>