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
tenantinutilisé - 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 10 months ago
- Related to Bug #33224: tests sur hobo_deploy.py (côté agents) added
#2 Updated by Nicolas Roche 10 months ago
- Patch proposed changed from No to Yes
- File 0001-hobo_deploy-code-revue-on-import_template-function-f.patch View added
- Status changed from Nouveau to Solution proposée
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 10 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 10 months ago
- Status changed from Solution proposée to En cours
(il faut que je rebase sur #33873...)
#5 Updated by Nicolas Roche 10 months ago
- File 0001-hobo_deploy-get-current-service-from-attribute-on-co.patch View added
- Status changed from En cours to Solution proposée
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'])
>>>
#6 Updated by Nicolas Roche 25 days ago
- Status changed from Solution proposée to Rejeté
Je réalise que ces méthodes peuvent être appelées directement sans passer par la méthode deploy.
Par exemple lors du déploiement de bijoe via bijoe/hobo_agent/management/commands/hobo_deploy.py :
class Command(hobo_deploy.Command):
def deploy_specifics(self, hobo_environment, tenant):
super(Command, self).deploy_specifics(hobo_environment, tenant)
Et donc, qu'on ne peut pas simplifier en supposant comme ici que self.me est renseigné.