Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-19 Thread Daniel P . Berrangé
On Sat, Nov 17, 2018 at 09:01:57PM -0600, Eric Blake wrote: > On 11/16/18 9:53 AM, Daniel P. Berrangé wrote: > > The helpers for starting/stopping qemu-nbd in 058 will be useful in > > other test cases, so move them into a common.nbd file. > > > > Signed-off-by: Daniel P. Berrangé > > --- > > >

Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-19 Thread Daniel P . Berrangé
On Fri, Nov 16, 2018 at 03:43:16PM -0600, Eric Blake wrote: > On 11/16/18 3:41 PM, Eric Blake wrote: > > > > +#!/bin/bash > > > > I know we're using bash, > > > > > + > > > +function nbd_server_stop() > > > +{ > > > > > +function nbd_server_wait_for_unix_socket() > > > > and bash supports

Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-17 Thread Eric Blake
On 11/16/18 9:53 AM, Daniel P. Berrangé wrote: The helpers for starting/stopping qemu-nbd in 058 will be useful in other test cases, so move them into a common.nbd file. Signed-off-by: Daniel P. Berrangé --- +function nbd_server_start_unix_socket() +{ +nbd_server_stop +$QEMU_NBD -v

Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-16 Thread Eric Blake
On 11/16/18 3:41 PM, Eric Blake wrote: +#!/bin/bash I know we're using bash, + +function nbd_server_stop() +{ +function nbd_server_wait_for_unix_socket() and bash supports 'function', but it is an obsolete syntactic sugar thing that I don't recommend using.  (In ksh, it actually makes

Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-16 Thread Eric Blake
On 11/16/18 9:53 AM, Daniel P. Berrangé wrote: The helpers for starting/stopping qemu-nbd in 058 will be useful in other test cases, so move them into a common.nbd file. Signed-off-by: Daniel P. Berrangé --- tests/qemu-iotests/058| 47 +

Re: [Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-16 Thread Eric Blake
On 11/16/18 9:53 AM, Daniel P. Berrangé wrote: The helpers for starting/stopping qemu-nbd in 058 will be useful in other test cases, so move them into a common.nbd file. In fact, I already have a proposed patch 228 that I'm trying to clean up for potential inclusion in 3.1 that could use

[Qemu-block] [PATCH 3/6] tests: pull qemu-nbd iotest helpers into common.nbd file

2018-11-16 Thread Daniel P . Berrangé
The helpers for starting/stopping qemu-nbd in 058 will be useful in other test cases, so move them into a common.nbd file. Signed-off-by: Daniel P. Berrangé --- tests/qemu-iotests/058| 47 + tests/qemu-iotests/common.nbd | 56