On 18 February 2018 at 13:44, Eryu Guan <eguan@xxxxxxxxxx> wrote: > Previously _scratch_mount didn't check the mount status and most > tests continue to run even if the mount failed (unless test checks > for the mount status explicitly). This would result in running tests > on the underlying filesystem (usually rootfs) and implicit test > failures, and such failures can be annoying and are usually hard to > debug. > > Now _fail test by default if _scratch_mount failed and introduce > _try_scratch_mount for tests that need to check mount results > themselves. Also introduce a new _try_scratch_mount helper that does > scratch mount without checking mount status for tests that do the > check themselves. The message above doesn't look quite right, but the patch looks good. Reviewed-by: Andreas Gruenbacher <agruenba@xxxxxxxxxx> Thanks, Andreas -- To unsubscribe from this list: send the line "unsubscribe fstests" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
