Merge lp:~rodsmith/checkbox/ipmi_pxe_test_removal into lp:checkbox

Proposed by Rod Smith
Status: Rejected
Rejected by: Zygmunt Krynicki
Proposed branch: lp:~rodsmith/checkbox/ipmi_pxe_test_removal
Merge into: lp:checkbox
Diff against target: 87 lines (+2/-57)
3 files modified
checkbox-old/data/whitelists/server-selftest.whitelist (+0/-5)
checkbox-old/debian/changelog (+2/-1)
checkbox-old/jobs/miscellanea.txt.in (+0/-51)
To merge this branch: bzr merge lp:~rodsmith/checkbox/ipmi_pxe_test_removal
Reviewer Review Type Date Requested Status
Brendan Donegan (community) Needs Information
Review via email: mp+201860@code.launchpad.net

Commit message

Removed IPMI and PXE manual server tests because MAAS testing will cover this.

Description of the change

Removed IPMI and PXE manual server tests because MAAS testing will cover this.

This should affect only 14.04, not 12.04.

To post a comment you must log in.
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

Hi Rod,

Can you elaborate on the part about 'only 14.04'? Currently we don't have a good way to release changes to whitelists independently for different releases so if we merge this now, it's going to impact on 12.04 testing as well.

review: Needs Information
Revision history for this message
Rod Smith (rodsmith) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/16/2014 02:52 AM, Brendan Donegan wrote:
> Review: Needs Information
>
> Hi Rod,
>
> Can you elaborate on the part about 'only 14.04'? Currently we
> don't have a good way to release changes to whitelists
> independently for different releases so if we merge this now, it's
> going to impact on 12.04 testing as well.

Better hold off, then. Jeff wants those tests removed for 14.04, but
not for 12.04, so we need some way to do that. I'm still new enough
that I don't fully understand the build and release process for
checkbox, so I'm referring this to Jeff for his comments.

- --
Rod Smith
Server and Cloud Certification Engineer
<email address hidden>
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQEcBAEBAgAGBQJS19SwAAoJEFgyRI+V0FjmHKEH/2vsPNCz99m9JcbuBwqm1Hur
MxT0qbJopve9WcLQ1Jisr6LegWA1xGd8DQtJOcDodAxTvVqxDKCXmkBSRekmQ/B5
KPlIouusNvUqF1Q58h0xvjKz9ZDkDM8oetnA2oE45eb1BFpj0kWREQ007syXw8No
LWYWKV9jv+kWgsTCN6ilbXJjHGFoU6nGyzaBt836wxC4CZNWWoxUl9ShJtkjnp6O
wWXHXMl9ZSplQpiKgEnFSJQnBoj3keoCZR+e5IlGTki7JGfygETX+np2EePAIQMd
O8iodu3NhPohbUSggWd3FjQQnBN8WufJXCfIQWFWL0M403TmQmJaLgYAdCig2YY=
=9dd5
-----END PGP SIGNATURE-----

Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I'm rejecting this based on your comments. Please seek another solution and re-propose.

Unmerged revisions

2595. By Rod Smith

