[libvirt] [libvirt-designer 2/3] Accept 'iso' format in gvir_designer_domain_add_disk_full()
Daniel P. Berrange
berrange at redhat.com
Tue Apr 2 10:12:29 UTC 2013
On Tue, Apr 02, 2013 at 11:45:10AM +0200, Christophe Fergeau wrote:
> We need this to be able to mark the device we are creating as a
> CDROM.
> ---
> examples/virtxml.c | 2 +-
> libvirt-designer/libvirt-designer-domain.c | 8 ++++++++
> 2 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/examples/virtxml.c b/examples/virtxml.c
> index d4a5fe2..9a36142 100644
> --- a/examples/virtxml.c
> +++ b/examples/virtxml.c
> @@ -712,7 +712,7 @@ Domain with Fedora 17 from locally stored ISO and one NIC with mac
>
> To add multiple devices just use appropriate argument multiple times:
>
> - # virtxml -d /tmp/Fedora-17-x86_64-Live-KDE.iso,raw \
> + # virtxml -d /tmp/Fedora-17-x86_64-Live-KDE.iso,iso \
> -d /var/lib/libvirt/images/f17.img,qcow2 \
> -i default,mac=00:11:22:33:44:55,link=down \
> -i blue_network \
> diff --git a/libvirt-designer/libvirt-designer-domain.c b/libvirt-designer/libvirt-designer-domain.c
> index 0d47d3c..5da8dd3 100644
> --- a/libvirt-designer/libvirt-designer-domain.c
> +++ b/libvirt-designer/libvirt-designer-domain.c
> @@ -894,6 +894,14 @@ gvir_designer_domain_add_disk_full(GVirDesignerDomain *design,
> gvir_config_domain_disk_set_type(disk, type);
> gvir_config_domain_disk_set_source(disk, path);
> gvir_config_domain_disk_set_driver_name(disk, driver_name);
> + if (g_strcmp0(format, "iso") == 0) {
> + /* FIXME: Should probably reorder the disk devices so that floppies
> + * go first, then disks, then CDROMs
> + */
> + gvir_config_domain_disk_set_guest_device_type(disk,
> + GVIR_CONFIG_DOMAIN_DISK_GUEST_DEVICE_CDROM);
> + format = "raw";
> + }
I don't think it is a good idea to overload 'format' for this purpose.
It is perfectly acceptable to back a CDROM device by a qcow2 files.
I think we should just have a gvir_designer_domain_add_cdrom() method
or some other indicator
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list