Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEATURE] Pouvoir logger en debug sur un scope réduit (PIX-15534) #10702

Merged
merged 1 commit into from
Dec 4, 2024

Conversation

nlepage
Copy link
Member

@nlepage nlepage commented Dec 3, 2024

🎄 Problème

En prod, on aimerait avoir des logs de niveau debug pour un ou des scopes particuliers.

🎁 Proposition

Permettre la création de loggers enfants appartenants à une section, et pouvoir activer les logs de niveau debug pour une ou plusieurs sections.

🧦 Remarques

Voir le commit ac426a5 pour l’utilisation.

🎅 Pour tester

Sur la RA #10679, vérifier que le niveau de log est info, mais qu’on a bien des logs de niveau debug (20) pour la section "learngingcontent".
Pour faire apparaitre ces logs, faire un rafraichissement du cache sur Pix Admin.
Des logs de debug avec le message "'clearing cache" doivent apparaitre.
Puis sur Pix App passer des compétences ou une certif...
Des logs de debug avec le message "loading from PG" doivent apparaitre.

@nlepage nlepage added 👀 Tech Review Needed cross-team Toutes les équipes de dev labels Dec 3, 2024
@nlepage nlepage self-assigned this Dec 3, 2024
@pix-bot-github
Copy link

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

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

@nlepage nlepage force-pushed the pix-15534-child-logger branch from 5c3abca to ebb902c Compare December 3, 2024 14:26
@nlepage nlepage force-pushed the pix-15534-child-logger branch from 99aa7ad to eadc7db Compare December 4, 2024 10:08
@pix-service-auto-merge pix-service-auto-merge merged commit e025664 into dev Dec 4, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants