Bug #66342
base adresse, le cron qui met à jour les rues ne tourne plus ?
0%
Description
En tout cas sur toutes les instances de recette, les données des rues ont été effacées et les connecteurs affichent le message « Les données des rues ne sont pas encore disponibles; elles devraient bientôt être téléchargées. ». Il est nécessaire de faire tourner la commande de synchronisation manuellement pour réparer.
Visible sur https://passerelle-validation.test.entrouvert.org/base-adresse/base-adresse/
Priorité haute parce que ça risque de se produire aussi en prod (?)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Nouveau à Rejeté
J'avais pas vu #66341, mettons que ce soit ça.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Rejeté à Nouveau
- Priorité changé de Haut à Normal
Plutôt je laisse ouvert, le bug décrit ici n'est pas le même que #66341, une fois ce dernier résolu il faudra passer ici dire que les rues sont bien réapparues partout, ou pas.
(aussi éventuellement comprendre pourquoi elles ont disparu)
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Lié à Bug #66341: Import base_adresse: en cas d'ajout de codes postaux, l'import ne se fait pas ajouté
Mis à jour par Thomas Noël il y a presque 2 ans
Ce matin on remonte que les rues ont à nouveau disparu en recette
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0001-ban-import-revert-last-optimizations-66342.patch 0001-ban-import-revert-last-optimizations-66342.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
seen_citycodes = set()
for department in departments:
....
if "Last-Modified" in ban_gz.headers:
# test 36H
... continue
...
for line in ban_file:
....
seen_citycodes.add(citycode)
....
....
....
self.get_streets_queryset().exclude(citycode__in=seen_citycodes).delete() # <-- ici, les données d'un département zappé sont supprimée
=> proposition: revert, mais on garde le changement ban_id null=True, les index, et les migrations associées
le patch: revert des 5 derniers commits de https://git.entrouvert.org/passerelle.git/log/passerelle/apps/base_adresse sur le sujet (squash en 1 seul), en gardant la modif schema+migration
Mis à jour par Lauréline Guérin il y a presque 2 ans
(pour la lisibilité on peut aussi garder les 5 reverts tels quels et ajouter un commit pour rétablir les modifs de schéma + migration ?)
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Lauréline Guerin a écrit :
(pour la lisibilité on peut aussi garder les 5 reverts tels quels et ajouter un commit pour rétablir les modifs de schéma + migration ?)
Le patch ne m'est pas illisible ainsi. J'allais presque dire "au contraire".
(et désolé Pierre... mais on va s'y remettre, y'a forcément des chemins d'optimisation de cette machinerie)
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
- Assigné à mis à Lauréline Guérin
commit da9b818451dc39ab528e98349dccfb09e366387a
Author: Lauréline Guérin <zebuline@entrouvert.com>
Date: Mon Jun 20 16:24:28 2022 +0200
ban import: revert last optimizations (#66342)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Lié à Bug #66572: IntegrityError: ERREUR: la valeur d'une clé dupliquée rompt la contrainte unique « base_adresse_streetmodel_reso... ajouté
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Lié à Development #67375: BAN: reprendre l'optimisation de l'import, qu'on a dû revert ajouté
ban import: revert last optimizations (#66342)
This reverts commit 7479f1d1430713de4304f1801590c80d5981b408.