Merge lp:~ce-infrastructure/capomastro/capomastro-django-logs into lp:capomastro
Proposed by
Caio Begotti
Status: | Merged |
---|---|
Approved by: | Caio Begotti |
Approved revision: | 155 |
Merged at revision: | 153 |
Proposed branch: | lp:~ce-infrastructure/capomastro/capomastro-django-logs |
Merge into: | lp:capomastro |
Diff against target: |
92 lines (+62/-1) 3 files modified
capomastro/settings.py (+55/-0) debian/capomastro.logrotate (+1/-1) testing/provision-testing-environment (+6/-0) |
To merge this branch: | bzr merge lp:~ce-infrastructure/capomastro/capomastro-django-logs |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sheila Miguez (community) | Approve | ||
Review via email: mp+244496@code.launchpad.net |
Description of the change
Sheila, I'm asking you specifically to review it as I stole this setting from Poke and I think you know Django logging better than I do :-) in fact I just wanted to be on the safe side so we don't loose any Dajngo traces even though the log won't be too verbose at first.
To post a comment you must log in.
It looks good to me.
I should borrow from you and remove the AdminEmailHandler in my loggers since I'm not doing anything with that yet.