Bug #33590
simplification de la fonction configure_template dans hobo_deploy.py
Début:
31 mai 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
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
Fichiers
Demandes liées
Historique
Mis à jour par Nicolas Roche il y a presque 5 ans
- Lié à Bug #33224: tests sur hobo_deploy.py (côté agents) ajouté
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-hobo_deploy-code-revue-on-import_template-function-f.patch 0001-hobo_deploy-code-revue-on-import_template-function-f.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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é ?
Mis à jour par Frédéric Péters il y a presque 5 ans
Ok avec le patch mais il faut revoir son intitulé. (genre "hobo_deploy: get current service from attribute (#.....)")
Mis à jour par Nicolas Roche il y a presque 5 ans
- Statut changé de Solution proposée à En cours
(il faut que je rebase sur #33873...)
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0001-hobo_deploy-get-current-service-from-attribute-on-co.patch 0001-hobo_deploy-get-current-service-from-attribute-on-co.patch ajouté
- Statut changé de En cours à 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'])
>>>
Mis à jour par Nicolas Roche il y a environ 4 ans
- Statut changé de Solution proposée à 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é.