Bug #58950
Crash sur delete_tenant
Début:
24 novembre 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
J'essaie un delete_tenant, ça veut pas parce que
psycopg2.OperationalError: database "........" is being accessed by other users DETAIL: There is 1 other session using the database.
Pas de souci avec ça, mais ensuite :
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/bin/wcsctl", line 11, in <module>
ctl.run(sys.argv[1:])
File "/usr/lib/python3/dist-packages/wcs/qommon/ctl.py", line 167, in run
return cmd.run(args, options)
File "/usr/lib/python3/dist-packages/wcs/qommon/ctl.py", line 71, in run
return self.execute(base_options, sub_options, args)
File "/usr/lib/python3/dist-packages/wcs/ctl/delete_tenant.py", line 45, in execute
self.delete_tenant(pub, sub_options, args)
File "/usr/lib/python3/dist-packages/wcs/ctl/delete_tenant.py", line 113, in delete_tenant
% (createdb_cfg['database'], psycopg2.errorcodes.lookup(e.pgcode)),
KeyError: 'database'
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Un tenant où ? Avec quoi dans le createdb_cfg ?
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
Sur la prod sudo -u wcs wcsctl delete_tenant formulaires.mesdemarches.labaule.fr (j'ai pas été patient et j'ai shooté la DB à la main, depuis il a été recréé).
Sur ce tenant nouvellement créé, la situation (je suppose que c'était pareil sur l'ancien) :
$ sudo -u wcs wcs-manage shell -d formulaires.mesdemarches.labaule.fr
...
In [1]: from quixote import get_publisher
In [2]: print(get_publisher().cfg['postgresql'].get('createdb-connection-params', {}))
{'host': 'XXXXXXXXXXXXXXXXXXX', 'port': '5432', 'user': 'wcs', 'password': 'XXXXXXXXXXXX', 'dbname': 'postgres'}
Mis à jour par Frédéric Péters il y a plus de 2 ans
Ok c'est bien le patch attaché cet endroit a du être zappé au moment s/database/dbname/.
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Et donc c'est raccord avec ton patch.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch ajouté
- Statut changé de En cours à Solution proposée
Version simplifiée.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch 0001-ctl-update-delete_tenant-error-message-for-new-db-ar.patch ajouté
Un peu trop simplifiée, voici.
Mis à jour par Emmanuel Cazenave il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 377496e8bca0b9df55251cf82e85659973dcf762
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed Nov 24 15:28:35 2021 +0100
ctl: update delete_tenant error message for new db arg name (#58950)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ctl: update delete_tenant error message for new db arg name (#58950)