Code review comment for lp:~allenap/maas/remove-squashfs

Revision history for this message
Julian Edwards (julian-edwards) wrote :

Dave, as Gavin says this feature is utterly broken right now, has no tests and I personally have never seen it work properly. I would be happy to re-instate it when the above are fixed, but I am under the impression (correct me if wrong) that you guys are working on something else anyway? In addition, I remember being told at UDS that it doesn't really speed up installation much over the d-i method.

I'll approve the change here on condition that Dave is ok with it too, based on my comments.

review: Approve

« Back to merge proposal