Merge lp:~fginther/britney/disable-boottest into lp:~ubuntu-release/britney/britney2-ubuntu

Proposed by Francis Ginther
Status: Merged
Merged at revision: 510
Proposed branch: lp:~fginther/britney/disable-boottest
Merge into: lp:~ubuntu-release/britney/britney2-ubuntu
Diff against target: 12 lines (+1/-1)
1 file modified
britney.conf (+1/-1)
To merge this branch: bzr merge lp:~fginther/britney/disable-boottest
Reviewer Review Type Date Requested Status
kevin gunn (community) Approve
Iain Lane Approve
Review via email: mp+272442@code.launchpad.net

Commit message

Disable boottest.

Description of the change

Disable boottest.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

This is evidently because the current phone images are broken due to https://launchpad.net/bugs/1496434, and so *all* boottests are failing and are blocking promotion.

Revision history for this message
Iain Lane (laney) wrote :

I'm okay with doing this from the RT's POV.

This could have been handled better in a couple of ways.

1) once the CI team had pinned boottests to an old image version due to this bug, we should have known that the bug was actually being worked on at a high priority. That should only have been a stop-gap.
2) It seems networking is in the critical path for boottesting, so the system should make sure that network (and anything else which is similarly critical) works with the new packages. n-m had been failing boottests for a while, but apparently the bug in question wouldn't have been caught anyway.

Preferably someone from the phone product team should sign off on this request as the owners of the boottests, so I'll hold off on merging for now.

review: Approve
Revision history for this message
kevin gunn (kgunn72) wrote :

in pat's absence I'll give the +1 for Phone Product Team

Revision history for this message
kevin gunn (kgunn72) wrote :

> in pat's absence I'll give the +1 for Phone Product Team

adding an approve even

review: Approve
Revision history for this message
Martin Pitt (pitti) wrote :

FTR, this is currently being discussed between the QA team (Jean-Baptiste and Evan), Steve Langasek, and me. We were all in favor of disabling this for multiple reasons, we just wanted to give Steve a chance to weigh in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'britney.conf'
2--- britney.conf 2015-08-26 14:25:25 +0000
3+++ britney.conf 2015-09-25 16:27:20 +0000
4@@ -70,7 +70,7 @@
5 # Swift base URL with the results (must be publicly readable and browsable)
6 ADT_SWIFT_URL = https://objectstorage.prodstack4-5.canonical.com/v1/AUTH_77e2ada1e7a84929a74ba3b87153c0ac
7
8-BOOTTEST_ENABLE = yes
9+BOOTTEST_ENABLE = no
10 BOOTTEST_DEBUG = yes
11 BOOTTEST_ARCHES = armhf amd64
12 BOOTTEST_FETCH = yes

Subscribers

People subscribed via source and target branches