Re: [Qemu-block] [PATCH v3 09/13] nbd: pick first exported volume if no export name is requested

2016-01-21 Thread Paolo Bonzini
On 19/01/2016 17:44, Daniel P. Berrange wrote: >> > As a first reaction, I would really avoid magic unless the server >> > provides a single exports. But even in that case, I would prefer to >> > have some synchronization between the server and client command-line. >> > >> > Is an empty

Re: [Qemu-block] [PATCH v3 09/13] nbd: pick first exported volume if no export name is requested

2016-01-21 Thread Daniel P. Berrange
On Thu, Jan 21, 2016 at 11:30:35AM +0100, Paolo Bonzini wrote: > > > On 19/01/2016 17:44, Daniel P. Berrange wrote: > >> > As a first reaction, I would really avoid magic unless the server > >> > provides a single exports. But even in that case, I would prefer to > >> > have some

[Qemu-block] [PATCH v3 09/13] nbd: pick first exported volume if no export name is requested

2016-01-19 Thread Daniel P. Berrange
The NBD client is currently only capable of using the new style protocol negotiation if an explicit export name is provided. This is a problem, because TLS support will require use of the new style protocol in all cases, and we wish to keep the export name as an optional request for backwards

Re: [Qemu-block] [PATCH v3 09/13] nbd: pick first exported volume if no export name is requested

2016-01-19 Thread Paolo Bonzini
On 19/01/2016 14:09, Daniel P. Berrange wrote: > The NBD client is currently only capable of using the new style > protocol negotiation if an explicit export name is provided. > This is a problem, because TLS support will require use of the > new style protocol in all cases, and we wish to keep

Re: [Qemu-block] [PATCH v3 09/13] nbd: pick first exported volume if no export name is requested

2016-01-19 Thread Daniel P. Berrange
On Tue, Jan 19, 2016 at 05:14:32PM +0100, Paolo Bonzini wrote: > > > On 19/01/2016 14:09, Daniel P. Berrange wrote: > > The NBD client is currently only capable of using the new style > > protocol negotiation if an explicit export name is provided. > > This is a problem, because TLS support will