Development #33874
"import_template" doit pouvoir au début échouer, puis être retentée
0%
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).
Fichiers
Demandes liées
Historique
Mis à jour par Nicolas Roche il y a presque 5 ans
- Lié à Development #29762: ordonnancement des import-template ajouté
Mis à jour par Nicolas Roche il y a presque 5 ans
- Fichier 0002-hobo_deploy-write-hobo.json-even-if-import_template-.patch 0002-hobo_deploy-write-hobo.json-even-if-import_template-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
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).
Mis à jour par Christophe Siraut il y a presque 5 ans
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)
Mis à jour par Nicolas Roche il y a presque 5 ans
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).
Mis à jour par Nicolas Roche il y a presque 5 ans
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.
Mis à jour par Frédéric Péters il y a 9 mois
- Statut changé de Solution proposée à Fermé
(cette histoire import-template se remplace/ra par des applications).