Project

General

Profile

Bug #33590

simplification de la fonction configure_template dans hobo_deploy.py

Added by Nicolas Roche 10 months ago. Updated 25 days ago.

Status:
Rejeté
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 10 months ago

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

#2 Updated by Nicolas Roche 10 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 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

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é.

Also available in: Atom PDF