betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
17 stars 2 forks source link

Ajout d’un tag sentry lorsqu’un RDV ANTS est supprimé dans le job de synchro #4385

Closed adipasquale closed 3 months ago

adipasquale commented 3 months ago

cf #4318

Le tag permettra de filtrer les issues et events pour comprendre si le problème se produit uniquement / majoritairement / de temps en temps lors d’un delete - recreate. Ça devrait nous donner des infos pour la piste à suivre 🔍

Les breadcrumbs paraitraient plus sémantiquement adaptés mais si je comprends bien on ne peut pas filtrer les issues et events dessus.

Je n’ai pas testé en local en revanche il y a un test qui passe par ce bout de code. Je l’avais cassé dans un premier commit en appelant mal set_tags.

Ça me parait à peu près suffisant comme niveau de confiance pour déployer ce petit tooling facilement annulable. Mais c’est discutable !

sentry-sentry-incubateur-net[bot] commented 3 months ago

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

Did you find this useful? React with a 👍 or 👎

adipasquale commented 3 months ago
image

il semble que le problème ne soit pas le delete-recreate !