On Wed, Apr 26, 2017 at 19:52:39 +0200, Peter Krempa wrote: > --- > .../generic-disk-network-http.xml | 44 ++++++++++++++++++++++ > tests/genericxml2xmltest.c | 1 + > 2 files changed, 45 insertions(+) > create mode 100644 tests/genericxml2xmlindata/generic-disk-network-http.xml > > diff --git a/tests/genericxml2xmlindata/generic-disk-network-http.xml b/tests/genericxml2xmlindata/generic-disk-network-http.xml > new file mode 100644 > index 000000000..51c779502 > --- /dev/null > +++ b/tests/genericxml2xmlindata/generic-disk-network-http.xml > @@ -0,0 +1,44 @@ > +<domain type='qemu'> > + <name>QEMUGuest1</name> > + <uuid>c7a5fdbd-edaf-9455-926a-d65c16db1809</uuid> > + <memory unit='KiB'>219136</memory> > + <currentMemory unit='KiB'>219136</currentMemory> > + <vcpu placement='static'>1</vcpu> > + <os> > + <type arch='i686' machine='pc'>hvm</type> > + <boot dev='hd'/> > + </os> > + <clock offset='utc'/> > + <on_poweroff>destroy</on_poweroff> > + <on_reboot>restart</on_reboot> > + <on_crash>destroy</on_crash> > + <devices> > + <emulator>/usr/bin/qemu-system-i686</emulator> > + <disk type='network' device='disk'> > + <driver name='qemu' type='raw'/> > + <source protocol='http' name='test.img'> > + <host name='example.org'/> > + </source> > + <target dev='vda' bus='virtio'/> > + </disk> ... I'd think a <readonly/> element would be mandatory for http(s) disks, but apparently it isn't... This patch ends the part of useful refactors and cleanups and my review ends here too. Feel free to push everything I acked. The rest of the series is related to HTTP cookies and some design decisions need to be solved first. BTW, I tend to agree with Dan B. Reviewed-by: Jiri Denemark <jdenemar@xxxxxxxxxx> -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
