Merge lp:~stevenk/launchpad/force-checkwatches-production into lp:launchpad

Proposed by Steve Kowalik
Status: Merged
Approved by: Steve Kowalik
Approved revision: no longer in the source branch.
Merged at revision: 14541
Proposed branch: lp:~stevenk/launchpad/force-checkwatches-production
Merge into: lp:launchpad
Diff against target: 25 lines (+2/-7)
1 file modified
lib/lp/bugs/scripts/checkwatches/base.py (+2/-7)
To merge this branch: bzr merge lp:~stevenk/launchpad/force-checkwatches-production
Reviewer Review Type Date Requested Status
William Grant code Approve
Review via email: mp+86187@code.launchpad.net

Commit message

[r=wgrant][bug=906151] Remove checkwatches special error reporting so that it uses production to write out oopsii.

Description of the change

Remove checkwatches special error reporting so that it uses production to write out oopsii.

To post a comment you must log in.
Revision history for this message
William Grant (wgrant) :
review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/lp/bugs/scripts/checkwatches/base.py'
2--- lib/lp/bugs/scripts/checkwatches/base.py 2011-03-11 03:19:07 +0000
3+++ lib/lp/bugs/scripts/checkwatches/base.py 2011-12-19 04:39:24 +0000
4@@ -39,12 +39,6 @@
5 MAX_SQL_STATEMENTS_LOGGED = 10000
6
7
8-class CheckWatchesErrorUtility(ErrorReportingUtility):
9- """An error utility that for the checkwatches process."""
10-
11- _default_config_section = 'checkwatches'
12-
13-
14 def report_oops(message=None, properties=None, info=None,
15 transaction_manager=None):
16 """Record an oops for the current exception.
17@@ -91,7 +85,8 @@
18
19 # Create the dummy request object.
20 request = ScriptRequest(properties, url)
21- error_utility = CheckWatchesErrorUtility()
22+ error_utility = ErrorReportingUtility()
23+ error_utility.configure(section_name='checkwatches')
24 error_utility.raising(info, request)
25 return request
26