common: no need to parse config file in new
authorEryu Guan <eguan@redhat.com>
Sun, 7 Feb 2016 22:27:14 +0000 (09:27 +1100)
committerDave Chinner <david@fromorbit.com>
Sun, 7 Feb 2016 22:27:14 +0000 (09:27 +1100)
I saw "new" failed the TEST_DEV/SCRATCH_DEV validation when TEST_DEV or
SCRATCH_DEV points to directory, which is needed in overlayfs testing.

"new" only generates test template and doesn't need common/config to
parse and validate TEST_DEV/SCRATCH_DEV etc., so just returns after
setting all needed commands.

Signed-off-by: Eryu Guan <eguan@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>
common/config

index 7a098ae1a5d3192e756f5fae0415da92b1c228e5..b5f6fd42c846af1763272a16332ad8d1884dacab 100644 (file)
@@ -270,6 +270,11 @@ fi
 rm -f /tmp/crc_check.img
 export XFS_MKFS_HAS_NO_META_SUPPORT
 
+# new doesn't need config file parsed, we can stop here
+if [ "$iam" == "new" ]; then
+       return 0
+fi
+
 _mount_opts()
 {
        case $FSTYP in