* [PATCH] generic/566: use _require_xfs_quota_foreign to replace the hard -f option
@ 2019-09-10 11:36 Zorro Lang
[not found] ` <2fc62795-d932-de07-a9ed-695893ead8f8@cn.fujitsu.com>
0 siblings, 1 reply; 2+ messages in thread
From: Zorro Lang @ 2019-09-10 11:36 UTC (permalink / raw)
To: fstests
xfstests should decide if xfs_quota need the -f option by
_require_xfs_quota_foreign, not write the -f option after
$XFS_QUOTA_PROG manually. The later way will cause unexpected
error on an old system which xfsprogs doesn't support the -f option.
Signed-off-by: Zorro Lang <zlang@redhat.com>
---
tests/generic/566 | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/generic/566 b/tests/generic/566
index e8491e54..fb239a00 100755
--- a/tests/generic/566
+++ b/tests/generic/566
@@ -34,6 +34,7 @@ _supported_os Linux
_supported_fs generic
_require_scratch
_require_quota
+_require_xfs_quota_foreign
_require_user
rm -f $seqres.full
@@ -45,7 +46,7 @@ _qmount
dir="$SCRATCH_MNT/dummy"
mkdir -p $dir
chown $qa_user $dir
-$XFS_QUOTA_PROG -x -f -c "limit -g bsoft=100k bhard=100k $qa_user" $SCRATCH_MNT
+$XFS_QUOTA_PROG -x -c "limit -g bsoft=100k bhard=100k $qa_user" $SCRATCH_MNT
$XFS_IO_PROG -f -c 'pwrite -S 0x58 0 1m' $dir/foo >> $seqres.full
chown $qa_user "${dir}/foo"
--
2.20.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] generic/566: use _require_xfs_quota_foreign to replace the hard -f option
[not found] ` <2fc62795-d932-de07-a9ed-695893ead8f8@cn.fujitsu.com>
@ 2019-09-11 2:42 ` Zorro Lang
0 siblings, 0 replies; 2+ messages in thread
From: Zorro Lang @ 2019-09-11 2:42 UTC (permalink / raw)
To: Xu, Yang/徐 杨; +Cc: fstests
On Wed, Sep 11, 2019 at 10:19:10AM +0800, Xu, Yang/徐 杨 wrote:
> on 2019/09/10 19:36, Zorro Lang wrote:
>
> > xfstests should decide if xfs_quota need the -f option by
> > _require_xfs_quota_foreign, not write the -f option after
> > $XFS_QUOTA_PROG manually. The later way will cause unexpected
> > error on an old system which xfsprogs doesn't support the -f option.
>
>
> Hi Zorro
>
> I also sent a patch[1] about _require_xfs_quota_foreign but i miss
> $XFS_QUOTA_PROG have used -f option for non-xfs filesystem.
> And you may forget to modify the last $XFS_QUOTA_PRO. As below:
>
> $XFS_QUOTA_PROG -x -f -c 'report -hag' $SCRATCH_MNT >> $seqres.full
>
> [1]https://patchwork.kernel.org/patch/11137635/
Oh, I didn't notice that, we sent the patch nearly same time. As you've
sent a patch, you can keep working on that, ignore this patch.
But the '-f' options in the case must be removed, or the case will fail
on RHEL-7 XFS. Due rhel7 xfsprogs doesn't support -f option, but
_require_xfs_quota_foreign won't _notrun if the FSTYP is xfs.
Thanks,
Zorro
>
> Thanks
> Yang Xu
>
> >
> > Signed-off-by: Zorro Lang <zlang@redhat.com>
> > ---
> > tests/generic/566 | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/tests/generic/566 b/tests/generic/566
> > index e8491e54..fb239a00 100755
> > --- a/tests/generic/566
> > +++ b/tests/generic/566
> > @@ -34,6 +34,7 @@ _supported_os Linux
> > _supported_fs generic
> > _require_scratch
> > _require_quota
> > +_require_xfs_quota_foreign
> > _require_user
> > rm -f $seqres.full
> > @@ -45,7 +46,7 @@ _qmount
> > dir="$SCRATCH_MNT/dummy"
> > mkdir -p $dir
> > chown $qa_user $dir
> > -$XFS_QUOTA_PROG -x -f -c "limit -g bsoft=100k bhard=100k $qa_user" $SCRATCH_MNT
> > +$XFS_QUOTA_PROG -x -c "limit -g bsoft=100k bhard=100k $qa_user" $SCRATCH_MNT
> > $XFS_IO_PROG -f -c 'pwrite -S 0x58 0 1m' $dir/foo >> $seqres.full
> > chown $qa_user "${dir}/foo"
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2019-09-11 2:34 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-09-10 11:36 [PATCH] generic/566: use _require_xfs_quota_foreign to replace the hard -f option Zorro Lang
[not found] ` <2fc62795-d932-de07-a9ed-695893ead8f8@cn.fujitsu.com>
2019-09-11 2:42 ` Zorro Lang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.