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