qemu-devel
[Top][All Lists]
Advanced

[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



reply via email to

[Prev in Thread] Current Thread [Next in Thread]