Project

General

Profile

Development #33874

"import_template" doit pouvoir au début échouer, puis être retentée

Added by Nicolas Roche 2 months ago. Updated about 1 month ago.

Status:
Solution proposée
Priority:
Normal
Assignee:
Category:
agent
Target version:
-
Start date:
12 Jun 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Chacune des briques logicielle apporte ses rôles techniques.
Or certaines configuration des briques peuvent dépendre des rôles des apportées par les autres briques.

Par exemple :
- le portail agent dépend du groupe "Agent" (il existe dans a2 mais il n'est pas encore récupéré par wcs)

$ combo-manage hobo_deploy https://agent-combo.dev.publik.love/ env.json --ignore-timestamp
Missing groups: Agent

- le template du portail utilisateur de signal-publik dépend de wcs :
$ combo-manage hobo_deploy https://combo.dev.publik.love/ env.json --ignore-timestamp
requests.exceptions.MissingSchema: Invalid URL 'api/formdefs/': No schema supplied.

peut-être aussi qu'il y ait de quoi différencier une erreur parce que les prérequis sont absents (ok) d'une autre erreur (pas ok).

0002-hobo_deploy-write-hobo.json-even-if-import_template-.patch View (3.09 KB) Nicolas Roche, 12 Jun 2019 08:22 PM


Related issues

Related to Hobo - Development #29762: ordonnancement des import-template Nouveau 15 Jan 2019

History

#1 Updated by Nicolas Roche 2 months ago

#2 Updated by Nicolas Roche 2 months ago

En fait, les commandes "hobo_deploy" qui échouent sont déjà retentées.
Plus explicitement dans le cadre de signal-publik évoqué dans ce ticket :
  • lors de l'import du portail agent, bien que le rôle "Agent" soit manquant, la commande "hobo_deploy" ne sort (en fait) pas en erreur.
  • pour le portail utilisateur, le problème a déjà été aperçu avec #33451 : lorsque "import_template" échoue, alors le fichier hobo.json n'est pas créé. Or il me semble que c'est justement ce qui provoque la seconde erreur :
      File "/home/nroche/src/combo/combo/apps/wcs/models.py", line 60, in save
        forms_response_json = get_wcs_json(wcs_site, 'api/formdefs/')
    

La modification proposée ajoute par ailleurs la commande "import_template" aux recettes de déploiement d'authentic (j'imagine que c'était un oubli).

#3 Updated by Christophe Siraut about 2 months ago

Plutôt que de déplacer self.configure_template(), peut-on y détecter l'exception sans la lever? (je n'ai pas vérifié si la commande est ensuite retentée)

#4 Updated by Nicolas Roche about 2 months ago

Oui, mais cependant j'ai déplacé l'appel non seulement pour décorréler l'import du déploiement mais également parce que l'import n'a rien de spécifique et devrait également pouvoir être utilisé avec authentic.

Actuellement il est lancé via la fonction deploy_specifics qui est potentiellement remplacée (comme c'est le cas avec authentic).

#5 Updated by Nicolas Roche about 1 month ago

  • Status changed from Solution proposée to Rejeté

#6 Updated by Benjamin Dauvergne about 1 month ago

Il y a une raison au rejet ?

#7 Updated by Nicolas Roche about 1 month ago

C'est parce que j'ai rejeté le ticket chapeau.
Maintenant ces sous tickets sont indépendants et pourraient effectivement intéresser quelqu'un.
J'ai peut-être été rejeté un peu vite.

#8 Updated by Nicolas Roche about 1 month ago

  • Status changed from Rejeté to Solution proposée

Also available in: Atom PDF