Merge lp:~joetalbott/ubuntu-ci-services-itself/branch_listener_update into lp:ubuntu-ci-services-itself

Proposed by Joe Talbott
Status: Merged
Approved by: Francis Ginther
Approved revision: 9
Merged at revision: 10
Proposed branch: lp:~joetalbott/ubuntu-ci-services-itself/branch_listener_update
Merge into: lp:ubuntu-ci-services-itself
Diff against target: 78 lines (+35/-28)
2 files modified
docs/api/branch-listener.rst (+1/-28)
docs/components/branch-listener.rst (+34/-0)
To merge this branch: bzr merge lp:~joetalbott/ubuntu-ci-services-itself/branch_listener_update
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+196627@code.launchpad.net

Commit message

branch-listener - Flesh out the design document a bit.

Description of the change

branch-listener - Flesh out the design document a bit.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

Approve.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'docs/api/branch-listener.rst'
2--- docs/api/branch-listener.rst 2013-11-20 14:40:42 +0000
3+++ docs/api/branch-listener.rst 2013-11-25 21:53:31 +0000
4@@ -1,31 +1,4 @@
5 Branch listener
6 ===============
7
8-Inbound API
9------------
10-
11-**/listen**
12- GET variables
13-
14- * branch - Branch to check for MPs. (required)
15- * notification URI - URI to send notifications to. (required)
16-
17-Outbound API
18-------------
19-
20-**<URI>/get_constraints**
21- GET variables
22-
23- * branch - Branch changes occurred on. (required)
24-
25-**<URI>/mp_status**
26- GET variables
27-
28- * branch - Branch
29- * status - Whether the MP meets the constraints for the branch. One of 'invalid', 'valid'
30-
31-Services Used
32--------------
33-
34-* LP API
35-* Ticket Manager API
36+To Be Determined.
37
38=== modified file 'docs/components/branch-listener.rst'
39--- docs/components/branch-listener.rst 2013-11-16 10:12:08 +0000
40+++ docs/components/branch-listener.rst 2013-11-25 21:53:31 +0000
41@@ -4,3 +4,37 @@
42 *Purpose:*
43
44 * Track MPs and the trunks for the components involved in a ticket. tarmac-ish.
45+
46+
47+Requirements
48+------------
49+
50+https://bugs.launchpad.net/tarmac/+bug/1253770
51+
52+* Determine when an MP has been updated and run tests accordingly.
53+* Given a trunk branch, return a list of all MPs available.
54+* Given an MP and a revision ID, check and merge.
55+* Apply tests to non-Approved branches and after any update/commit.
56+
57+Tarmac meets most of our needs.
58+
59+Service Requirements (Juju)
60+~~~~~~~~~~~~~~~~~~~~~~~~~~~
61+
62+* Read access to LP.
63+* MP comment write access to LP.
64+* Tarmac installed (currently the stable release is being investigated but a fork might be in order)
65+
66+Proposed API
67+------------
68+
69+Inbound
70+~~~~~~~
71+
72+* check_branch/?branch=<branch>[&revision=<revision>]
73+* merge_branch/?branch=<branch>[&revision=<revision>]
74+
75+Outbound
76+~~~~~~~~
77+
78+Currently all outbound APIs will be those tarmac already uses.

Subscribers

People subscribed via source and target branches