Re: [PATCH RESEND v6 00/36] Initial support for multi-process qemu

[Eric Blake]

On 4/24/20 7:48 AM, Stefan Hajnoczi wrote:
> On Wed, Apr 22, 2020 at 09:13:35PM -0700, address@hidden wrote:
> > There is an error in travis-ci build test which does not get reproduced.
> >
> >   TEST    iotest-qcow2: 041 [fail]
> > QEMU          -- 
> > "/home/travis/build/elena-ufimtseva/qemu-multiprocess/out-of-tree/build/dir/tests/qemu-iotests/../../x86_64-softmmu/qemu-system-x86_64"
> >  -nodefaults -display none -accel qtest

> > +Traceback (most recent call last):
> > +  File "041", line 1049, in test_with_other_parent
> > +    self.assert_qmp(result, 'return', {})
> > +  File 
> > "/home/travis/build/elena-ufimtseva/qemu-multiprocess/tests/qemu-iotests/iotests.py",
> >  line 821, in assert_qmp
> > +    result = self.dictpath(d, path)
> > +  File 
> > "/home/travis/build/elena-ufimtseva/qemu-multiprocess/tests/qemu-iotests/iotests.py",
> >  line 797, in dictpath
> > +    self.fail('failed path traversal for "%s" in "%s"' % (path, str(d)))
> > +AssertionError: failed path traversal for "return" in "{'error': {'class': 'GenericError', 
> > 'desc': "UNIX socket path 
> > '/home/travis/build/elena-ufimtseva/qemu-multiprocess/out-of-tree/build/dir/tests/qemu-iotests/scratch/nbd.sock'
> >  is too long"}}"
>
> UNIX Domain Socket paths have to be 108 characters or less.  The path in
> the failed test case is 110 characters long.  You could rename your
> branch to "mpqemu" to solve this failure.

We recently fixed the iotests to prefer sticking NBD sockets under 
$SOCK_DIR (see commits f0e24942 and friends); did we miss test 41?

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org