On Tue, 04/22 12:04, Kevin Wolf wrote:
> Am 21.04.2014 um 09:01 hat Fam Zheng geschrieben:
> > On Thu, 04/17 17:48, Kevin Wolf wrote:
> > > A name that is taken by an ID can't be taken by a node-name at the same
> > > time. Check that conflicts are correctly detected.
> > >
> > > Signed-off-by: Kevin Wolf <kw...@redhat.com>
> > > ---
> > > tests/qemu-iotests/087 | 52
> > > ++++++++++++++++++++++++++++++++++++++++++++++
> > > tests/qemu-iotests/087.out | 5 +++++
> > > 2 files changed, 57 insertions(+)
> > >
> > >
> > > diff --git a/tests/qemu-iotests/087.out b/tests/qemu-iotests/087.out
> > > index 479bf86..7fbee3f 100644
> > > --- a/tests/qemu-iotests/087.out
> > > +++ b/tests/qemu-iotests/087.out
> > > @@ -20,6 +20,11 @@ QMP_VERSION
> > > {"return": {}}
> > > {"return": {}}
> > > {"error": {"class": "GenericError", "desc": "Device with id 'disk'
> > > already exists"}}
> > > +{"error": {"class": "GenericError", "desc": "Device with node-name
> > > 'test-node' already exists"}}
> > > +main-loop: WARNING: I/O thread spun for 1000 iterations
> >
> > Is this warning message deterministic? I think we could avoid the guest
> > code by
> > '-machine accel=qtest'.
>
> At least it's deterministic enough that the test case never failed for
> me in multiple runs on two different machines with different OS versions
> (RHEL 6 and 7).
>
> > BTW I intended to write a patch for this but it would have some overlap with
> > Jeff's qemu-iotests common.qemu series:
>
> Then let's commit this as it is and once Jeff's series is in, you can
> fix up everything at once?
>
OK, works for me:
Reviewed-by: Fam Zheng <f...@redhat.com>
