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.

Sign up for GitHub

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

feat(logs): make celery logs structured #17603

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
+40 −0
Draft

feat(logs): make celery logs structured #17603

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
47639ed
feat(logs): make celery logs structured
JacobCoffee Feb 13, 2025
f1248ea
fix: add missing logger for app trace logs
JacobCoffee Feb 13, 2025
c6ff14c
tests: adjust logging tests
JacobCoffee Feb 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
feat(logs): make celery logs structured
@JacobCoffee
JacobCoffee committed Feb 13, 2025

Verified

This commit was signed with the committer’s verified signature.
DarkaMaul dm
SSH Key Fingerprint: 1j1fAhrHrse4Sl9tud1sadiDAn4wHsjQcPJvZBbf1Yw
Verified
Learn about vigilant mode
commit 47639ed4a73d6aeac10c25e1a8d8c8e0078ba747
15 changes: 15 additions & 0 deletions warehouse/logging.py
View file
Original file line number Diff line number Diff line change
@@ -83,6 +83,21 @@ def includeme(config):
"handlers": ["primary"],
"level": config.registry.settings.get("logging.level", "INFO"),
},
"celery": {
"propagate": False,
"handlers": ["primary"],
"level": config.registry.settings.get("logging.level", "INFO"),
},
"celery.task": {
"propagate": False,
"handlers": ["primary"],
"level": config.registry.settings.get("logging.level", "INFO"),
},
"celery.worker": {
"propagate": False,
"handlers": ["primary"],
"level": config.registry.settings.get("logging.level", "INFO"),
},
},
"root": {
"level": config.registry.settings.get("logging.level", "INFO"),