Merge lp:~deryck/launchpad/fix-checkwatches-logging-661441 into lp:launchpad
Proposed by
Deryck Hodge
Status: | Merged |
---|---|
Approved by: | Robert Collins |
Approved revision: | no longer in the source branch. |
Merged at revision: | 11889 |
Proposed branch: | lp:~deryck/launchpad/fix-checkwatches-logging-661441 |
Merge into: | lp:launchpad |
Diff against target: |
34 lines (+2/-4) 2 files modified
lib/lp/bugs/doc/checkwatches.txt (+1/-3) lib/lp/bugs/scripts/checkwatches/base.py (+1/-1) |
To merge this branch: | bzr merge lp:~deryck/launchpad/fix-checkwatches-logging-661441 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Robert Collins (community) | Approve | ||
Review via email: mp+40367@code.launchpad.net |
Commit message
[r=lifeless]
Description of the change
There are duplicate checkwatches OOPS being reported because checkwatches also logs to error when a problem occurs. This branch fixes that by logging to info instead of error. We do want to log something, since the checkwatches logs are often used to debug issues, but logging to info will allow us to collect the info while also preventing the duplicate OOPS reports.
To post a comment you must log in.