Ubuntu CI Dashboard

Merge lp:~cjohnston/qa-dashboard/1-5mail-admins into lp:qa-dashboard

  1. 1-5mail-admins
  2. Merge into dev
Proposed by Chris Johnston
Status: Merged
Approved by: Chris Johnston
Approved revision: 595
Merged at revision: 595
Proposed branch: lp:~cjohnston/qa-dashboard/1-5mail-admins
Merge into: lp:qa-dashboard
Diff against target: 21 lines (+7/-1)
1 file modified
qa_dashboard/settings.py (+7/-1)
To merge this branch: bzr merge lp:~cjohnston/qa-dashboard/1-5mail-admins
Reviewer Review Type Date Requested Status
Andy Doan (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Review via email: mp+183775@code.launchpad.net

Commit message

Create a filter to disable emails to mail_admins when in debug mode

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:595
http://s-jenkins:8080/job/dashboard-ci/172/
Executed test runs:

Click here to trigger a rebuild:
http://s-jenkins:8080/job/dashboard-ci/172/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Andy Doan (doanac) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'qa_dashboard/settings.py'
2--- qa_dashboard/settings.py 2013-08-06 01:04:25 +0000
3+++ qa_dashboard/settings.py 2013-09-03 21:58:26 +0000
4@@ -218,10 +218,16 @@
5 LOGGING = {
6 'version': 1,
7 'disable_existing_loggers': False,
8+ 'filters': {
9+ 'require_debug_false': {
10+ '()': 'django.utils.log.RequireDebugFalse',
11+ },
12+ },
13 'handlers': {
14 'mail_admins': {
15 'level': 'ERROR',
16- 'class': 'django.utils.log.AdminEmailHandler'
17+ 'class': 'django.utils.log.AdminEmailHandler',
18+ 'filters': ['require_debug_false'],
19 }
20 },
21 'loggers': {

Subscribers

People subscribed via source and target branches