Bug #28263
hobo-deploy : appel à wcsctl convert-to-sql qui n'existe plus
0%
Description
#20410 a passé convert-to-sql en commande de management, sauf qu'on comptait sur la commande dans hobo-deploy, et plouf.
Traceback (most recent call last): File "/home/cazino/envs/publik-env/bin/wcsctl.py", line 7, in <module> exec(compile(f.read(), __file__, 'exec')) File "/home/cazino/src/wcs/wcsctl.py", line 11, in <module> ctl.run(sys.argv[1:]) File "/home/cazino/src/wcs/wcs/qommon/ctl.py", line 159, in run command_class = self.get_commands()[command] KeyError: 'convert-to-sql'
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-hobo-deploy-don-t-call-convert-to-sql-to-create-tabl.patch 0001-hobo-deploy-don-t-call-convert-to-sql-to-create-tabl.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
En tests locaux, ça le fait, mais ça me dit bien que quelqu'un vérifie un déroulé complet publik-devinst etc.
(pour mémoire, le test avant fonctionnait parce que l'appel à convert-to-sql était un subprocess.call, pas check_call)
(et il aurait été un check_call, vu que la commande utilisée est sys.argv0..., ça aurait échoué dans les tests sur un appel façon py.test convert-to-sql...)
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Tu peux supprimer un import subprocess
.
Tout ce que je peux dire c'est que "ça passe", le tenant est là.
Pas capable de de dire pour si tout est ok, il me manque par exemple les entrées formulaires et workflow dans la barre latérale (ça n'a peut-être absolument rien à voir avec ce ticket, je suis un peu empêtré dans #28231), mais ça m'est déjà arrivé que ces entrées apparaissent avec un peu de retard.
Bref j'y comprends rien, encore moins à comment on on pu faire des déploiements depuis #20410.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Quelques précisions plus utiles.
Après un déploiement de tenant, je me retrouve avec seulement 4 tables dans la base :
- sessions
- tracking_codes
- users
- wcs_meta
Normal !?
Mis à jour par Frédéric Péters il y a plus de 5 ans
Oui, les autres tables qui sont créées elles le sont au fur et à mesure de la création de formulaires. (mais ça n'a pas de rapport avec le menu latéral où il te manque des éléments).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Oui ok et j'ai refait un run avec wcs en mode pre #20410 pour m'en convaincre avant de lire ton commentaire, et c'est pareil au niveau des tables, donc allons y.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9f7d8aac973529bec5b6a02138df71587caf1fa7 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Nov 22 19:59:10 2018 +0100 hobo-deploy: don't call convert-to-sql to create tables (#28263)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
hobo-deploy: don't call convert-to-sql to create tables (#28263)