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

[Max Reitz]

On 24.04.20 14:53, Eric Blake wrote:
> 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?

Looks more like I broke it.  Oops.  Will fix.

Max

signature.asc
Description: OpenPGP digital signature