diff mbox

[3/4] xfs/045: can't change UUID on v5 filesystems.

Message ID 1430776893-25158-4-git-send-email-david@fromorbit.com (mailing list archive)
State New, archived
Headers show

Commit Message

Dave Chinner May 4, 2015, 10:01 p.m. UTC
From: Dave Chinner <dchinner@redhat.com>

So pass "-m crc=0" to the scratch_mkfs command so that we only run
on old v4 format filesystems where the UUID can be changed.

Signed-off-by: Dave Chinner <dchinner@redhat.com>
---
 tests/xfs/045 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Eric Sandeen May 4, 2015, 10:48 p.m. UTC | #1
On 5/4/15 5:01 PM, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
> 
> So pass "-m crc=0" to the scratch_mkfs command so that we only run
> on old v4 format filesystems where the UUID can be changed.
> 
> Signed-off-by: Dave Chinner <dchinner@redhat.com>

hm, given that I might "fix" this should we instead trap
on an xfs_db uuid failure, and _notrun the test?
(oh, but older xfs_db let it go, didn't it.  sigh).

I guess nothing about the presence of crc vs. non-crc
matters for this test, so turning it off by default is probably ok.
It could use a comment about why it's got "-m crc=0" though, at
least.

-Eric

> ---
>  tests/xfs/045 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/xfs/045 b/tests/xfs/045
> index 7d8a4a1..2f5508f 100755
> --- a/tests/xfs/045
> +++ b/tests/xfs/045
> @@ -50,7 +50,7 @@ _require_scratch_nocheck
>  echo "*** get uuid"
>  uuid=`_get_existing_uuid`
>  echo "*** mkfs"
> -if ! _scratch_mkfs_xfs >$tmp.out 2>&1
> +if ! _scratch_mkfs_xfs -m crc=0 >$tmp.out 2>&1
>  then
>      cat $tmp.out
>      echo "!!! failed to mkfs on $SCRATCH_DEV"
> 

--
To unsubscribe from this list: send the line "unsubscribe fstests" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/tests/xfs/045 b/tests/xfs/045
index 7d8a4a1..2f5508f 100755
--- a/tests/xfs/045
+++ b/tests/xfs/045
@@ -50,7 +50,7 @@  _require_scratch_nocheck
 echo "*** get uuid"
 uuid=`_get_existing_uuid`
 echo "*** mkfs"
-if ! _scratch_mkfs_xfs >$tmp.out 2>&1
+if ! _scratch_mkfs_xfs -m crc=0 >$tmp.out 2>&1
 then
     cat $tmp.out
     echo "!!! failed to mkfs on $SCRATCH_DEV"