launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:py3-RunJobCommand into launchpad:master.

Commit message:
RunJobCommand: Treat oops_id as text

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/398376
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:py3-RunJobCommand into launchpad:master.

diff --git a/lib/lp/services/job/runner.py b/lib/lp/services/job/runner.py
index 4d302d5..5368050 100644
--- a/lib/lp/services/job/runner.py
+++ b/lib/lp/services/job/runner.py
@@ -451,7 +451,7 @@ class JobRunner(BaseJobRunner):
 class RunJobCommand(amp.Command):
 
     arguments = [(b'job_id', amp.Integer())]
-    response = [(b'success', amp.Integer()), (b'oops_id', amp.String())]
+    response = [(b'success', amp.Integer()), (b'oops_id', amp.Unicode())]
 
 
 def import_source(job_source_name):