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
=== modified file 'checkbox-old/data/whitelists/server-selftest.whitelist'
--- checkbox-old/data/whitelists/server-selftest.whitelist 2013-12-18 16:20:21 +0000
+++ checkbox-old/data/whitelists/server-selftest.whitelist 2014-01-15 21:58:48 +0000
@@ -44,11 +44,6 @@
44# Moved the miscellanea to put manual tests at the beginning44# Moved the miscellanea to put manual tests at the beginning
45__miscellanea__45__miscellanea__
46miscellanea/tester-info46miscellanea/tester-info
47miscellanea/pxe_boot
48miscellanea/pxe_boot_.*
49miscellanea/remote_shared_ipmi
50miscellanea/remote_shared_ipmi_.*
51miscellanea/remote_dedicated_ipmi
52miscellanea/fwts_test47miscellanea/fwts_test
53miscellanea/fwts_results.log48miscellanea/fwts_results.log
54miscellanea/ipmi_test49miscellanea/ipmi_test
5550
=== modified file 'checkbox-old/debian/changelog'
--- checkbox-old/debian/changelog 2014-01-10 11:02:47 +0000
+++ checkbox-old/debian/changelog 2014-01-15 21:58:48 +0000
@@ -1,6 +1,7 @@
1checkbox (0.17.3) UNRELEASED; urgency=low1checkbox (0.17.3) UNRELEASED; urgency=low
22
3 * Incremented changelog3 [ Rod Smith ]
4 * jobs/miscellanea.txt.in: Removed manual PXE and IPMI tests
45
5 -- Brendan Donegan <brendan.donegan@canonical.com> Fri, 10 Jan 2014 11:02:47 +00006 -- Brendan Donegan <brendan.donegan@canonical.com> Fri, 10 Jan 2014 11:02:47 +0000
67
78
=== modified file 'checkbox-old/jobs/miscellanea.txt.in'
--- checkbox-old/jobs/miscellanea.txt.in 2013-12-17 20:16:24 +0000
+++ checkbox-old/jobs/miscellanea.txt.in 2014-01-15 21:58:48 +0000
@@ -77,54 +77,3 @@
77name: miscellanea/sources-list77name: miscellanea/sources-list
78command: sources_test $SOURCES_LIST "$REPOSITORIES"78command: sources_test $SOURCES_LIST "$REPOSITORIES"
79_description: Checks that a specified sources list file contains the requested repositories79_description: Checks that a specified sources list file contains the requested repositories
80
81plugin: local
82name: miscellanea/pxe_boot
83requires: device.category == 'NETWORK'
84_description: Automated job to generate the PXE verification test for each NIC.
85command:
86 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"'
87 plugin: manual
88 name: miscellanea/pxe_boot_$2
89 description:
90 PURPOSE:
91 This test will verify that you have attempted to PXE boot this machine from
92 the network device $2.
93 STEPS:
94 1. Prior to running this test, you should have attempted to boot this system via PXE on every Network Port available.
95 VERIFICATION:
96 1. Select Yes if you successfully used PXE boot this system using the NIC $2
97 2. Select No if you did not attempt to PXE boot this system using the NIC $2
98 3. Select No if you attempted to PXE boot via $2 and it failed for some reason.
99 EOF
100
101plugin: local
102name: miscellanea/remote_shared_ipmi
103requires: device.category == 'NETWORK'
104_description: Automated job to generate the Remote Shared IPMI verification test for each NIC.
105command:
106 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"'
107 plugin: manual
108 name: miscellanea/remote_shared_ipmi_$2
109 description:
110 PURPOSE:
111 This test will verify that you have attempted IPMI control of this machine from the network device $2.
112 STEPS:
113 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.
114 VERIFICATION:
115 1. Select Yes if you successfully used IPMI to remotely power this system off and on using the NIC $2
116 2. Select No if you did not attempt to use IPMI to remotely power this sytem off and on via the NIC $2
117 3. Select No if you attempted to use IPMI to remotely power off/on this system via $2 and it failed for some reason.
118 EOF
119
120plugin: manual
121name: miscellanea/remote_dedicated_ipmi
122_description:
123 PURPOSE:
124 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.
125 STEPS:
126 1. Prior to running the test, you should have configured and used the Dedicated Management Port to remotely power off/on this sytem.
127 VERIFICATION:
128 Skip this test if this system ONLY uses shared management/ethernet ports OR if this system does not have a BMC (Management Console)
129 1. Select Yes if you successfully used IPMI to remotely power this system off and on using the dedicated management port.
130 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