Re: [Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-23 Thread Sascha Silbe
Dear Max, Max Reitz writes: > On 2016-08-23 at 10:17, Sascha Silbe wrote: >> Max Reitz writes: [tests/qemu-iotests/162] >>> Good idea. We can just let the script generate a random port; >>> $(($RANDOM+32768)) should do the trick. >> >> Which will fail just the same as the original version if an

Re: [Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-23 Thread Max Reitz
On 2016-08-23 at 10:17, Sascha Silbe wrote: Dear Max, Max Reitz writes: On 2016-08-23 at 07:44, Sascha Silbe wrote: Max Reitz writes: [tests/qemu-iotests/162] Using a fixed port number means multiple users won't be able to run this in parallel. That it's only open for a short time actuall

Re: [Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-23 Thread Sascha Silbe
Dear Max, Max Reitz writes: > On 2016-08-23 at 07:44, Sascha Silbe wrote: >> Max Reitz writes: [tests/qemu-iotests/162] >> Using a fixed port number means multiple users won't be able to run this >> in parallel. That it's only open for a short time actually makes the >> issue a bit worse as it'

Re: [Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-23 Thread Max Reitz
On 2016-08-23 at 07:44, Sascha Silbe wrote: Dear Max, Max Reitz writes: [tests/qemu-iotests/162] [...] +# (We need to set up a server here, because the error message for "Connection +# refused" does not contain the destination port) +$QEMU_NBD -p 42424 -f raw null-co:// & +sleep 0.5 +$QEMU_I

Re: [Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-23 Thread Sascha Silbe
Dear Max, Max Reitz writes: [tests/qemu-iotests/162] [...] > +# (We need to set up a server here, because the error message for "Connection > +# refused" does not contain the destination port) > +$QEMU_NBD -p 42424 -f raw null-co:// & > +sleep 0.5 > +$QEMU_IMG info 'json:{"driver": "nbd", "host

[Qemu-devel] [PATCH for-2.8] iotests: Do not rely on unavailable domains in 162

2016-08-19 Thread Max Reitz
There are some (mostly ISP-specific) name servers who will redirect non-existing domains to special hosts. In this case, we will get a different error message when trying to connect to such a host, which breaks test 162. 162 needed this specific error message so it can confirm that qemu was indeed