Projet

Général

Profil

Bug #28263

hobo-deploy : appel à wcsctl convert-to-sql qui n'existe plus

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
22 novembre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

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

Révision 9f7d8aac (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

hobo-deploy: don't call convert-to-sql to create tables (#28263)

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

#2

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.

#3

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 !?

#4

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

#5

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.

#6

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)
#7

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

Formats disponibles : Atom PDF