Bug #33590
simplification de la fonction configure_template dans hobo_deploy.py
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
Related issues
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
- Patch proposed changed from No to Yes
- Status changed from Nouveau to Solution proposée
- File 0001-hobo_deploy-code-revue-on-import_template-function-f.patch View added
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
- Status changed from En cours to Solution proposée
- File 0001-hobo_deploy-get-current-service-from-attribute-on-co.patch View added
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']) >>>