Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-19 Thread Vladimir Sementsov-Ogievskiy
19.01.2018 01:43, Eric Blake wrote: On 01/18/2018 12:11 PM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/201 | 159 + tests/qemu-iotests/201.out | 5 ++ tests/qemu-iotests/group |

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-18 Thread Eric Blake
On 01/18/2018 12:11 PM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > tests/qemu-iotests/201 | 159 > + > tests/qemu-iotests/201.out | 5 ++ > tests/qemu-iotests/group | 1 + > 3 files changed, 165

[Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-18 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/201 | 159 + tests/qemu-iotests/201.out | 5 ++ tests/qemu-iotests/group | 1 + 3 files changed, 165 insertions(+) create mode 100644 tests/qemu-iotests/201 create mode 10064

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-15 Thread Vladimir Sementsov-Ogievskiy
15.01.2018 18:05, Eric Blake wrote: On 01/15/2018 08:40 AM, Vladimir Sementsov-Ogievskiy wrote: And the list archives show several threads of people complaining that ./check failing with a diff that merely shows: -. +..E.. didn't see that. usually, for failed iotests I see -. +..E..

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-15 Thread Eric Blake
On 01/15/2018 08:40 AM, Vladimir Sementsov-Ogievskiy wrote: >> And the list archives show several threads of people complaining that >> ./check failing with a diff that merely shows: >> >> -. >> +..E.. > > didn't see that. usually, for failed iotests I see > > -. > +..E.. > > + some kin

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-15 Thread Vladimir Sementsov-Ogievskiy
12.01.2018 19:54, Eric Blake wrote: On 01/12/2018 05:43 AM, Vladimir Sementsov-Ogievskiy wrote: +++ b/tests/qemu-iotests/201.out @@ -0,0 +1,5 @@ +... +-- +Ran 7 tests I'm not a fan of python tests that are difficult to deb

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-12 Thread Eric Blake
On 01/12/2018 05:43 AM, Vladimir Sementsov-Ogievskiy wrote: >>> +++ b/tests/qemu-iotests/201.out >>> @@ -0,0 +1,5 @@ >>> +... >>> +-- >>> +Ran 7 tests >> I'm not a fan of python tests that are difficult to debug.  Your >> addi

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-12 Thread Vladimir Sementsov-Ogievskiy
09.01.2018 23:49, Eric Blake wrote: On 12/07/2017 09:51 AM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/201 | 130 + tests/qemu-iotests/201.out | 5 ++ tests/qemu-iotests/group |

Re: [Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2018-01-09 Thread Eric Blake
On 12/07/2017 09:51 AM, Vladimir Sementsov-Ogievskiy wrote: > Signed-off-by: Vladimir Sementsov-Ogievskiy > --- > tests/qemu-iotests/201 | 130 > + > tests/qemu-iotests/201.out | 5 ++ > tests/qemu-iotests/group | 1 + > 3 files changed, 136

[Qemu-block] [PATCH v2 6/6] iotest 201: new test for qmp nbd-server-remove

2017-12-07 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- tests/qemu-iotests/201 | 130 + tests/qemu-iotests/201.out | 5 ++ tests/qemu-iotests/group | 1 + 3 files changed, 136 insertions(+) create mode 100644 tests/qemu-iotests/201 create mode 10064