diff mbox

generic/47[23]: remove from auto/quick groups

Message ID 20180223050112.4090-1-david@fromorbit.com (mailing list archive)
State New, archived
Headers show

Commit Message

Dave Chinner Feb. 23, 2018, 5:01 a.m. UTC
From: Dave Chinner <dchinner@redhat.com>

generic/472 is for changes that are not upstream and seem dead in
the water at the moment, so remove this test from the auto and quick
groups until it's been resolved upstream and the changes merged.

generic/473 really doesn't seme useful. FIEMAP is a debugging
interface, xfs_io is a debugging tool and so trying to make every
filesytem report exactly the same thing for a ranged query just
strikes me as the wrong thing to be doing. This fails on XFS, and
there's no apparent resolution to that in sight, so remove the test
from the auto and quick, too.

Signed-Off-By: Dave Chinner <dchinner@redhat.com>
---
 tests/generic/group | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Xiao Yang Nov. 28, 2018, 7:12 a.m. UTC | #1
On 2018/02/23 13:01, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
>
> generic/472 is for changes that are not upstream and seem dead in
> the water at the moment, so remove this test from the auto and quick
> groups until it's been resolved upstream and the changes merged.
>
> generic/473 really doesn't seme useful. FIEMAP is a debugging
> interface, xfs_io is a debugging tool and so trying to make every
> filesytem report exactly the same thing for a ranged query just
> strikes me as the wrong thing to be doing. This fails on XFS, and
> there's no apparent resolution to that in sight, so remove the test
> from the auto and quick, too.
Hi Dave, Eryu

generic/473 with XFS got the mismatched extent info and was marked as a
broken test:
-------------------------------
Hole + Data
0: [0..127]: hole
-1: [128..255]: data
+1: [128..135]: data
-------------------------------

Is this a correct behavior? If not, is this issue going to be fixed in
the future?

Best Regards,
Xiao Yang
> Signed-Off-By: Dave Chinner <dchinner@redhat.com>
> ---
>  tests/generic/group | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/generic/group b/tests/generic/group
> index b2ce7a66ce81..1e808865772f 100644
> --- a/tests/generic/group
> +++ b/tests/generic/group
> @@ -474,8 +474,8 @@
>  469 auto quick
>  470 auto quick dax
>  471 auto quick rw
> -472 auto quick rw
> -473 auto quick
> +472 rw
> +473 broken
>  474 auto quick shutdown metadata
>  475 shutdown auto log metadata
>  476 auto rw
diff mbox

Patch

diff --git a/tests/generic/group b/tests/generic/group
index b2ce7a66ce81..1e808865772f 100644
--- a/tests/generic/group
+++ b/tests/generic/group
@@ -474,8 +474,8 @@ 
 469 auto quick
 470 auto quick dax
 471 auto quick rw
-472 auto quick rw
-473 auto quick
+472 rw
+473 broken
 474 auto quick shutdown metadata
 475 shutdown auto log metadata
 476 auto rw