Code review comment for lp:~fginther/britney/disable-boottest

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

« Back to merge proposal