betagouv / ma-cantine

Accompagner les acteurs restau co dans la loi EGalim
https://ma-cantine.beta.gouv.fr/
MIT License
14 stars 9 forks source link

[Bug]: CSM Staging - erreur 500 lors de l'enregistrement d'une nouvelle fiche action #4439

Closed Tdauvet89 closed 1 day ago

Tdauvet89 commented 5 days ago

Ton prénom

Thomas

C'est de quel prio ?

{"label"=>"Bloquant, pas urgent (sprint prochain idéalement)"}

Le(s) url(s) où tu vois le bug

https://ma-cantine-staging.cleverapps.io/cms/snippets/data/wasteaction/add/

Quel type d'utilisateur est concerné ?

Qu'est-ce que tu veux faire ?

Je veux enregistrer une nouvelle fiche action sur le CMS staging

Qu'est-ce que t'as bloqué ?

J'ai cliqué sur le bouton nouvelle fiche Remplis un titre, un sous titre, les caractérisques et j'ai sauvegardé. Au clic sur le bouton, j'ai eu une erreur 500

image

Tu utilises quel navigateur ?

{"label"=>"Chrome"}

Les détails de ce bug sont ouvert au grand public. Cocher ce cas pour confirmer que t'as pas ajouté des infos qui pourraient identifier les utilisateurs.

raphodn commented 5 days ago

Ok zut c'est lié à #4383

psycopg2.errors.UniqueViolation: duplicate key value violates unique constraint "data_wasteaction_pkey" DETAIL: Key (id)=(1) already exists.

Je pense qu'en réimportant les données existantes, cela n'a pas pour autant mis à jour "l'index SQL de la table"

Solutions

A) lancer une commande manuelle qui répare : https://stackoverflow.com/a/44113124 / https://stackoverflow.com/questions/14589634/how-to-reset-the-sequence-for-ids-on-postgresql-tables#comment75277746_14589706

python manage.py sqlsequencereset data | python manage.py dbshell

B) faire une migration avec un RunSQL

Tdauvet89 commented 5 days ago

Infos complémentaires, je viens de refaire une création de fiche, je n'ai plus l'erreur 500. Le correctif est déjà déployé sur staging ?

raphodn commented 1 day ago

Non pas encore déployé, la PR est ici : #4440 J'ai l'erreur en local. Mais peut-être qu'elle n'a lieu qu'une fois ?