Merge lp:~sil2100/ubuntu-archive-tools/in-progress-kernels into lp:ubuntu-archive-tools

Proposed by Łukasz Zemczak
Status: Merged
Merged at revision: 1108
Proposed branch: lp:~sil2100/ubuntu-archive-tools/in-progress-kernels
Merge into: lp:ubuntu-archive-tools
Diff against target: 21 lines (+11/-0)
1 file modified
kernel-sru-review (+11/-0)
To merge this branch: bzr merge lp:~sil2100/ubuntu-archive-tools/in-progress-kernels
Reviewer Review Type Date Requested Status
Steve Langasek Needs Fixing
Review via email: mp+326584@code.launchpad.net

Commit message

Add a check to kernel-sru-review that allows processing of promote-to-proposed In Progress bugs.

Description of the change

Add a check to kernel-sru-review that allows processing of promote-to-proposed In Progress bugs.

This basically checks if the In Progress task is assigned to the person running the tool. If yes, we continue, if not then it asks if the user is sure about that - since someone else might be working on it.

To post a comment you must log in.
1105. By Łukasz Zemczak

Whoops, typo

Revision history for this message
Steve Langasek (vorlon) wrote :

+1 with modifications

review: Needs Fixing
1106. By Łukasz Zemczak

Change wording, wrap lines.

Revision history for this message
Łukasz Zemczak (sil2100) wrote :

Steve, could you take a look if with these changes it's good to go?

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'kernel-sru-review'
--- kernel-sru-review 2017-03-17 12:17:00 +0000
+++ kernel-sru-review 2017-07-04 15:47:05 +0000
@@ -50,6 +50,17 @@
50 return {}50 return {}
51 if task.status == 'Confirmed':51 if task.status == 'Confirmed':
52 return {'proposed': task}52 return {'proposed': task}
53 elif task.status == 'In Progress':
54 if lp.me.self_link != task.assignee_link:
55 print("This bug is in progress and not assigned to you. Do you "
56 "still want to review \nit? [yN]".
57 end="")
58 sys.stdout.flush()
59 response = sys.stdin.readline()
60 if not response.strip().lower().startswith('y'):
61 raise KernelWorkflowError("Skipping bug %s" % bugnum)
62 return {'proposed': task}
63
53 raise KernelWorkflowError(64 raise KernelWorkflowError(
54 "Ignoring bug %s, not ready to promote-to-proposed"65 "Ignoring bug %s, not ready to promote-to-proposed"
55 % bugnum)66 % bugnum)

Subscribers

People subscribed via source and target branches