Bug #27987
convert_to_sql échoue quand site-options.cfg est absent
0%
Description
Suite à #22794 dans lequel on a ajouté l'adaptation de site-options.cfg, l'opération échoue quand le fichier n'est pas présent ou n'a pas de section options.
Par exemple :
$ sudo -u wcs wcs-manage convert_to_sql -d demo.test.au-quotidien.com --database wcs_demo_test_au_quotidien_com converting users converting test date##################################################################### ] 80% converting Demande d'intervention […] Traceback (most recent call last): File "/usr/lib/wcs/manage.py", line 10, in <module> execute_from_command_line(sys.argv) File "/usr/lib/python2.7/dist-packages/django/core/management/__init__.py", line 354, in execute_from_command_line utility.execute() File "/usr/lib/python2.7/dist-packages/django/core/management/__init__.py", line 346, in execute self.fetch_command(subcommand).run_from_argv(self.argv) File "/usr/lib/python2.7/dist-packages/django/core/management/base.py", line 394, in run_from_argv self.execute(*args, **cmd_options) File "/usr/lib/python2.7/dist-packages/django/core/management/base.py", line 445, in execute output = self.handle(*args, **options) File "/usr/lib/python2.7/dist-packages/wcs/ctl/management/commands/convert_to_sql.py", line 55, in handle self.enable_connection() File "/usr/lib/python2.7/dist-packages/wcs/ctl/management/commands/convert_to_sql.py", line 77, in enable_connection self.publisher.site_options.set('options', 'postgresql', 'true') File "/usr/lib/python2.7/ConfigParser.py", line 396, in set raise NoSectionError(section) ConfigParser.NoSectionError: No section: 'options'
Est-ce que dans ce cas un site-options.cfg minimal est suffisant? :
[options] postgresql = true
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 5 ans
Christophe Siraut a écrit :
Est-ce que dans ce cas un site-options.cfg minimal est suffisant? :
Yep (comme je te disais trop rapidement via jabber, c'est l'étape 0, "poser des site-options.cfg")
On peut aller un peu plus loin que le minimal :
[options] postgresql = true default-to-global-view = false workflow-global-actions = true workflow-criticality-levels = true formdef-captcha-option = false workflow-resubmit-action = false nominatim_url = https://passerelle-publik.entrouvert.com/base-adresse/ban
Mis à jour par Christophe Siraut il y a plus de 5 ans
On peut aller un peu plus loin que le minimal :
<snip>
Mais ces options dépassent la fonctionnalité conversion vers postgresql,
elles relèvent de la gestion d'une configuration par défaut, ce qui ne me
semble pas exister dans le code actuel.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Fichier 0001-convert-to-sql-add-options-section-if-it-doesn-t-exi.patch 0001-convert-to-sql-add-options-section-if-it-doesn-t-exi.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ces commentaires me font perdre l'idée d'un objectif à ce ticket.
Voici donc de quoi le fermer.
Mis à jour par Christophe Siraut il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
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)
(en corrigeant le double point)
commit 16571e6ff89c7bac1f95fbc902440d860ce5487d Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Nov 14 14:43:42 2018 +0100 convert to sql: add [options] section if it doesn't exist (#27987)
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
convert to sql: add [options] section if it doesn't exist (#27987)