Merge lp:~cjohnston/ubuntu-ci-services-itself/generic-doc-fixes into lp:ubuntu-ci-services-itself

Proposed by Chris Johnston
Status: Merged
Approved by: Francis Ginther
Approved revision: 136
Merged at revision: 136
Proposed branch: lp:~cjohnston/ubuntu-ci-services-itself/generic-doc-fixes
Merge into: lp:ubuntu-ci-services-itself
Diff against target: 344 lines (+41/-42)
12 files modified
docs/api/initial-request.rst (+3/-3)
docs/components/branch-source-builder.rst (+4/-4)
docs/components/existing-pieces.rst (+2/-2)
docs/components/image-builder.rst (+2/-2)
docs/components/lander.rst (+8/-8)
docs/components/planned.rst (+0/-1)
docs/components/queue-service.rst (+1/-1)
docs/components/test-runner.rst (+2/-2)
docs/components/ticket-system.rst (+10/-10)
docs/introduction.rst (+3/-3)
docs/ticket.rst (+4/-4)
docs/usage.rst (+2/-2)
To merge this branch: bzr merge lp:~cjohnston/ubuntu-ci-services-itself/generic-doc-fixes
Reviewer Review Type Date Requested Status
Francis Ginther Approve
Review via email: mp+203151@code.launchpad.net

Commit message

Misc doc fixes across the board

Description of the change

Just general doc fixes across the board

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

