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
=== modified file 'docs/api/branch-listener.rst'
--- docs/api/branch-listener.rst 2013-11-20 14:40:42 +0000
+++ docs/api/branch-listener.rst 2013-11-25 21:53:31 +0000
@@ -1,31 +1,4 @@
1Branch listener1Branch listener
2===============2===============
33
4Inbound API4To Be Determined.
5-----------
6
7**/listen**
8 GET variables
9
10 * branch - Branch to check for MPs. (required)
11 * notification URI - URI to send notifications to. (required)
12
13Outbound API
14------------
15
16**<URI>/get_constraints**
17 GET variables
18
19 * branch - Branch changes occurred on. (required)
20
21**<URI>/mp_status**
22 GET variables
23
24 * branch - Branch
25 * status - Whether the MP meets the constraints for the branch. One of 'invalid', 'valid'
26
27Services Used
28-------------
29
30* LP API
31* Ticket Manager API
325
=== modified file 'docs/components/branch-listener.rst'
--- docs/components/branch-listener.rst 2013-11-16 10:12:08 +0000
+++ docs/components/branch-listener.rst 2013-11-25 21:53:31 +0000
@@ -4,3 +4,37 @@
4*Purpose:*4*Purpose:*
55
6* Track MPs and the trunks for the components involved in a ticket. tarmac-ish. 6* Track MPs and the trunks for the components involved in a ticket. tarmac-ish.
7
8
9Requirements
10------------
11
12https://bugs.launchpad.net/tarmac/+bug/1253770
13
14* Determine when an MP has been updated and run tests accordingly.
15* Given a trunk branch, return a list of all MPs available.
16* Given an MP and a revision ID, check and merge.
17* Apply tests to non-Approved branches and after any update/commit.
18
19Tarmac meets most of our needs.
20
21Service Requirements (Juju)
22~~~~~~~~~~~~~~~~~~~~~~~~~~~
23
24* Read access to LP.
25* MP comment write access to LP.
26* Tarmac installed (currently the stable release is being investigated but a fork might be in order)
27
28Proposed API
29------------
30
31Inbound
32~~~~~~~
33
34* check_branch/?branch=<branch>[&revision=<revision>]
35* merge_branch/?branch=<branch>[&revision=<revision>]
36
37Outbound
38~~~~~~~~
39
40Currently all outbound APIs will be those tarmac already uses.

Subscribers

People subscribed via source and target branches