launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

Steve Kowalik has proposed merging lp:~stevenk/launchpad/ptj-oops-prefix into lp:launchpad.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)
Related bugs:
  Bug #805546 in Launchpad itself: "persontransferjob does not have a unique oops prefix"
  https://bugs.launchpad.net/launchpad/+bug/805546

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/ptj-oops-prefix/+merge/78799

Add error_dir and oops_prefix to two schemas for PersonTransferJobs.
-- 
https://code.launchpad.net/~stevenk/launchpad/ptj-oops-prefix/+merge/78799
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/ptj-oops-prefix into lp:launchpad.

=== modified file 'lib/canonical/config/schema-lazr.conf'
--- lib/canonical/config/schema-lazr.conf	2011-10-05 18:02:45 +0000
+++ lib/canonical/config/schema-lazr.conf	2011-10-10 07:25:26 +0000
@@ -2094,12 +2094,16 @@
 module: lp.registry.interfaces.persontransferjob
 dbuser: person-transfer-job
 crontab_group: MAIN
+error_dir: none
+oops_prefix: none
 
 [IPersonMergeJobSource]
 # This section is used by cronscripts/process-job-source.py.
 module: lp.registry.interfaces.persontransferjob
 dbuser: person-merge-job
 crontab_group: MAIN
+error_dir: none
+oops_prefix: none
 
 [IPlainPackageCopyJobSource]
 # This section is used by cronscripts/process-job-source.py.