Bug #5039
hobos: configure postgresql
0%
Description
Dans le cas d'un déploiement avec un site-options avec postgresql=true, il devrait y avoir création et configuration de la DB (comme wcsinstd le fait).
Fichiers
Historique
Mis à jour par Frédéric Péters il y a plus de 9 ans
- Fichier 0001-hobo-configure-sql-support-if-required-5039.patch 0001-hobo-configure-sql-support-if-required-5039.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed mis à Oui
Mis à jour par Thomas Noël il y a plus de 9 ans
Je crois qu'il manque un return après le "failed to connect" ligne 178.
J'ajouterais un print de warning en cas de base déjà existante (ligne 187). En fait, dans le cas d'un DUPLICATE_DATABASE, si pub.cfg['postgresql'].get('database') est None, est-ce qu'il ne faudrait pas un message d'erreur ?
Enfin je ne suis pas sûr que la présence de postgresql=true dans le site_options soit bien testée...?
Mis à jour par Frédéric Péters il y a plus de 9 ans
- Fichier 0001-hobo-configure-sql-support-if-required-5039.patch 0001-hobo-configure-sql-support-if-required-5039.patch ajouté
Je crois qu'il manque un return après le "failed to connect" ligne 178.
Ouaip, ajouté.
J'ajouterais un print de warning en cas de base déjà existante (ligne 187). En fait, dans le cas d'un DUPLICATE_DATABASE, si pub.cfg['postgresql'].get('database') est None, est-ce qu'il ne faudrait pas un message d'erreur ?
Ça devrait être possible pour le code d'être appelé plusieurs fois, il y aurait alors l'existence de la base de donnée et ce serait tout à fait normal.
Je ne vois pas trop comment pub.cfg['postgresql'].get('database') pourrait être None (le champ "database" est obligatoire dans la page de paramétrage, et cfg['postgresql'] ne peut pas être créé en-dehors de cette page).
Enfin je ne suis pas sûr que la présence de postgresql=true dans le site_options soit bien testée...?
Ce n'était pas testé, j'ai ajouté ça.
Mis à jour par Thomas Noël il y a plus de 9 ans
Yep, ça va.
Je reste quand même un peu inquiet (par nature, sans doute) sur le fait de laisser un peu trop filer le cas "DUPLICATE_DATABASE".
Mis à jour par Frédéric Péters il y a plus de 9 ans
- Statut changé de En cours à Résolu (à déployer)
commit 83199773dce37e3d29abe6f609f91b3b1c8c1650 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Oct 31 14:06:44 2014 +0100 hobo: configure sql support if required (#5039)