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

Revision history for this message
Gavin Panella (allenap) wrote :

> This doesn't comment a bug, or rational for removing this support.

Good point. I've linked to 3 related bugs.

Rationale: this feature is not ready.

It has no automated tests, is attracting a disproportionate number of bugs, and doesn't work by most/all accounts. It should not have been landed in the first place. For an optimisation it's causing too many issues, so we (Julian, Raphael, Martin, myself) agreed this morning to remove it. We can reintroduce it, or its successor, once it's rock-solid outside of MAAS.

« Back to merge proposal