On 2/6/19 2:18 PM, Eric Blake wrote:
> Right now, we only use NBD exports during storage migration,
> where we control the NBD client and therefore don't care about
> the export name (qemu's default of naming the export after the
> device is fine). But upcoming patches for exposing backups over
> NBD wants to use a different export name (matching the libvirt
> domain XML rather than the qemu device name); so enhance the
> QMP glue code to allow this flexibility.
> 
> Signed-off-by: Eric Blake <ebl...@redhat.com>
> ---
>  src/qemu/qemu_monitor.h      | 1 +
>  src/qemu/qemu_monitor_json.h | 1 +
>  src/qemu/qemu_migration.c    | 2 +-
>  src/qemu/qemu_monitor.c      | 5 +++--
>  src/qemu/qemu_monitor_json.c | 2 ++
>  tests/qemumonitorjsontest.c  | 2 +-
>  6 files changed, 9 insertions(+), 4 deletions(-)
> 

I assume/hope some day a test will be added that will show what the
@export variable does.  I haven't read ahead at all - I'm trying to just
go in order and get some feedback sooner so you can start pushing stuff.

Reviewed-by: John Ferlan <jfer...@redhat.com>

John

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to