Development #32091
cityweb: ne pas compresser les fichiers avec les demandes
0%
Description
Avoir une option, activée par défaut, pour archiver les fichiers .xml avec les demandes.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-cityweb-add-option-to-zip-demand-files-32091.patch 0001-cityweb-add-option-to-zip-demand-files-32091.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai il y a environ 5 ans
Benjamin Dauvergne a écrit :
Le seul moyen de se prémunier des histoires d'upload/lecture concurrente c'est de créer le fichier dans un répertoire, de faire un fsync, puis de le déplacer dans celui où il sera lu, voir atomic_write() dans w.c.s.
Cela a été fait aussi dans le connecteur ActesWeb
.
Il faut donc effectivement revoir la façon d'écrire les fichiers et arrêter de faire du ZIP.
Je m'arrange avec Orléans qui utilise Cityweb en prod si l'impact est important pour eux, en sachant qu'ils récuperent déjà les fichiers via rsync.
Mis à jour par Serghei Mihai il y a environ 5 ans
On n'a pas de retour de Cityweb. Les échanges se font dans un seul sens.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Il n'y a pas de réponse à ma question principale: « Je ne vois aucune lien avec le ticket client lié, ni avec ces histoires de CRC. C'est pour quoi faire tout ce barouf ? »
Mis à jour par Serghei Mihai il y a environ 5 ans
My bad, j'ai fait le lien vers le mauvais ticket.
J'ai discuté avec Mik de cette demande par jabber en privé, en plus du classement des fichiers dans des répértoires.
Mis à jour par Serghei Mihai il y a environ 5 ans
En fait, la demande client est exprimée dans https://dev.entrouvert.org/issues/29501#note-20
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-cityweb-add-option-to-compress-demand-files-32091.patch 0001-cityweb-add-option-to-compress-demand-files-32091.patch ajouté
- Assigné à mis à Serghei Mihai
Renommage de l'option en "compress".
Mis à jour par Frédéric Péters il y a environ 5 ans
(sans juger du fond, l'option devrait plutôt être "bundle_as_zip" (verbose_name="Bundle generated files in a Zip file")).
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Dans le cas où on compresse j'éviterai de faire un fichier temporaire qu'ensuite on supprime, je mettrai directement le fichier dans le zip.
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
Ensuite il reste le problème du atomic_write, il faut faire comme dans actesweb, car canne vient récupérer les fichiers par sftp sur le serveur du sictiam, #32413 créé.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Sujet changé de cityweb: permettre de choisir si les fichiers avec les demandes doivent être archivés ou pas à cityweb: ne pas compresser les fichiers avec les demandes
Et donc confirmation dans #32421 qu'il n'y a pas besoin de compresser les fichiers.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Lié à Development #32413: avoir un outil pour écrire de manière atomique ajouté
Mis à jour par Serghei Mihai il y a environ 5 ans
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Ack, faut faire passer atomic_write maintenant.
Mis à jour par Serghei Mihai il y a environ 5 ans
- Fichier 0001-cityweb-do-not-compress-demand-files-32091.patch 0001-cityweb-do-not-compress-demand-files-32091.patch ajouté
- Statut changé de Solution validée à Solution proposée
Patch à jour avec vérification si le répértoire destination existe.
Mis à jour par Serghei Mihai il y a presque 5 ans
- Statut changé de Solution proposée à Résolu (à déployer)
Testé en local, les demandes sont bien créées en XML:
commit cf3957fb9280194b095b4cc939462a412d57210b (HEAD -> master, origin/master, origin/HEAD) Author: Serghei Mihai <smihai@entrouvert.com> Date: Mon Apr 15 16:13:19 2019 +0200 cityweb: do not compress demand files (#32091)
Mis à jour par Serghei Mihai il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
cityweb: do not compress demand files (#32091)