diff options
| author | Stephen Finucane <stephen@that.guru> | 2018-09-21 17:59:48 +0100 |
|---|---|---|
| committer | Stephen Finucane <stephen@that.guru> | 2018-09-24 11:26:00 +0100 |
| commit | aa266a28c6a022ad6952fac8f36afff0c540dccf (patch) | |
| tree | 5e2694cea70fad10fae3f11efa14db352f8da6df /patchwork/settings | |
| parent | ab0c443691c4f97b945fbb0d39261e51f5a1d74a (diff) | |
| download | patchwork-aa266a28c6a022ad6952fac8f36afff0c540dccf.tar patchwork-aa266a28c6a022ad6952fac8f36afff0c540dccf.tar.gz | |
settings: Don't configure logging for parsearchive
A recent change added additional ERROR level logs to the 'parsearchive'
tool. This highlighted an issue, whereby we had configured all modules
in 'patchwork.management.command' to email administrators on ERROR logs.
We clearly shouldn't be doing this for the 'parsearchive' command or for
anything other than 'parsemail', so fix this.
Along the way, we remove a now-unnecessary 'extra' argument to one of
the logging calls in 'parsearchive' and resolve a pep8 issue.
Signed-off-by: Stephen Finucane <stephen@that.guru>
Fixes: 133091da ("parsearchive: Fix logging")
Cc: Daniel Axtens <dja@axtens.net>
Diffstat (limited to 'patchwork/settings')
| -rw-r--r-- | patchwork/settings/base.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/patchwork/settings/base.py b/patchwork/settings/base.py index 3eb1f0e..1a62404 100644 --- a/patchwork/settings/base.py +++ b/patchwork/settings/base.py @@ -185,7 +185,7 @@ LOGGING = { 'level': 'WARNING', 'propagate': False, }, - 'patchwork.management.commands': { + 'patchwork.management.commands.parsemail': { 'handlers': ['console', 'mail_admins'], 'level': 'WARNING', 'propagate': True, |