Merge lp:~fginther/britney/disable-boottest into lp:~ubuntu-release/britney/britney2-ubuntu
| 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 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| kevin gunn (community) | Approve on 2015-09-25 | ||
| Iain Lane | 2015-09-25 | Approve on 2015-09-25 | |
|
Review via email:
|
|||
Commit Message
Disable boottest.
Description of the Change
Disable boottest.
| Iain Lane (laney) wrote : | # |
| 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.
| kevin gunn (kgunn72) wrote : | # |
in pat's absence I'll give the +1 for Phone Product Team
| kevin gunn (kgunn72) wrote : | # |
> in pat's absence I'll give the +1 for Phone Product Team
adding an approve even
| 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.
This is evidently because the current phone images are broken due to https:/