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

Proposed by Rod Smith
Status: Rejected
Rejected by: Zygmunt Krynicki
Proposed branch: lp://staging/~rodsmith/checkbox/ipmi_pxe_test_removal
Merge into: lp://staging/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://staging/~rodsmith/checkbox/ipmi_pxe_test_removal
Reviewer Review Type Date Requested Status
Brendan Donegan (community) Needs Information
Review via email: mp+201860@code.staging.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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches