[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2 3/3] qemu-io-test: Disable Quorum test when n
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH V2 3/3] qemu-io-test: Disable Quorum test when not compiled in. |
Date: |
Sun, 23 Feb 2014 00:00:38 +0800 |
User-agent: |
Mutt/1.5.22 (2013-10-16) |
On Sat, 02/22 12:58, Benoît Canet wrote:
> The Saturday 22 Feb 2014 à 15:38:15 (+0800), Fam Zheng wrote :
> > On Sat, 02/22 02:20, Benoît Canet wrote:
> > > Quorum is not compiled by default: make the quorum 081 test aware of this.
> > >
> > > Signed-off-by: Benoit Canet <address@hidden>
> > > ---
> > > tests/qemu-iotests/081 | 3 +++
> > > 1 file changed, 3 insertions(+)
> > >
> > > diff --git a/tests/qemu-iotests/081 b/tests/qemu-iotests/081
> > > index f053f11..c1078f6 100755
> > > --- a/tests/qemu-iotests/081
> > > +++ b/tests/qemu-iotests/081
> > > @@ -56,6 +56,9 @@ function run_qemu()
> > > do_run_qemu "$@" 2>&1 | _filter_testdir | _filter_qmp |
> > > _filter_qemu_io
> > > }
> > >
> > > +test_quorum=$(qemu-img --help|grep quorum)
> > > +[ "$test_quorum" = "" ] && _supported_fmt quorum
> > > +
> >
> > How does this disable the test? When quorum is not compiled, the two lines
> > are
> > no-op. Did you test it?
>
> Yes I did test it. It works.
>
Yes, I read it wrong, it should overwrite raw with quorum, so it works.
Still, qemu-img should be written $QEMU_IMG.
Thanks,
Fam