pix icon indicating copy to clipboard operation
pix copied to clipboard

[TECH] Retirer le package Hapi/sentry au profit de l'integration native de Sentry

Open xav-car opened this issue 1 year ago • 3 comments

:unicorn: Problème

Hapi/sentry n'est plus maintenu

:robot: Proposition

Utiliser l'integration native de Sentry

:rainbow: Remarques

RAS

:100: Pour tester

CI au vert, déploiement ok

Comment check l'integration Sentry en RA?

xav-car avatar Oct 14 '24 12:10 xav-car

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

pix-bot-github avatar Oct 14 '24 12:10 pix-bot-github

Je ne mets pas un approve car mon avis serait de : tranformer cette PR et retirer completement Sentry de l'API. Il y avait eu un sondage sur le sujet, est-ce que il y a des gens qui utilisent vraiment Sentry sur l'API ?

Steph0 avatar Oct 14 '24 16:10 Steph0

Je ne mets pas un approve car mon avis serait de : tranformer cette PR et retirer completement Sentry de l'API. Il y avait eu un sondage sur le sujet, est-ce que il y a des gens qui utilisent vraiment Sentry sur l'API ?

C'était plus dans le but de supprimer un package plus maintenu. Il me semble qu'il est utilisé encore. un point avec les captains sera fait pour connaitre l'avenir de cette PR.

Après il y a toujours le flags à poser à false si on ne l'utilise pas avant de supprimer tout.

xav-car avatar Oct 16 '24 08:10 xav-car

J'ai fait le rebase de cette PR, mais effectivement, la PR #9870 pour la suppression de sentry est passée... Donc celle-ci pourrait être fermée il me semble, vous en pensez quoi ?

yaf avatar Jan 07 '25 15:01 yaf