Thanks for the fixes.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'docs/api/initial-request.rst'
2--- docs/api/initial-request.rst 2014-01-16 22:29:43 +0000
3+++ docs/api/initial-request.rst 2014-01-24 18:34:54 +0000
4@@ -10,7 +10,7 @@
5
6 ::
7
8- make_request autopilot_1.5~dev.1_source.changes --addpkgs python-autopilot python3-autopilot
9+ python ubuntu-ci create_ticket -t "Ticket name" -d "Ticket description" -b 123 -o user@example.com -a "foo,bar" -r "baz" -s /full/path/to/_source.changes -s /full/path/to/_source.changes
10
11 CLI -> Ticket System
12 --------------------
13@@ -264,14 +264,14 @@
14 -----------------------
15
16 The lander provides progress to the ticket system through the ticket
17-system's progress api:
18+system's progress API:
19
20 ::
21
22 TBD
23
24 Completion of a build is provided to the ticket system through the ticket
25-system's progress api:
26+system's progress API:
27
28 ::
29
30
31=== modified file 'docs/components/branch-source-builder.rst'
32--- docs/components/branch-source-builder.rst 2014-01-17 20:11:51 +0000
33+++ docs/components/branch-source-builder.rst 2014-01-24 18:34:54 +0000
34@@ -22,7 +22,7 @@
35 ************
36
37 * Lander sends a "build_source" request to the service (this includes swift
38- urls from the data-store on what to build)
39+ URLs from the data-store on what to build)
40
41 * A request is placed in the queue (at each step below, the progress_trigger
42 will be called to notify of status changes).
43@@ -46,12 +46,12 @@
44
45 Request a PPA build of the provided sources.
46
47-*Url Pattern*
48+*URL Pattern*
49
50 http://bsbuilder-url:8080/api/v1/build_source (HTTP POST)
51
52 *Parameters*
53- * source_packages: An array of data-store URL's containing all of the
54+ * source_packages: An array of data-store URLs containing all of the
55 source package files.
56 * ppa: The PPA allocated by the ppa-assigner for this operation.
57 * progress_trigger: A string used to create a dedicated message queue
58@@ -116,7 +116,7 @@
59 Useful for debug and monitoring. This method will return information like
60 the number of worker queues and if they are busy or not.
61
62-*Url Pattern*
63+*URL Pattern*
64
65 http://bsbuilder-url:8080/api/v1/status (HTTP GET)
66
67
68=== modified file 'docs/components/existing-pieces.rst'
69--- docs/components/existing-pieces.rst 2014-01-21 15:09:33 +0000
70+++ docs/components/existing-pieces.rst 2014-01-24 18:34:54 +0000
71@@ -16,7 +16,7 @@
72
73 **Project Manager**
74
75-* Project specific configuration - We are doing this now with lp:cupstream2distro-config but with a very jenkins focused implementation.
76+* Project specific configuration - We are doing this now with lp:cupstream2distro-config but with a very Jenkins focused implementation.
77
78 **Test Runner**
79
80@@ -25,7 +25,7 @@
81
82 **Image Building**
83
84-* Didier has done this manually for ISO images, using a squash FS process. Need to automate this and modify to work with touch images (which have an ubuntu and platform specific component). (takes about 15 minutes)
85+* Didier has done this manually for ISO images, using a squash FS process. Need to automate this and modify to work with touch images (which have an Ubuntu and platform specific component). (takes about 15 minutes)
86
87 **Ubuntu CD Image & Germinate**
88
89
90=== modified file 'docs/components/image-builder.rst'
91--- docs/components/image-builder.rst 2014-01-21 14:56:40 +0000
92+++ docs/components/image-builder.rst 2014-01-24 18:34:54 +0000
93@@ -12,7 +12,7 @@
94 Deployment
95 ----------
96
97-* Can run as a juju service.
98+* Can run as a Juju service.
99 * Needs relationship to the Lander and the Data Store.
100 * Operations are transient, no need to save state.
101 * Will need to have some configuration data though, in particular, for
102@@ -48,7 +48,7 @@
103 * base_image : {"image_type": IMAGE_TYPE, "url_list": [...], "series": SERIES}
104
105 * A json object containing the image_type (cloud for now), list of
106- URLs pointing to the image artifact(s), and the ubuntu series
107+ URLs pointing to the image artifact(s), and the Ubuntu series
108 name.
109
110 * ppa_list : [...]
111
112=== modified file 'docs/components/lander.rst'
113--- docs/components/lander.rst 2014-01-21 15:09:33 +0000
114+++ docs/components/lander.rst 2014-01-24 18:34:54 +0000
115@@ -26,7 +26,7 @@
116 Phase 0
117 -------
118
119-* Can run as a juju service.
120+* Can run as a Juju service.
121 * Needs relationship to Ticket System, PPA Assigner, Branch/Source Builder,
122 Image Builder and Test Runner.
123 * No public access needed.
124@@ -109,27 +109,27 @@
125
126 A Lander service handles the workflow by using Jenkins to schedule individual
127 tasks. When a request is received to the Lander from the Ticket System, it
128-triggers the master jenkins job with the request parameters. The master job
129+triggers the master Jenkins job with the request parameters. The master job
130 then triggers a series of child jobs to execute the workflow.
131
132 The child jobs themselves execute a service handler. The service handler is
133 responsible for setting up the progress queue and triggering the service
134 via its REST API. As the service handler runs, it outputs the progress updates
135-it receives to the console (to be viewed via jenkins) and pushes the progress
136+it receives to the console (to be viewed via Jenkins) and pushes the progress
137 update to the Ticket System.
138
139 When the service completes, the service handler closes with a return code
140 matching the status of the service itself. The result data from the service is
141-archived as a jenkins artifact. That data is combined with the existing set of
142-job parameters to be used as input to the next jenkins child job.
143+archived as a Jenkins artifact. That data is combined with the existing set of
144+job parameters to be used as input to the next Jenkins child job.
145
146 Only one master job may execute at a time. If additional build requests are
147-received, they will be queued by jenkins.
148+received, they will be queued by Jenkins.
149
150 The Lander supplies regular notification events to the Ticket System while a
151 job is executing. There are no notifications sent when the service is idle.
152
153-The Lander archives the jenkins console logs and the archived results of each
154+The Lander archives the Jenkins console logs and the archived results of each
155 job to the data store.
156
157 Future
158@@ -151,7 +151,7 @@
159 Schedules a new request for building source packages and creation and test of
160 an image.
161
162-*Url Pattern*
163+*URL Pattern*
164
165 http://lander-url:8080/api/v1/execute_request (HTTP POST)
166
167
168=== modified file 'docs/components/planned.rst'
169--- docs/components/planned.rst 2014-01-14 02:44:54 +0000
170+++ docs/components/planned.rst 2014-01-24 18:34:54 +0000
171@@ -5,7 +5,6 @@
172 :maxdepth: 2
173
174 ticket-system
175- ticket-manager
176 web-server
177 branch-listener
178 ppa-assigner
179
180=== modified file 'docs/components/queue-service.rst'
181--- docs/components/queue-service.rst 2013-12-09 20:06:06 +0000
182+++ docs/components/queue-service.rst 2014-01-24 18:34:54 +0000
183@@ -4,7 +4,7 @@
184 Purpose
185 *******
186
187-Provides a generic queue service for the entire CI engine. Common uses are:
188+Provides a generic queue service for the entire CI Engine. Common uses are:
189
190 * Task queues to manage work between a service accepting requests and a pool
191 of workers to actually execute the request.
192
193=== modified file 'docs/components/test-runner.rst'
194--- docs/components/test-runner.rst 2014-01-21 16:10:57 +0000
195+++ docs/components/test-runner.rst 2014-01-24 18:34:54 +0000
196@@ -10,7 +10,7 @@
197 Future
198 ======
199
200-Expands upon phase 0 by allowing a set of ppas and additional packages to be
201+Expands upon phase 0 by allowing a set of PPAs and additional packages to be
202 added before running the tests, support using autopilot on a touch device,
203 perform a custom test (TBD), support executing tests on bare metal.
204
205@@ -85,7 +85,7 @@
206
207 * Setup an instance from the image builder url
208 * Run the dep8 tests
209-* Collecte the artifacts
210+* Collect the artifacts
211 * Send test results and artifacts to the lander (via the data store).
212
213 Future
214
215=== modified file 'docs/components/ticket-system.rst'
216--- docs/components/ticket-system.rst 2014-01-21 19:36:04 +0000
217+++ docs/components/ticket-system.rst 2014-01-24 18:34:54 +0000
218@@ -442,14 +442,14 @@
219
220 ::
221
222- curl --dump-header - -H "Content-Type: application/json" -X POST --data '{"name": "Chris Johnston", "email": "chris.johnston@canonical.com", "is_team": "False"}' http://localhost:8000/api/v1/person/
223+ curl --dump-header - -H "Content-Type: application/json" -X POST --data '{"name": "Chris Johnston", "email": "user@example.com", "is_team": "False"}' http://localhost:8000/api/v1/person/
224
225
226 *team*
227
228 ::
229
230- curl --dump-header - -H "Content-Type: application/json" -X POST --data '{"name": "Canonical CI Engineering", "email": "canonical-ci-engineering@lists.launchpad.net", "is_team": "True"}' http://localhost:8000/api/v1/person/
231+ curl --dump-header - -H "Content-Type: application/json" -X POST --data '{"name": "Canonical CI Engineering", "email": "team@lists.example", "is_team": "True"}' http://localhost:8000/api/v1/person/
232
233
234 get_person
235@@ -465,19 +465,19 @@
236
237 ::
238
239- curl --dump-header - http://localhost:8000/api/v1/person/?name__exact=Chris%20Johnston
240- curl --dump-header - http://localhost:8000/api/v1/person/?name__iexact=chris%20johnston
241- curl --dump-header - http://localhost:8000/api/v1/person/?name__startswith=Chris
242- curl --dump-header - http://localhost:8000/api/v1/person/?name__istartswith=chris
243+ curl --dump-header - http://localhost:8000/api/v1/person/?name__exact=My%20Name
244+ curl --dump-header - http://localhost:8000/api/v1/person/?name__iexact=my%20name
245+ curl --dump-header - http://localhost:8000/api/v1/person/?name__startswith=My
246+ curl --dump-header - http://localhost:8000/api/v1/person/?name__istartswith=my
247
248 *search by email*
249
250 ::
251
252- curl --dump-header - http://localhost:8000/api/v1/person/?email__exact=chris.johnston@canonical.com
253- curl --dump-header - http://localhost:8000/api/v1/person/?email__iexact=Chris.Johnston@canonical.com
254- curl --dump-header - http://localhost:8000/api/v1/person/?email__startswith=chris
255- curl --dump-header - http://localhost:8000/api/v1/person/?email__istartswith=Chris
256+ curl --dump-header - http://localhost:8000/api/v1/person/?email__exact=user@example.com
257+ curl --dump-header - http://localhost:8000/api/v1/person/?email__iexact=User@example.com
258+ curl --dump-header - http://localhost:8000/api/v1/person/?email__startswith=user
259+ curl --dump-header - http://localhost:8000/api/v1/person/?email__istartswith=User
260
261 *show/don't show teams*
262
263
264=== modified file 'docs/introduction.rst'
265--- docs/introduction.rst 2014-01-23 19:06:13 +0000
266+++ docs/introduction.rst 2014-01-24 18:34:54 +0000
267@@ -28,12 +28,12 @@
268
269 The essiential set of Phase 0 features include:
270
271- * CI from debian source packages.
272+ * CI from Debian source packages.
273 * Building of binary packages and complete images.
274 * Tests are executed on the produced image.
275 * Tests are defined via autopackage testing.
276 * Results are archived and retrieved via a web interface.
277- * A micro-service oriented architecture deployed in an openstack cloud
278+ * A micro-service oriented architecture deployed in an OpenStack cloud
279 environment.
280
281 Constraints
282@@ -49,7 +49,7 @@
283 * Build and test results and logs are provided as raw artifacts.
284 * Source packages are used is input.
285 * Binary packages and images are built from a default series and image.
286- * Openstack cloud images are produced and used for testing.
287+ * OpenStack cloud images are produced and used for testing.
288 * Tests are limited to autopackage tests defined in the source packages.
289
290 Future Phases
291
292=== modified file 'docs/ticket.rst'
293--- docs/ticket.rst 2013-11-19 19:13:56 +0000
294+++ docs/ticket.rst 2014-01-24 18:34:54 +0000
295@@ -20,15 +20,15 @@
296 * knowing where we stands on all those feature branches and sources compared to latest in distro (if the source package isn't the latest version in the development version anymore)
297 * what MP are pending against this ticket, meaning against all attached feature branches.
298 * what delta do we have between the various feature branches in this ticket and their corresponding trunks.
299-* knowing if we are mergeable or not against those trunks. If we can merge at a T time (and tests are all passing), propose a way to merge trunks easily into those branches.
300+* knowing if we are able to merge or not against those trunks. If we can merge at a T time (and tests are all passing), propose a way to merge trunks easily into those branches.
301 * after getting the progress on their build, attaching latest available specific image (3 images per ticket): feature branch + fixed image number, feature branch + latest available image, trunk + feature branch merged + latest available image.
302 * knowing what latest image number is available, be able to change with it if test on latest image passed.
303 * getting tests progress while they are run dynamically. Represents them clearly against those previous 3 image tests
304 * ensuring that involved parties like core-devs and design team are involved if the ticket needs their review. A packaging change will require core-devs to ack their change. The design team will be in the process if there is a design change involved. Those should work through credentials.
305 * CI general health and global warnings if needed
306 * status on the corresponding components health
307-* gives an easy way once all those criterias are met (third-party acking, everything built and all tests passing) to give a "go" to the engine to deliver those different trunks.
308-* show the progress on the merging back to trunk, building packages there, tests passing, migration in UNAPPROVED/NEW, -proposed, release pocket and close the ticket completely once the next image is kicked in. Demonstrate explicitely when something is blocking there the whole pipeline for other delivery.
309+* gives an easy way once all those criteria are met (third-party acking, everything built and all tests passing) to give a "go" to the engine to deliver those different trunks.
310+* show the progress on the merging back to trunk, building packages there, tests passing, migration in UNAPPROVED/NEW, -proposed, release pocket and close the ticket completely once the next image is kicked in. Demonstrate explicitly when something is blocking there the whole pipeline for other delivery.
311
312 **Tickets interactions**
313
314@@ -39,7 +39,7 @@
315 Finally, in this global view, we want to show the health of all projects:
316
317 * seeing all components (projects/branches) that we have in the CI system with global/general metadata (what test environment is going to be used, what tests are associated with that components, number of tickets opened against them and so on)
318-* giving a view for the managers to see what ticket their team are working on, and what's the progress on them as well as global status (build/tests/mergeable to trunk).
319+* giving a view for the managers to see what ticket their team are working on, and what's the progress on them as well as global status (build/tests/ability to merge to trunk).
320 * if a direct commit to trunk blocked the project and that's the only way to fix it back (another direct commit to trunk), surface that. All other tickets being blocked by that state (as touching that same component) should reflect that info as well.
321 * when tickets are expected to be delivered (based on the ETA), so that we can identify hot spots (times where a lot of landing will happen simultaneously and will clash) and try to shuffle them around to not having them in one landing (eventually by a global override on all tickets)
322 * a single point to see across all projects where different teams need to assess/review before the delivery takes place (pending packaging changes triggering a core-dev review, design review needed)
323
324=== modified file 'docs/usage.rst'
325--- docs/usage.rst 2014-01-23 19:06:13 +0000
326+++ docs/usage.rst 2014-01-24 18:34:54 +0000
327@@ -10,7 +10,7 @@
328
329 CI requests, known as tickets, are created through a command line interface. ::
330
331- make_request my_source_package_1.0.1_source.changes --addpkgs my_binary_package
332+ python ubuntu-ci create_ticket -t "Ticket name" -d "Ticket description" -b 123 -o user@example.com -a "foo,bar" -r "baz" -s /full/path/to/_source.changes -s /full/path/to/_source.changes
333
334 This returns a ticket ID which can later be used to monitor and check for
335 results. Once the ticket is created, the Ubuntu CI Engine does the rest. The
336@@ -33,7 +33,7 @@
337 Specification of a source package
338 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
339
340-A source package is a standard debian source package which may optionally
341+A source package is a standard Debian source package which may optionally
342 contain dep8 autopackage tests. If autopackage tests are defined, they will
343 be used to validate the image that is produced. All tests must pass for a
344 ticket to complete CI successfully.

Subscribers

People subscribed via source and target branches