Merge lp:~joetalbott/utah/document_reboot into lp:utah

Proposed by Joe Talbott
Status: Merged
Approved by: Javier Collado
Approved revision: 715
Merged at revision: 716
Proposed branch: lp:~joetalbott/utah/document_reboot
Merge into: lp:utah
Diff against target: 39 lines (+15/-2)
1 file modified
docs/source/introduction.rst (+15/-2)
To merge this branch: bzr merge lp:~joetalbott/utah/document_reboot
Reviewer Review Type Date Requested Status
Javier Collado (community) Approve
Joe Talbott (community) Needs Resubmitting
Review via email: mp+130447@code.launchpad.net

Description of the change

Add some notes to the documentation regarding reboot testcases.

To post a comment you must log in.
Revision history for this message
Javier Collado (javier.collado) wrote :

I see the same issues as in the other documentation review. I think we should
agree on when to use markup and when to use single quotes. I prefer markup, but
I'm fine with whatever is consistent.

lp:~joetalbott/utah/document_reboot updated
715. By Joe Talbott

Fix markup

Revision history for this message
Joe Talbott (joetalbott) wrote :

Should be fixed. I can't confirm locally.

review: Needs Resubmitting
Revision history for this message
Javier Collado (javier.collado) wrote :

@Joe

Looks good to me now. Thanks

Well, still there's /etc/rc.local that could be between double backquotes, but I leave it to your criteria.

review: Approve
lp:~joetalbott/utah/document_reboot updated
716. By Joe Talbott

Double backquote file name.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'docs/source/introduction.rst'
2--- docs/source/introduction.rst 2012-09-26 14:14:13 +0000
3+++ docs/source/introduction.rst 2012-10-19 19:31:22 +0000
4@@ -53,7 +53,7 @@
5 UTAH is very simple in terms of testing reliably and being able to reproduce a
6 test run in a consistent manner. A command line specifies the test
7 environment, type of machine to test on and a runlist. Then wait for it to finish and
8-analyse the results:
9+analyze the results:
10
11 .. image:: img/utah.jpg
12 :align: center
13@@ -62,7 +62,7 @@
14 by provisioning it with a particular provisioning method.
15
16 **WARNING: Your machine will be modified, not to interfere with your current
17-set-up or virtualised environment, so UTAH will create a new user called utah
18+set-up or virtualized environment, so UTAH will create a new user called utah
19 with sudo privileges to run the tests and have an inventory of existing VMs
20 that you could be interested in for running tests.**
21
22@@ -135,4 +135,17 @@
23 /usr/share/utah/client/examples/master.run \
24 'http://people.canonical.com/~max/max_test.run'
25
26+Running tests that need to reboot
27+=================================
28+
29+In a situation where a reboot is required. This is handled in UTAH by marking a testcase (in the ``tc_control`` file) with::
30+
31+ reboot: <always, pass, never>
32+
33+This will make the **utah client** reboot the system under test at the **end** of the testcase after any ``tc_cleanup`` command has finished. Any thing that needs to be tested after the reboot should be in the testcase immediately following the testcase marked for reboot.
34+
35+The possible choices are ``always``, ``pass``, and ``never``. ``never`` is the default and simply means the system under test will not reboot. ``pass`` means the reboot will only happen if the testcase passes. ``always``, means just that, the reboot will happen even if the test fails.
36+
37+Some things to keep in mind. The **utah-client** resumes by replacing ``/etc/rc.local`` with a file that will resume the current run and will resume with the testcase following the one that triggered the reboot. Also note that in order to have both pre- and post-reboot logging the ``-o`` flag should be used so that post-reboot output will be appended to the pre-reboot output.
38+
39 .. _Ubuntu Test Automation Harness: https://launchpad.net/utah

Subscribers

People subscribed via source and target branches