notification-api icon indicating copy to clipboard operation
notification-api copied to clipboard

Adding debug stack trace feature to celery tasks

Open ben851 opened this issue 1 year ago • 0 comments

Summary | Résumé

I've added a debug stack trace task to celery that can be enabled/disabled using the environment variable FF_DEBUG_STACK_TRACE.

When enabled, a celery task will be created that will raise an exception in the same manner as @sastels changes to admin.

Test instructions | Instructions pour tester la modification

  • Set FF_DEBUG_STACK_TRACE to true
  • View the celery logs and make sure the stack trace is shown
  • Set FF_DEBUG_STACK_TRACE to false
  • View the celery logs and make sure the stack trace is not shown, and that the task does not execute at all.

Release Instructions | Instructions pour le déploiement

None.

Reviewer checklist | Liste de vérification du réviseur

  • [ ] Is the code maintainable? | Est-ce que le code peut être maintenu?
  • [ ] Have you tested it? | L’avez-vous testé?
  • [ ] Are there automated tests? | Y a-t-il des tests automatisés?
  • [ ] Does this cause automated test coverage to drop? | Est-ce que ça entraîne une baisse de la quantité de code couvert par les tests automatisés?
  • [ ] Does this break existing functionality? | Est-ce que ça brise une fonctionnalité existante?
  • [ ] Does this change the privacy policy? | Est-ce que ça entraîne une modification de la politique de confidentialité?
  • [ ] Does this introduce any security concerns? | Est-ce que ça introduit des préoccupations liées à la sécurité?
  • [ ] Does this significantly alter performance? | Est-ce que ça modifie de façon importante la performance?
  • [ ] What is the risk level of using added dependencies? | Quel est le degré de risque d’utiliser des dépendances ajoutées?
  • [ ] Should any documentation be updated as a result of this? (i.e. README setup, etc.) | Faudra-t-il mettre à jour la documentation à la suite de ce changement (fichier README, etc.)?

ben851 avatar Jul 26 '23 15:07 ben851