Project

General

Profile

Bug #33590

simplification de la fonction configure_template dans hobo_deploy.py

Added by Nicolas Roche 7 months ago. Updated 6 months ago.

Status:
Solution proposée
Priority:
Bas
Assignee:
Category:
agent
Target version:
-
Start date:
31 May 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

proposition de simplification de la fonction :
  • supprimer le paramètre tenant inutilisé
  • la fonction n'est pas appelée directement (comme c'est prévu dans cook.py) et donc on n'a pas besoin de recalculer la variable me
  • import_template.py est placé dans agent/common/management/commands/ au même niveau que hobo_deploy.sh, il est donc inutile de tester la disponibilité de cette commande qui sera toujours présente

0001-hobo_deploy-code-revue-on-import_template-function-f.patch View (4.75 KB) Nicolas Roche, 31 May 2019 02:24 PM

0001-hobo_deploy-get-current-service-from-attribute-on-co.patch View (3.57 KB) Nicolas Roche, 28 Jun 2019 03:13 PM


Related issues

Related to Hobo - Bug #33224: tests sur hobo_deploy.py (côté agents) Solution déployée 17 May 2019

History

#1 Updated by Nicolas Roche 7 months ago

  • Related to Bug #33224: tests sur hobo_deploy.py (côté agents) added

#2 Updated by Nicolas Roche 7 months ago

La fonction ne fait plus que 2 lignes et n'est appelée que par deploy_specifics,
peut-être faut-il la supprimer pour plus de lisibilité ?

#3 Updated by Frédéric Péters 6 months ago

Ok avec le patch mais il faut revoir son intitulé. (genre "hobo_deploy: get current service from attribute (#.....)")

#4 Updated by Nicolas Roche 6 months ago

  • Status changed from Solution proposée to En cours

(il faut que je rebase sur #33873...)

#5 Updated by Nicolas Roche 6 months ago

maj de l'intitulé,
et désolé, j'ai introduit un effet de bord dans #33873 que je corrige ici

<<<
call_command('import_template', me['template_name'])
---
call_command('import_template', self.me['template_name'])
>>>

Also available in: Atom PDF