jobs/miscellanea.txt.in: Removed manual PXE and IPMI tests

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'checkbox-old/data/whitelists/server-selftest.whitelist'
2--- checkbox-old/data/whitelists/server-selftest.whitelist 2013-12-18 16:20:21 +0000
3+++ checkbox-old/data/whitelists/server-selftest.whitelist 2014-01-15 21:58:48 +0000
4@@ -44,11 +44,6 @@
5 # Moved the miscellanea to put manual tests at the beginning
6 __miscellanea__
7 miscellanea/tester-info
8-miscellanea/pxe_boot
9-miscellanea/pxe_boot_.*
10-miscellanea/remote_shared_ipmi
11-miscellanea/remote_shared_ipmi_.*
12-miscellanea/remote_dedicated_ipmi
13 miscellanea/fwts_test
14 miscellanea/fwts_results.log
15 miscellanea/ipmi_test
16
17=== modified file 'checkbox-old/debian/changelog'
18--- checkbox-old/debian/changelog 2014-01-10 11:02:47 +0000
19+++ checkbox-old/debian/changelog 2014-01-15 21:58:48 +0000
20@@ -1,6 +1,7 @@
21 checkbox (0.17.3) UNRELEASED; urgency=low
22
23- * Incremented changelog
24+ [ Rod Smith ]
25+ * jobs/miscellanea.txt.in: Removed manual PXE and IPMI tests
26
27 -- Brendan Donegan <brendan.donegan@canonical.com> Fri, 10 Jan 2014 11:02:47 +0000
28
29
30=== modified file 'checkbox-old/jobs/miscellanea.txt.in'
31--- checkbox-old/jobs/miscellanea.txt.in 2013-12-17 20:16:24 +0000
32+++ checkbox-old/jobs/miscellanea.txt.in 2014-01-15 21:58:48 +0000
33@@ -77,54 +77,3 @@
34 name: miscellanea/sources-list
35 command: sources_test $SOURCES_LIST "$REPOSITORIES"
36 _description: Checks that a specified sources list file contains the requested repositories
37-
38-plugin: local
39-name: miscellanea/pxe_boot
40-requires: device.category == 'NETWORK'
41-_description: Automated job to generate the PXE verification test for each NIC.
42-command:
43- cat <<'EOF' | run_templates -s 'udev_resource | filter_templates -w "category=NETWORK" | awk "/path: / { print \$2 }" | xargs -n 1 sh -c "for i in \`ls /sys\$0/net 2>/dev/null\`; do echo \$0 \$i; done"'
44- plugin: manual
45- name: miscellanea/pxe_boot_$2
46- description:
47- PURPOSE:
48- This test will verify that you have attempted to PXE boot this machine from
49- the network device $2.
50- STEPS:
51- 1. Prior to running this test, you should have attempted to boot this system via PXE on every Network Port available.
52- VERIFICATION:
53- 1. Select Yes if you successfully used PXE boot this system using the NIC $2
54- 2. Select No if you did not attempt to PXE boot this system using the NIC $2
55- 3. Select No if you attempted to PXE boot via $2 and it failed for some reason.
56- EOF
57-
58-plugin: local
59-name: miscellanea/remote_shared_ipmi
60-requires: device.category == 'NETWORK'
61-_description: Automated job to generate the Remote Shared IPMI verification test for each NIC.
62-command:
63- cat <<'EOF' | run_templates -s 'udev_resource | filter_templates -w "category=NETWORK" | awk "/path: / { print \$2 }" | xargs -n 1 sh -c "for i in \`ls /sys\$0/net 2>/dev/null\`; do echo \$0 \$i; done"'
64- plugin: manual
65- name: miscellanea/remote_shared_ipmi_$2
66- description:
67- PURPOSE:
68- This test will verify that you have attempted IPMI control of this machine from the network device $2.
69- STEPS:
70- 1. Prior to running this test, you should have configured and used IPMI to power this machine off and on using every Network Port available.
71- VERIFICATION:
72- 1. Select Yes if you successfully used IPMI to remotely power this system off and on using the NIC $2
73- 2. Select No if you did not attempt to use IPMI to remotely power this sytem off and on via the NIC $2
74- 3. Select No if you attempted to use IPMI to remotely power off/on this system via $2 and it failed for some reason.
75- EOF
76-
77-plugin: manual
78-name: miscellanea/remote_dedicated_ipmi
79-_description:
80- PURPOSE:
81- Some systems do not share IPMI over all NICs but instead have a dedicated management port directly connected to the BMC. This test verifies that you have used that port for remote IPMI connections and actions.
82- STEPS:
83- 1. Prior to running the test, you should have configured and used the Dedicated Management Port to remotely power off/on this sytem.
84- VERIFICATION:
85- Skip this test if this system ONLY uses shared management/ethernet ports OR if this system does not have a BMC (Management Console)
86- 1. Select Yes if you successfully used IPMI to remotely power this system off and on using the dedicated management port.
87- 2. Select No if you attempted to use the dedicated management port to remotely power this system off/on and it failed for some reason.

Subscribers

People subscribed via source and target branches