https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-05-31T11:56:26ZRedmine Entr’ouvertHobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=1763922019-05-31T11:56:26ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-1 status-5 priority-4 priority-default closed" href="/issues/33224">Bug #33224</a>: tests sur hobo_deploy.py (côté agents)</i> ajouté</li></ul> Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=1764202019-05-31T12:28:52ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/34865">0001-hobo_deploy-code-revue-on-import_template-function-f.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/34865/0001-hobo_deploy-code-revue-on-import_template-function-f.patch">0001-hobo_deploy-code-revue-on-import_template-function-f.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>La fonction ne fait plus que 2 lignes et n'est appelée que par <code>deploy_specifics</code>,<br />peut-être faut-il la supprimer pour plus de lisibilité ?</p> Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=1802002019-06-21T08:53:13ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Ok avec le patch mais il faut revoir son intitulé. (genre "hobo_deploy: get current service from attribute (#.....)")</p> Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=1804702019-06-24T10:16:40ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>En cours</i></li></ul><p>(il faut que je rebase sur <a class="issue tracker-2 status-5 priority-4 priority-default closed child" title="Development: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenant (Fermé)" href="https://dev.entrouvert.org/issues/33873">#33873</a>...)</p> Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=1813992019-06-28T13:14:02ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/35687">0001-hobo_deploy-get-current-service-from-attribute-on-co.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/35687/0001-hobo_deploy-get-current-service-from-attribute-on-co.patch">0001-hobo_deploy-get-current-service-from-attribute-on-co.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li></ul><p>maj de l'intitulé,<br />et désolé, j'ai introduit un effet de bord dans <a class="issue tracker-2 status-5 priority-4 priority-default closed child" title="Development: l'appel à "import_template" via "hobo_deploy" doit spécifier un tenant (Fermé)" href="https://dev.entrouvert.org/issues/33873">#33873</a> que je corrige ici<br /><pre>
<<<
call_command('import_template', me['template_name'])
---
call_command('import_template', self.me['template_name'])
>>>
</pre></p> Hobo - Bug #33590: simplification de la fonction configure_template dans hobo_deploy.pyhttps://dev.entrouvert.org/issues/33590?journal_id=2209012020-03-14T16:32:27ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Rejeté</i></li></ul><p>Je réalise que ces méthodes peuvent être appelées directement sans passer par la méthode <code>deploy</code>.<br />Par exemple lors du déploiement de bijoe via <em>bijoe/hobo_agent/management/commands/hobo_deploy.py</em> :<br /><pre>
class Command(hobo_deploy.Command):
def deploy_specifics(self, hobo_environment, tenant):
super(Command, self).deploy_specifics(hobo_environment, tenant)
</pre><br />Et donc, qu'on ne peut pas simplifier en supposant comme ici que self.me est renseigné.</p>