Merge lp:~abentley/launchpad/twisted-oopsdir into lp:launchpad
Proposed by
Aaron Bentley
Status: | Merged |
---|---|
Approved by: | Aaron Bentley |
Approved revision: | not available |
Merged at revision: | not available |
Proposed branch: | lp:~abentley/launchpad/twisted-oopsdir |
Merge into: | lp:launchpad |
Diff against target: |
109 lines (+47/-8) 2 files modified
lib/lp/services/job/runner.py (+5/-6) lib/lp/services/job/tests/test_runner.py (+42/-2) |
To merge this branch: | bzr merge lp:~abentley/launchpad/twisted-oopsdir |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code | Approve | |
Review via email: mp+19431@code.launchpad.net |
Commit message
Use correct OOPS dir for twisted job runner
To post a comment you must log in.
= Summary =
Use the correct oops directory for twisted job runner
== Proposed fix ==
Set the global error utility to the values from self.config_name.
== Pre-implementation notes ==
None
== Implementation details ==
We must set the global settings because scripts may not have write access to
the default error directory.
== Tests ==
bin/test test_runner TestJobCronScript
== Demo and Q/A == preview_ diffs.
Get a LOSA to add --twisted to the cron entry for update_
Cause the script to be run. Ensure that oopses go to the correct directory.
= Launchpad lint =
Checking for conflicts. and issues in doctests and templates.
Running jslint, xmllint, pyflakes, and pylint.
Using normal rules.
Linting changed files: services/ job/tests/ test_runner. py services/ job/runner. py
lib/lp/
lib/lp/
== Pylint notices ==
lib/lp/ services/ job/runner. py
35: [F0401] Unable to import 'lazr.delegates' (No module named delegates)