Merge ~kissiel/plainbox-provider-checkbox:poweroff-autoresume into plainbox-provider-checkbox:master

Proposed by Maciej Kisielewski
Status: Merged
Approved by: Sylvain Pineau
Approved revision: e039c467cc9513549c2a2c90ba292aae05c6c725
Merged at revision: a2cbc59e55ed21f755f45a5db2df5695a2220315
Proposed branch: ~kissiel/plainbox-provider-checkbox:poweroff-autoresume
Merge into: plainbox-provider-checkbox:master
Diff against target: 118 lines (+6/-32)
3 files modified
jobs/daemons.txt.in (+0/-1)
jobs/power-management.txt.in (+6/-27)
jobs/stress.txt.in (+0/-4)
Reviewer Review Type Date Requested Status
Sylvain Pineau (community) Approve
Maciej Kisielewski Needs Resubmitting
Review via email: mp+330410@code.launchpad.net

Description of the change

this time the single poweroff / reboot tests

To post a comment you must log in.
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

Let's take the opportunity to also remove all package.name == 'upstart' from our job definitions.

Revision history for this message
Maciej Kisielewski (kissiel) wrote :

> Let's take the opportunity to also remove all package.name == 'upstart' from
> our job definitions.

Done in e039c46.

review: Needs Resubmitting
Revision history for this message
Sylvain Pineau (sylvain-pineau) wrote :

+1, thanks.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/jobs/daemons.txt.in b/jobs/daemons.txt.in
2index 362fee4..d528bdf 100644
3--- a/jobs/daemons.txt.in
4+++ b/jobs/daemons.txt.in
5@@ -29,7 +29,6 @@ _description: Test if the getty daemon is running when the package is installed.
6 plugin: shell
7 category_id: com.canonical.plainbox::daemons
8 id: daemons/init
9-requires: package.name == 'upstart'
10 command: pgrep -f '/sbin/init' >/dev/null
11 _description: Test if the init daemon is running when the package is installed.
12
13diff --git a/jobs/power-management.txt.in b/jobs/power-management.txt.in
14index c0d73da..2521f81 100644
15--- a/jobs/power-management.txt.in
16+++ b/jobs/power-management.txt.in
17@@ -31,7 +31,7 @@ estimated_duration: 2.0
18 _description: Attach log from fwts wakealarm test
19 command: [ -e ${PLAINBOX_SESSION_SHARE}/fwts-wakealarm.log ] && cat ${PLAINBOX_SESSION_SHARE}/fwts-wakealarm.log
20
21-plugin: user-interact-verify
22+plugin: shell
23 category_id: com.canonical.plainbox::power-management
24 id: power-management/poweroff
25 estimated_duration: 120.0
26@@ -39,22 +39,11 @@ depends: power-management/fwts_wakealarm
27 user: root
28 environ: PLAINBOX_SESSION_SHARE
29 requires:
30- package.name == 'upstart'
31 package.name == 'fwts'
32-command: pm_test poweroff --log-level=debug --log-dir=$PLAINBOX_SESSION_SHARE
33-flags: noreturn
34+command: pm_test --silent --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox poweroff --log-level=debug --log-dir=$PLAINBOX_SESSION_SHARE
35+flags: noreturn autorestart
36 _description:
37- PURPOSE:
38 This test will check the system's ability to power-off and boot.
39- STEPS:
40- 1. Press "Enter" to begin.
41- 2. The machine will shut down.
42- 3. Power the machine back on.
43- 4. After rebooting, wait for the test prompts to inform you that the test is complete.
44- 5. Once the test has completed, restart checkbox and select 'Re-run' when prompted.
45- VERIFICATION:
46- If the machine successfully shuts down and boots, select Pass, otherwise,
47- select Fail.
48
49 plugin: attachment
50 category_id: com.canonical.plainbox::power-management
51@@ -66,28 +55,18 @@ command:
52 _description:
53 This will attach any logs from the power-management/poweroff test to the results.
54
55-plugin: user-interact-verify
56+plugin: shell
57 category_id: com.canonical.plainbox::power-management
58 id: power-management/reboot
59 estimated_duration: 120.0
60 user: root
61 environ: PLAINBOX_SESSION_SHARE
62 requires:
63- package.name == 'upstart'
64 package.name == 'fwts'
65-command: pm_test reboot --log-level=debug --log-dir=$PLAINBOX_SESSION_SHARE
66-flags: noreturn
67+command: pm_test --silent --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox reboot --log-level=debug --log-dir=$PLAINBOX_SESSION_SHARE
68+flags: noreturn autorestart
69 _description:
70- PURPOSE:
71 This test will check the system's ability to reboot cleanly.
72- STEPS:
73- 1. Press "Enter" to begin.
74- 2. The machine will reboot.
75- 3. After rebooting, wait for the test prompts to inform you that the test is complete.
76- 4. Once the test has completed, restart checkbox and select Re-Run when prompted.
77- VERIFICATION:
78- If the machine successfully reboots, select Pass, otherwise,
79- select Fail.
80
81 plugin: attachment
82 category_id: com.canonical.plainbox::power-management
83diff --git a/jobs/stress.txt.in b/jobs/stress.txt.in
84index 37f58f8..9749a4e 100644
85--- a/jobs/stress.txt.in
86+++ b/jobs/stress.txt.in
87@@ -228,7 +228,6 @@ category_id: com.canonical.plainbox::stress
88 id: stress/reboot
89 estimated_duration: 4500.0
90 requires:
91- package.name == 'upstart'
92 package.name == 'fwts'
93 command: pm_test --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox -r 100 --silent --log-level=notset reboot --log-dir=$PLAINBOX_SESSION_SHARE
94 flags: noreturn autorestart
95@@ -250,7 +249,6 @@ plugin: shell
96 category_id: com.canonical.plainbox::stress
97 id: stress/reboot_30
98 requires:
99- package.name == 'upstart'
100 package.name == 'fwts'
101 executable.name == 'x-terminal-emulator'
102 command: pm_test --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox -r 30 --silent --log-level=notset reboot --log-dir=$PLAINBOX_SESSION_SHARE
103@@ -274,7 +272,6 @@ category_id: com.canonical.plainbox::stress
104 id: stress/poweroff
105 estimated_duration: 4500.0
106 requires:
107- package.name == 'upstart'
108 package.name == 'fwts'
109 executable.name == 'x-terminal-emulator'
110 command: pm_test --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox --r 100 --silent --log-level=notset poweroff --log-dir=$PLAINBOX_SESSION_SHARE
111@@ -297,7 +294,6 @@ plugin: shell
112 category_id: com.canonical.plainbox::stress
113 id: stress/poweroff_30
114 requires:
115- package.name == 'upstart'
116 package.name == 'fwts'
117 executable.name == 'x-terminal-emulator'
118 command: pm_test --checkbox-respawn-cmd $PLAINBOX_SESSION_SHARE/__respawn_checkbox --r 30 --silent --log-level=notset poweroff --log-dir=$PLAINBOX_SESSION_SHARE

Subscribers

People subscribed via source and target branches